+2011-03-09 Paul Eggert <eggert@cs.ucla.edu>
+
+ * xterm.h (x_mouse_leave): New decl.
+
+ * xterm.c (x_copy_dpy_color, x_focus_on_frame, x_unfocus_frame):
+ Remove unused functions.
+ (x_shift_glyphs_for_insert, XTflash, XTring_bell):
+ (x_calc_absolute_position): Now static.
+ (XTread_socket): Don't define label "out" unless it's used.
+ Don't declare local "event" unless it's used.
+ (x_iconify_frame, x_free_frame_resources): Don't declare locals
+ unless they are used.
+ (XEMBED_VERSION, xembed_set_info): Don't define unless needed.
+ (x_fatal_error_signal): Remove; not used.
+ (x_draw_image_foreground, redo_mouse_highlight, XTmouse_position):
+ (x_scroll_bar_report_motion, handle_one_xevent, x_draw_bar_cursor):
+ (x_error_catcher, x_connection_closed, x_error_handler):
+ (x_error_quitter, xembed_send_message, x_iconify_frame):
+ (my_log_handler): Rename locals to avoid shadowing.
+ (x_delete_glyphs, x_ins_del_lines): Mark with NO_RETURN.
+ (x_connection_closed): Tell GCC not to suggest NO_RETURN.
+
+ * xfaces.c (clear_face_cache, Fx_list_fonts, Fface_font): Rename
+ or move locals to avoid shadowing.
+ (tty_defined_color, merge_face_heights): Now static.
+ (free_realized_faces_for_fontset): Remove; not used.
+ (Fx_list_fonts): Mark variable that gcc -Wuninitialized
+ does not deduce is never used uninitialized.
+ (STRDUPA, LSTRDUPA, FONT_POINT_SIZE_QUANTUM): Remove; not used.
+ (LFACEP): Define only if XASSERTS, as it's not needed otherwise.
+
+ * terminal.c (store_terminal_param): Now static.
+
+ * 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.
+ * dispextern.h (bidi_dump_cached_states): Likewise.
+
+ * 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.
+ (load_charset_map): Mark variables that gcc -Wuninitialized
+ does not deduce are never used uninitialized.
+ (load_charset): Abort instead of using uninitialized var.
+
+ * coding.c (coding_set_source, coding_set_destination):
+ Use "else { /* comment */ }" rather than "else /* comment */;"
+ for clarity, and to avoid gcc -Wempty-body warning.
+ (Fdefine_coding_system_internal): Don't redeclare 'i' inside
+ a block, when the outer 'i' will do.
+ (decode_coding_utf_8, decode_coding_utf_16, detect_coding_emacs_mule):
+ (emacs_mule_char, decode_coding_emacs_mule, detect_coding_iso_2022):
+ (decode_coding_iso_2022, decode_coding_sjis, decode_coding_big5):
+ (decode_coding_raw_text, decode_coding_charset, get_translation_table):
+ (Fdecode_sjis_char, Fdefine_coding_system_internal):
+ Rename locals to avoid shadowing.
+ * character.h (FETCH_STRING_CHAR_ADVANCE): Likewise.
+ * coding.c (emacs_mule_char, encode_invocation_designation):
+ Now static, since they're not used elsewhere.
+ (decode_coding_iso_2022): Add "default: abort ();" as a safety check.
+ (decode_coding_object, encode_coding_object, detect_coding_system):
+ (decode_coding_emacs_mule): Mark variables that gcc
+ -Wuninitialized does not deduce are never used uninitialized.
+ (detect_coding_iso_2022): Initialize a local variable that might
+ be used uninitialized. Leave a FIXME because it's not clear that
+ this initialization is needed.
+ (ISO_CODE_LF, ISO_CODE_CR, CODING_ISO_FLAG_EUC_TW_SHIFT):
+ (ONE_MORE_BYTE_NO_CHECK, UTF_BOM, UTF_16_INVALID_P):
+ (SHIFT_OUT_OK, ENCODE_CONTROL_SEQUENCE_INTRODUCER):
+ (ENCODE_DIRECTION_R2L, ENCODE_DIRECTION_L2R):
+ Remove unused macros.
+
+ * category.c (hash_get_category_set): Remove unused local var.
+ (copy_category_table): Now static, since it's not used elsewhere.
+ * character.c (string_count_byte8): Likewise.
+
+ * ccl.c (CCL_WRITE_STRING, CCL_ENCODE_CHAR, Fccl_execute_on_string):
+ (Fregister_code_conversion_map): Rename locals to avoid shadowing.
+
+ * chartab.c (copy_sub_char_table): Now static, since it's not used
+ elsewhere.
+ (sub_char_table_ref_and_range, char_table_ref_and_range):
+ Rename locals to avoid shadowing.
+ (ASET_RANGE, GET_SUB_CHAR_TABLE): Remove unused macros.
+
+ * bidi.c (bidi_check_type): Now static, since it's not used elsewhere.
+ (BIDI_BOB): Remove unused macro.
+
+ * cm.c (cmgoto): Mark variables that gcc -Wuninitialized does not
+ deduce are never used uninitialized.
+ * term.c (encode_terminal_code): Likewise.
+
+ * term.c (encode_terminal_code): Now static. Remove unused local.
+
+ * tparam.h: New file.
+ * term.c, tparam.h: Include it.
+ * deps.mk (term.o, tparam.o): Depend on tparam.h.
+ * term.c (tputs, tgetent, tgetflag, tgetnum, tparam, tgetstr):
+ Move these decls to tparam.h, and make them agree with what
+ is actually in tparam.c. The previous trick of using incompatible
+ decls in different modules does not conform to the C standard.
+ All callers of tparam changed to use tparam's actual API.
+ * tparam.c (tparam1, tparam, tgoto):
+ Use const pointers where appropriate.
+
+ * cm.c (calccost, cmgoto): Use const pointers where appropriate.
+ * cm.h (struct cm): Likewise.
+ * dispextern.h (do_line_insertion_deletion_costs): Likewise.
+ * scroll.c (ins_del_costs, do_line_insertion_deletion_costs): Likewise.
+ * term.c (tty_ins_del_lines, calculate_costs, struct fkey_table):
+ (term_get_fkeys_1, append_glyphless_glyph, produce_glyphless_glyph):
+ (turn_on_face, init_tty): Likewise.
+ * termchar.h (struct tty_display_info): Likewise.
+
+ * term.c (term_mouse_position): Rename local to avoid shadowing.
+
+ * alloc.c (mark_ttys): Move decl from here ...
+ * lisp.h (mark_ttys): ... to here, so that it's checked against defn.
+
+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.
+
+ * 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.
(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.