]> sjero.net Git - wget/blobdiff - src/ChangeLog
Fix compiler warnings
[wget] / src / ChangeLog
index 43997f88d5ae2111616c698e72e4682e7d341914..f07ddc5f2000e6b724b754d39e26a8974403082b 100644 (file)
@@ -1,3 +1,245 @@
+2014-05-03  Tim Ruehsen  <tim.ruehsen@gmx.de>
+
+       * ftp-ls.c (ftp_parse_vms_ls): Explicitly typecast strlen's output
+       * ftp.c (getftp): Prevent declaration of shadow variable
+       * html-url.c (tag_handle_base, tag_handle_form, tag_handle_link,
+       tag_handle_meta): Compiler directive to ignore unused variable
+       * html-url.h (cleanup_html_url): Add function declaration
+       * http-ntlm.c (ntlm_input, mkhash): Use ssize_t to declare variables that store
+       sizes.
+       (mkhash): Explicitly typecast output of c_toupper
+       (short_pair): Add typecasts to prevent compiler warnings
+       (ntlm-output): Fix datatypes of various variables
+       * http.c (gethttp): Prevent declaration of shadow variable err
+       (gethttp): remove unreachable code
+       (test_parse_content_disposition): Fix variable declarations and use
+       countof() macro
+       * init.c (run_command): Prevent declaration of ghost variable
+       (cmd_string, cmd_string_uppercase, cmd_file, cnd_vector,
+       cmd_directory_vector, cmd_spec_dirstruct, cmd_spec_header,
+       cmd_spec_warc_header, cmd_spec_htmlify, cmd_spec_mirror,
+       cmd_spec_prefer_family, cmd_spec_progress, cmd_spec_recursive,
+       cmd_spec_regex_type, cmd_spec_restrict_file_names, cmd_spec_report_speed,
+       cmd_spec_timeout, cmd_spec_useragent, cmd_spec_verbose): Add compiler
+       directive to ignore unused variable
+       (cleanup_html_url, spider_cleanup): Remove declarations
+       (test_commands_sorted): Cleanup code
+       (test_cmd_spec_restrict_file_names): Use correct data types
+       * iri.c (remote_to_utf8): Use more verbose variable name
+       * main.c (init_switches): Prevent declaration of ghost variable
+       * netrc.c (netrc_cleanup): Create cleanup function on common naming scheme
+       * netrc.h (netrc_cleanup): Declare function
+       * openssl.c (openssl_write, openssl_errstr): Compiler directive to ignore
+       unused parameter
+       (openssl_errstr): Explcicitly typecasr output of ASN1_STRING_length
+       * options.h (struct options): Declare includes and excludes as const char **
+       * progress.c (progress_interactive_p, progress_handle_sigwinch): Compiler
+       directive to ignore unused parameter
+       * res.c (test_is_robots_txt_url): Fix datatypes
+       * retr.c (line_terminator): Add compiler directive for ignoring unused
+       paramter. Remove unused variable.
+       * spider.h (spider_cleanup): Declare function
+       * test,c (main): declare unused paramter
+       * test.h (test_*): Declare functions
+       * url.c (url_string): Explicit typecast of password strings
+       (run_test): Declare *test, struct tests and *expected_result as const
+       (test_path_simplify, test_append_uri_pathl, test_are_urls_equal): Fix datatypes
+       * utls.c (fork_to_background): Be more verbose when errors occur
+       (dir_matches_p, test_dir_matches_p): Declare char * as const as required
+       (base64_encode, base64_decode, get_max_length): Set correct return type
+       (match_pcre_regex, match_posix_regex): Use correct datatypes and typecasts
+       (test_subdir_p): static const struct
+       * utils.h (base64_encode, base64_decode, get_max_length): Fix return type
+       * warc.c (ward_write_cdx_record): unused parameter
+       (struct hash_table *warc_cdx_dedup_table): Declare as static
+
+2014-05-01  Benjamin Goose  <gans+wget@tngtech.com>
+
+       * http.c: Fix small memory leak
+
+2014-05-01  Darshit Shah  <darnir@gmail.com> (tiny change)
+
+       * progress.c (dot_finish): Do not print extra newlines when not in verbose
+       mode. (Purely aesthetic change)
+       (get_eta): Add extra space when eta is printed.
+       (create_image): Remove erroneous space from being added to progress bar when
+       filename > MAX_FILENAME_LEN
+       (create_image): Remove extra space before printed download speeds
+       Make the filename a fixed width column.
+
+2014-04-19  Darshit Shah  <darnir@gmail.com>
+
+       * log.h (log_options): Add new logging options, LOG_PROGRESS. All progress
+       bar related output should use LOG_PROGRESS.
+       * log.c (CHECK_VERBOSE): Implement LOG_PROGRESS output
+       * progress.c (dot_create, print_row_stats, dot_update, dot_finish,
+       bar_finish, display_image): Output progress information through LOG_PROGRESS
+       (progress_implementation, dot_create, bar_create, progress_create): The
+       progress bar create functions accept an extra paramter for the filename of the
+       local file
+       (bar_progress): Add new variable to store filename of currently downloading
+       file
+       (bar_finish): Aesthetic change. Print two newlines if in verbose mode, else
+       only one.
+       (MACRO): Define new macro, MIN
+       (create_image): Implement displaying filename in progress bar output
+       Filename tick implementation copied from Giuseppe's patch on parallel-wget
+       * progress.h (progress_create): Accept another parameter for filename
+       * http.c (gethttp): Remove unnecessary conditional
+       (read_response_body): Send local filename to fd_read_body so that it can be
+       printed on the progress bar
+       * main.c (option_data): Add new switch, --show-progress
+       (main): If in verbose output, show progress bar by default
+       (main): Set progress implemetation when displaying progress bar
+       (no_prefix): Increase buffer size to 2048 to prevent overflows
+       * init.c (commands): Add new command, showprogress
+       (defaults): By default initialize show_progress to false
+       * options.h (options): Add new option, show_progress
+       * retr.c (fd_read_body): Accept new parameter for filename of currently
+       downlaoding file
+       (fd_read_body): Create and update progress bar when opt.show_progress is set
+       (fd_read_body): Display progress information in Windows console titlebars
+       * retr.h (fd_read_body): Update declaration
+       * ftp.c (getftp): Send filename of the local file so that it can be printed
+       with the progress bar.
+
+2013-11-02  Giuseppe Scrivano  <gscrivan@redhat.com>
+
+       * progress.c (struct progress_implementation): New method draw.
+       (bar_update): Split into...
+       (bar_draw): ...this new function.
+       (dot_update): Split into...
+       (dot_draw): New function.
+       (progress_update): Also invoke draw on `current_impl'.
+
+2014-04-22  Tim Ruehsen  <tim.ruehsen@gmx.de>
+
+       * http.c (gethttp): Fix 204 response handling
+
+2014-03-26  Darshit Shah  <darnir@gmail.com>
+
+       * ftp.c (getftp): Rearrange parameters to fix compiler warning
+       * utils.c (get_dir_matches_p): Do not pass a const char** to a function that
+       expects char**
+
+2014-03-25  Daniel Stenberg  <daniel@haxx.se>
+
+       * url.c (shorten_length): Remove unused function.
+
+2014-03-19  Yousong Zhou  <yszhou4tech@gmail.com>
+
+       * init.c, main.c, options.h: Add option --start-pos for specifying
+       start position of a download.
+       * http.c: Utilize opt.start_pos for HTTP download.
+       * ftp.c: Utilize opt.start_pos for FTP retrieval.
+
+2014-03-04  Giuseppe Scrivano  <gscrivan@redhat.com>
+
+       * http.c (modify_param_value, extract_param): Aesthetic change.
+
+2014-02-14  Vladimír Pýcha  <vpycha@gmail.com> (tiny change)
+
+       * http.c (parse_content_disposition, extract_param)
+       (append_value_to_filename, digest_authentication_encode): URL-decode the
+       filename parameter of Content-Disposition HTTP header if it is encoded. This
+       is related to --content-disposition.
+       New parameter of extract_param(), "is_url_encoded".
+       Add argument NULL to the call of extract_param() in
+       digest_authentication_encode().
+       * http.h: Add the new parameter to the declaration of extract_param().
+       * cookies.c (parse_set_cookie, test_cookies): Add argument NULL to the calls
+       of extract_param().
+       * url.c (url_unescape): Remove "static" modifier.
+       * url.h: Add declaration of url_unescape().
+
+2014-02-06  Giuseppe Scrivano  <gscrivan@redhat.com>
+
+       * main.c (print_version): Move copyright year out of the localized
+       string and update it.
+
+2014-01-29  Darshit Shah  <darnir@gmail.com>
+
+       * main.c: Remove pre-processor variable WHEN_DEBUG
+       (option_data[]): Do not fail on --debug even if debug support is not
+       compiled in.
+       (main): Explicitly set opt.debug to false in case debugging support was not
+       compiled.
+       * init.c (commands[]): Support --debug wven when support is not compiled in.
+       * options.h: Same
+
+2014-01-17  Darshit Shah  <darnir@gmail.com>
+
+       * init.c (commands[]): Add --no-config.
+       * options.h: Same.
+       * main.c (option_data[]): Same.
+       (print_help): Same.
+       (main): If --no-config is set, then do not read the wgetrc files.
+
+2014-01-05  Håkon Vågsether <hauk142@gmail.com> (tiny change)
+
+       * http.c (http_loop): Fix checking the URL length when filename is
+       specified.
+
+2013-12-29  Giuseppe Scrivano  <gscrivan@redhat.com>
+
+       * init.c (home_dir): Remove useless 'if'.
+       * warc.c (warc_start_new_file): Likewise.
+       (warc_process_cdx_line): Likewise.
+       (warc_write_response_record): Likewise.
+
+2013-12-26  Tim Ruehsen  <tim.ruehsen@gmx.de>
+
+       * gnutls.c (ssl_connect_wget): Fix connect timeout failure
+
+2013-11-02  Giuseppe Scrivano  <gscrivan@redhat.com>
+
+       * http.c (gethttp): Increase max header value length to 512.
+
+2013-01-30  Pavel Mateja  <pavel@netsafe.cz> (tiny change)
+
+       * http.c (gethttp): Specify "Host" for CONNECT method.
+
+2013-10-30  Giuseppe Scrivano  <gscrivan@redhat.com>
+
+       * http.c (skip_short_body): Remove assert which is always true.
+       Reported by: David Binderman <dcb314@hotmail.com>
+
+2013-10-17  Andrea Urbani  <matfanjol@mail.com>
+
+       * ftp.c (getftp): force "LIST" or "LIST -a"
+         according to the remote system type. If the remote
+         system is not known, it tries, only the first time,
+         "LIST -a", after "LIST" and decides which one to use.
+         For more information look for "__LIST_A_EXPLANATION__"
+       * ftp.h (enum ustype): New ustype enum.
+       * ftp.h (ftp_syst): New enum ustype *unix_type parameter.
+       * ftp.h (ftp_list): Removed enum stype rs parameter, added.
+         bool avoid_list_a, bool avoid_list, bool *list_a_used
+         parameters.
+       * ftp.h (wget_ftp_fstatus): New AVOID_LIST_A, AVOID_LIST,
+         LIST_AFTER_LIST_A_CHECK_DONE values.
+       * ftp-basic.c (ftp_list): it handles the new avoid_list_a,
+         avoid_list and list_a_used parameters.
+       * ftp.h (ftp_syst): it stores information about the "215 UNIX"
+         systems into the new unix_type parameter.
+
+2013-10-26  Bykov Aleksey <gnfalex@rambler.ru>
+
+       * utils.c (match_tail): Fix cookies reject
+
+       * ftp-ls.c (ftp_parse_unix_ls): Fix parsing month name in uppercase
+
+2013-09-09  Tim Ruehsen  <tim.ruehsen@gmx.de>
+
+       * gnutls.c (ssl_connect_wget): changed checking of option "PFS"
+         to be better prepared for some kinds of backports.
+         Reported by: Daniel Kahn Gillmor <dkg@fifthhorseman.net>
+
+2013-10-10  Giuseppe Scrivano  <gscrivan@redhat.com>
+
+       * url.c (url_parse): Try to convert UTF-8 URLs to IDN.
+       * html-url.c (append_url): Parse URLs specifying an IRI structure.
+
 2013-09-13  Tim Ruehsen  <tim.ruehsen@gmx.de>
 
        * recur.c (download_child_p): fix compile error when