X-Git-Url: http://git.hcoop.net/bpt/emacs.git/blobdiff_plain/2b0c7330457b8ca42375c92ada7dc7cefb0fa9fb..77f2391201c5014f9ec83250f753cad4de814c95:/src/ChangeLog diff --git a/src/ChangeLog b/src/ChangeLog index 0e5e295a62..fcb3cbc5de 100644 --- a/src/ChangeLog +++ b/src/ChangeLog @@ -1,3 +1,213 @@ +2011-03-09 Paul Eggert + + * xterm.c (x_copy_dpy_color, x_focus_on_frame, x_unfocus_frame): + Remove unused functions. + + * 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 . + (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 + + * xdisp.c (redisplay_window): Revert incorrect logic in 2011-03-06 + change. + +2011-03-06 Paul Eggert + + current_column: Now returns EMACS_INT, fixing some iftc + that was introduced in the 2002-06-02 change "temporarily"; see + . + * 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 (tiny change) + + * window.c (Fnext_window): Doc fix. (Bug#5567) + +2011-03-05 Chong Yidong + + * nsterm.m (ns_draw_window_cursor): Fix typo in 2011-02-23 commit. + +2011-03-02 Ken Brown + + * sheap.c (STATIC_HEAP_SIZE): Increase to 13MB. + +2011-03-02 Paul Eggert + + 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 + . + (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 * frame.c (store_frame_param): Don't store value directly in @@ -8,7 +218,7 @@ * 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. @@ -1154,7 +1364,7 @@ (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. @@ -2871,7 +3081,7 @@ (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. @@ -3157,7 +3367,7 @@ 2010-10-12 Jan Djärv * 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. @@ -3181,7 +3391,7 @@ * 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. @@ -6118,7 +6328,7 @@ * eval.c: Fix indentation problem. - * keyboard.c: Include "process.h" + * keyboard.c: Include "process.h". * eval.c: Remove obsolete noinline declaration. * fns.c: Likewise. @@ -8524,7 +8734,7 @@ 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.