* keyboard.c (read_key_sequence): Don't check last_real_key_start redundantly.
[bpt/emacs.git] / lib-src / ChangeLog
index 698e7b6..f594efa 100644 (file)
@@ -1,5 +1,154 @@
+2011-03-27  Glenn Morris  <rgm@gnu.org>
+
+       * emacsclient.c: Replace SIGTYPE with void.
+
+2011-03-23  Juanma Barranquero  <lekktu@gmail.com>
+
+       * ntlib.c: Include <ctype.h>.
+
+2011-03-23  Glenn Morris  <rgm@gnu.org>
+
+       * Makefile.in ($(DESTDIR)${archlibdir}):
+       Use `install-sh -d' rather than mkinstalldirs.
+
+2011-03-23  Paul Eggert  <eggert@cs.ucla.edu>
+
+       * ebrowse.c: Use size_t, not int, for sizes.
+       This avoids a warning with gcc -Wstrict-overflow, and works
+       better for very large objects.
+       (inbuffer_size): Now size_t.  All uses changed.
+       (xmalloc, xrealloc, operator_name, process_file): Use size_t for
+       sizes.  Don't bother testing whether a size_t value can be negative.
+
+       * etags.c (Ada_funcs): Redo slightly to avoid overflow warning.
+
+       etags: In Prolog functions, don't assume int fits in size_t.
+       This avoids a warning with gcc -Wstrict-overflow.
+       * etags.c (Prolog_functions, prolog_pr, prolog_atom): Use size_t,
+       not int, to store sizes.
+       (prolog_atom): Return 0, not -1, on error.  All callers changed.
+
+       update-game-score: fix bug with -r
+       * update-game-score.c (main): Don't set 'scores' to garbage when
+       -r is specified and scorecount != MAX_SCORES (Bug#8310).  This bug
+       was introduced in the 2002-04-10 change, and was found with gcc
+       -Wstrict-overflow (GCC 4.5.2, x86-64).
+
+       fakemail: Remove dependency on ignore-value.
+       This undoes some of the recent fakemail-related changes.
+       It is made possible due to recent changes to gnulib's stdio module.
+       * Makefile.in (fakemail${EXEEXT}): Do not depend on ignore-value.h.
+       * fakemail.c: Do not include ignore-value.h.
+       (put_line): Do not use ignore_value.
+
+2011-03-03  Drake Wilson  <drake@begriffli.ch>  (tiny change)
+
+       * emacsclient.c (longopts): Add quiet.
+       (decode_options): Handle q/quiet.
+       (print_help_and_exit): Add q/quiet.
+       (main): Suppress some messages if quiet option is used.
+
+2011-02-26  Eli Zaretskii  <eliz@gnu.org>
+
+       * Makefile.in (fakemail${EXEEXT}): Depend on lib/ignore-value.h.
+
+       * emacsclient.c (xstrdup) [WINDOWSNT]: Function added back.
+       (w32_getenv): Use xstrdup to return all values in malloc'ed
+       storage.
+
+2011-02-26  Paul Eggert  <eggert@cs.ucla.edu>
+
+       * ebrowse.c (parse_qualified_param_ident_or_type): Make it clear
+       to reader (and to the compiler) that the loop always executes at
+       least once.  This prevents a warning with recent GCC.
+       (BROWSE_STRUCT): Remove unused macro.
+
+       * fakemail.c: Include <ignore-value.h>.
+       (put_line): Explicitly ignore fwrite return value, for benefit of
+       recent glibc + gcc.
+       (close_the_streams): Diagnose output errors instead of merely
+       exiting with nonzero status.
+       (my_fclose, main): Diagnose input errors, and exit with nonzero status.
+       Formerly, input errors were silently ignored.
+
+       * ebrowse.c (putstr): Rename from PUTSTR and turn into a function.
+       All callers changed.  This is cleaner, and avoids GCC warnings about
+       passing NULL to fputs.
+       (insert_keyword): Rename parameter to avoid shadowing diagnostic.
+
+2011-02-25  Paul Eggert  <eggert@cs.ucla.edu>
+
+       * emacsclient.c (main): Avoid dangling 'if'.
+       (xstrdup): Remove; no longer needed.
+       (get_current_dir_name, w32_getenv, get_server_config, find_tty):
+       (set_local_socket, main):
+       Use const char *, not char *, for pointers that are not assigned
+       through.
+       (IF_LINT): New macro.
+       (set_local_socket, main): Use it to suppress warnings with
+       GCC -Wuninitialized.
+
+       * emacsclient.c: Redo local variables to avoid shadowing problems.
+       (message, socket_status, start_daemon_and_retry_set_socket):
+       Rename locals.
+       (main): Move decl of "i".
+
+       * etags.c (ISUPPER): Move to inside the only #ifdef where it's used.
+       This avoids an unused-macro warning with some GCC settings.
+
+       * make-docfile.c (write_globals): Change char * to char const *
+       to avoid a GCC "assignment discards qualifiers" diagnostic
+       in some configurations.
+       (scan_c_file): Refactor local variable decls to make their scope
+       more accurate and to avoid a GCC -Wuninitialized diagnostic.
+
+2011-02-22  Eli Zaretskii  <eliz@gnu.org>
+
+       * etags.c (canonicalize_filename, ISUPPER): Fix last change.
+
+       * makefile.w32-in ($(BLD)/ebrowse.$(O), $(BLD)/pop.$(O)): Depend
+       on ../lib/min-max.h.
+
+2011-02-22  Paul Eggert  <eggert@cs.ucla.edu>
+
+       etags: Downcase drive letters, for consistency with Emacs proper.
+       * etags.c (upcase): Remove; no longer used.
+       (canonicalize_filename): Downcase drive letters.
+
+       Assume S_ISLNK etc. work, since gnulib supports this.
+       * etags.c (S_ISREG): Remove.
+
+2011-02-22  Paul Eggert  <eggert@cs.ucla.edu>
+
+       Assume S_ISLNK etc. work, since gnulib supports this.
+       * etags.c (S_ISREG): Remove.
+
+2011-02-22  Juanma Barranquero  <lekktu@gmail.com>
+
+       * makefile.w32-in (obj): Remove filemode.o.
+
 2011-02-21  Paul Eggert  <eggert@cs.ucla.edu>
 
+       New file "lib/min-max.h".
+       * ebrowse.c (min, max): Define them by including <min-max.h>
+       instead of defining it ourselves.
+       * pop.c (min): Likewise.
+       * Makefile.in (ebrowse${EXEEXT}, pop.o): Depend on min-max.h.
+
+       * movemail.c (popmail): Report fchown failure instead of ignoring it.
+       But if the file already has the right ownership, don't worry about it.
+
+       * make-docfile.c (input_buffer): Rename variables to avoid shadowing.
+       * test-distrib.c (buf): Make this local, to avoid shadowing.
+
+       * movemail.c (main, pop_retr): Rename locals to avoid shadowing.
+       (progname, sfi, sfo, ibuffer, obuffer): Remove unused vars.
+       (DONE): Remove unused macro.
+       (DIRECTORY_SEP, IS_DIRECTORY_SEP, IS_FROM_LINE):
+       Define these macros only in the contexts that need them.
+       * pop.c (index): Remove unused macro.
+       (KPOP_PORT): Define only if KERBEROS is defined.
+
        Declare file-scope functions and variables static if not exported.
        This is more consistent, and is nicer with gcc -Wstrict-prototypes.
        * ebrowse.c, emacsclient.c, fakemail.c, make-docfile.c, movemail.c:
        Declare non-'main' functions and variables to be static.
        * ebrowse.c: Omit redundant function prototypes.
 
+2011-02-21  Eli Zaretskii  <eliz@gnu.org>
+
+       * makefile.w32-in ($(BLD)/ctags.$(O), $(BLD)/emacsclient.$(O))
+       ($(BLD)/etags.$(O), $(BLD)/movemail.$(O), $(BLD)/ntlib.$(O)):
+       Depend on $(EMACS_ROOT)/nt/inc/sys/stat.h.
+
+2011-02-21  Ben Key  <bkey76@gmail.com>
+
+       * make-docfile.c (scan_c_file): Adapt DEFVAR_PER_BUFFER case to
+       the new BVAR macro.
+
 2011-02-20  Juanma Barranquero  <lekktu@gmail.com>
 
        * makefile.w32-in (obj): Remove md5.o.
 
 2011-02-05  Paul Eggert  <eggert@cs.ucla.edu>
 
-       * emacsclient.c: conform to C89 pointer rules
+       * emacsclient.c: Conform to C89 pointer rules.
        (file_name_absolute_p): Accept const char *, not const unsigned
        char *, to satisfy C89 rules.
 
        * test-distrib.c (cool_read):
        * movemail.c (main, concat):
        * make-docfile.c (scan_file, write_c_args):
-       * emacsclient.c (get_server_config): Fix -Wconversion warning.
+       * emacsclient.c (get_server_config): Fix -Wconversion warning.
        (egetenv): Move conditional definition earlier.
        (progname): Use const.
        * sorted-doc.c (xstrdup): Use const.
        autoconf, not cpp.
        (ALL_CFLAGS): Use them as make variables.
 
-2010-04-07  Christoph  <cschol2112@googlemail.com>  (tiny change)
+2010-04-07  Christoph Scholtes  <cschol2112@googlemail.com>
 
        * makefile.w32-in (OTHER_PLATFORM_SUPPORT): Use parenthesis
        for macros for nmake compatibility.