* charset.c: Include <limits.h>.
[bpt/emacs.git] / src / ChangeLog
index 3288a79..39f8062 100644 (file)
@@ -1,8 +1,167 @@
+2011-03-08  Paul Eggert  <eggert@cs.ucla.edu>
+
+       * xmenu.c (menu_highlight_callback): Now static.
+       (set_frame_menubar): Remove unused local.
+       (xmenu_show): Rename parameter to avoid shadowing.
+       (xmenu_show, xdialog_show, xmenu_show): Make local pointers "const"
+       since they might point to immutable storage.
+       (next_menubar_widget_id): Declare only if USE_X_TOOLKIT,
+       since it's unused otherwise.
+
+       * xdisp.c (produce_glyphless_glyph): Initialize lower_xoff.
+       Add a FIXME comment, since the code still doesn't look right.
+       (Fcurrent_bidi_paragraph_direction): Simplify slightly; this
+       avoids a gcc -Wuninitialized diagnostic.
+       (display_line, BUILD_COMPOSITE_GLYPH_STRING, draw_glyphs):
+       (note_mouse_highlight): Mark variables that gcc -Wuninitialized
+       does not deduce are never used uninitialized.
+
+       * lisp.h (IF_LINT): New macro, copied from ../lib-src/emacsclient.c.
+
+       * xdisp.c (redisplay_window): Rename local to avoid shadowing.
+       * window.c (window_loop, size_window):
+       (run_window_configuration_change_hook, enlarge_window): Likewise.
+
+       * window.c (display_buffer): Now static.
+       (size_window): Mark variables that gcc -Wuninitialized
+       does not deduce are never used uninitialized.
+       * window.h (check_all_windows): New decl, to forestall
+       gcc -Wmissing-prototypes diagnostic.
+
+       * charset.h (CHECK_CHARSET_GET_CHARSET): Rename locals to avoid
+       shadowing.
+       * charset.c (map_charset_for_dump, Fchar_charset): Likewise.
+       Include <limits.h>.
+       (Fsort_charsets): Redo min/max calculation to shorten the code a bit
+       and to avoid gcc -Wuninitialized warning.
+
+2011-03-06  Chong Yidong  <cyd@stupidchicken.com>
+
+       * xdisp.c (redisplay_window): Revert incorrect logic in 2011-03-06
+       change.
+
+2011-03-06  Paul Eggert  <eggert@cs.ucla.edu>
+
+       current_column: Now returns EMACS_INT, fixing some iftc
+       that was introduced in the 2002-06-02 change "temporarily"; see
+       <http://lists.gnu.org/archive/html/emacs-devel/2002-06/msg00039.html>.
+       * bytecode.c (Fbyte_code): Don't cast current_column () to int.
+       * cmds.c (internal_self_insert): Likewise.
+       * indent.c (Fcurrent_column): Likewise.
+       * keymap.c (describe_command): Likewise.
+       * minibuf.c (read_minibuf): Likewise.
+       * indent.c (Fcurrent_indentation): Don't cast position_indentation ()
+       to int.
+       * xdisp.c (redisplay_internal, redisplay_window, decode_mode_spec):
+       Likewise.
+       * cmds.c (internal_self_insert): Declare locals to be EMACS_INT,
+       not int or double, if they might contain a column number.
+       * indent.c (current_column, Findent_to, indented_beyond_p):
+       (compute_motion, vmotion): Likewise.
+       * keymap.c (describe_command): Likewise.
+       * xdisp.c (pint2str): Likewise.
+       * indent.c (last_known_column): Now EMACS_INT, not int.
+       * minibuf.c (minibuf_prompt_width): Likewise.
+       * indent.c (current_column, current_column_1, position_indentation):
+       Return EMACS_INT, not double.
+       * lisp.h (current_column): Likewise.
+       * indent.c (indented_beyond_p): Last arg is now EMACS_INT, not double.
+       All callers changed.
+       * lisp.h (indented_beyond_p): Likewise.
+
+       * minibuf.c (minibuf_prompt, minibuf_prompt_width): Move here
+       from xdisp.c, and make static, since these are used only here.
+       * window.h, xdisp.c (minibuf_prompt, minibuf_prompt_width):
+       Remove decls.
+
+       * cmds.c (internal_self_insert): Reindent to match Emacs style.
+       * xdisp.c (redisplay_window): Likewise.
+
+       * xdisp.c: Rename or move local decls to avoid shadowing.
+       (init_iterator, handle_fontified_prop, handle_single_display_spec):
+       (message_dolog, message_with_string, redisplay_internal):
+       (redisplay_window, try_window_reusing_current_matrix, try_window_id):
+       (compute_line_metrics, highlight_trailing_whitespace, cursor_row_p):
+       (display_line, display_string, rows_from_pos_range):
+       (mouse_face_from_buffer_pos, note_mouse_highlight, expose_frame):
+       Rename or move local decls.
+       * xdisp.c (BUILD_GLYPHLESS_GLYPH_STRING): Omit unused local var.
+       (produce_glyphless_glyph): Make a pointer "const"
+       since it might point to immutable storage.
+       (update_window_cursor):  Now static, since it's not used elsewhere.
+       (SKIP_GLYPHS): Removed unused macro.
+
+2011-03-06  Michael Shields  <shields@msrl.com>  (tiny change)
+
+       * window.c (Fnext_window): Doc fix.  (Bug#5567)
+
+2011-03-05  Chong Yidong  <cyd@stupidchicken.com>
+
+       * nsterm.m (ns_draw_window_cursor): Fix typo in 2011-02-23 commit.
+
+2011-03-02  Ken Brown  <kbrown@cornell.edu>
+
+       * sheap.c (STATIC_HEAP_SIZE): Increase to 13MB.
+
+2011-03-02  Paul Eggert  <eggert@cs.ucla.edu>
+
+       Work around some portability problems with symlinks.
+
+       * fileio.c (Fmake_symbolic_link): Treat ENOSYS specially, and
+       generate a special message for it.  Suggested by Eli Zaretskii in
+       <http://lists.gnu.org/archive/html/emacs-devel/2011-02/msg00995.html>.
+       (Frename_file, Fmake_symbolic_link, Ffile_symlink_p):
+       Simplify the code by assuming that the readlink and symlink calls
+       exist, even if they always fail on this host.
+       (Ffile_readable_p): Likewise, for fifos.
+       * config.in: Regenerate.
+
+2011-02-27  Chong Yidong  <cyd@stupidchicken.com>
+
+       * frame.c (store_frame_param): Don't store value directly in
+       buffer_list and buried_buffer_list; copy the list and remove dead
+       buffers (Bug#7898).
+
 2011-02-27  Eli Zaretskii  <eliz@gnu.org>
 
        * msdos.c (readlink) [DJGPP < 2.04]: New stub function.
 
-       * msdos.h: (readlink) [DJGPP < 2.04]: Declare prototype.
+       * msdos.h (readlink) [DJGPP < 2.04]: Declare prototype.
+
+       * w32.c (symlink, readlink): New stub functions.
+
+2011-02-27  Paul Eggert  <eggert@cs.ucla.edu>
+
+       * scroll.c (CHECK_BOUNDS): #define only if GLYPH_DEBUG.
+       This avoids a gcc warning in some configurations.
+
+       * frame.c (x_set_screen_gamma): Rename local to avoid shadowing.
+
+       * frame.h: Avoid gcc -Wmissing-prototypes diagnostics.
+       (set_menu_bar_lines, x_get_resource_string): New decls.
+       * msdos.c (set_menu_bar_lines): Omit decl.
+
+       * dispextern.h (struct glyph): Make u.img_id int, not unsigned.
+       It's always given int values and used as an int.  This suppresses
+       a gcc "comparison of unsigned expression >= 0" warning in some
+       configurations.
+
+       * dispnew.c: Rename locals to avoid shadowing.
+       (update_text_area, scrolling_window, update_frame_1): Rename locals.
+
+2011-02-26  Paul Eggert  <eggert@cs.ucla.edu>
+
+       * dispnew.c: Fix problems uncovered by gcc -Wstrict-prototypes.
+       (copy_glyph_row_contents): Remove; not used.
+       (frame_row_to_window, check_current_matrix_flags):
+       (window_change_signal): Now static, since they're not used elsewhere.
+       (check_current_matrix_flags): Surround with "#if 0", since its
+       only use is in a comment.  Maybe both the comment and the "#if 0"
+       stuff should be removed?
+
+       * dispnew.c: Fix problem uncovered by gcc -Wunused-variable.
+       (adjust_frame_glyphs_for_window_redisplay): Make 'w' local to the
+       contexts that actually need it.
 
 2011-02-26  Eli Zaretskii  <eliz@gnu.org>
 
        (x_underline_at_descent_line): Remove declaration.
        (syms_of_nsterm): Remove & from DEFVAR_LISP and DEFVAR_BOOL.
 
-       * nsselect.m (Vns_sent_selection_hooks, Vns_lost_selection_hooks
+       * nsselect.m (Vns_sent_selection_hooks, Vns_lost_selection_hooks)
        (Vselection_alist, Vselection_converter_alist): Move to globals.h.
        (syms_of_nsselect): Remove & from DEFVAR_LISP.
 
        (Qglyphless_char_display, Qhexa_code, Qempty_box, Qthin_space)
        (Qzero_width): New variables.
        (THIN_SPACE_WIDTH): New macro.
-       (lookup_glyphless_char_display): New funciton.
+       (lookup_glyphless_char_display): New function.
        (last_glyphless_glyph_frame, last_glyphless_glyph_face_id)
        (last_glyphless_glyph_merged_face_id): New variables.
        (get_next_display_element): Check glyphless characters.
 2010-10-12  Jan Djärv  <jan.h.d@swipnet.se>
 
        * nsterm.m (Qleft): Declare.
-       (ns_right_alternate_modifier): New variable
+       (ns_right_alternate_modifier): New variable.
        (NSRightAlternateKeyMask): New define.
        (EV_MODIFIERS): Parse NSRightAlternateKeyMask if
        ns_right_alternate_modifier isn't Qleft.
        * Makefile.in (temacs): Use $(ALL_CFLAGS) on the link line.
        (PROFILING_LDFLAGS): Remove, not needed anymore.
 
-       * Makefile.in: Use $(...) everywhere instead of ${...}
+       * Makefile.in: Use $(...) everywhere instead of ${...}.
        (CRT_DIR): Move near potential user.
        (START_FILE): Move near CRT_DIR, it might use it.
 
 
        * eval.c: Fix indentation problem.
 
-       * keyboard.c: Include "process.h"
+       * keyboard.c: Include "process.h".
 
        * eval.c: Remove obsolete noinline declaration.
        * fns.c: Likewise.
        Qtool_bar_style.  Initialize current_tool_bar_style to nil.
        defsubr Stool_bar_get_system_style.  Fprovide on
        dynamic-setting.
-       Move misplaced HAVE_GCONF
+       Move misplaced HAVE_GCONF.
 
        * xsettings.h (Ftool_bar_get_system_style): Declare.