1. 31 Mar, 2011 1 commit
    • Juanma Barranquero's avatar
      src/xdisp.c: Remove unused parameters. · e4cc2dfc
      Juanma Barranquero authored
      * dispextern.h (move_it_by_lines):
      * xdisp.c (move_it_by_lines): Remove parameter `need_y_p', unused
        since 2000-12-29T14:24:09Z!gerd@gnu.org.  All callers changed.
        (message_log_check_duplicate): Remove parameters `prev_bol' and
        `this_bol', unused since 1998-01-01T02:27:27Z!rms@gnu.org.  All callers changed.
        (redisplay_internal): Remove parameter `preserve_echo_area',
        unused since 1999-07-21T21:43:52Z!gerd@gnu.org.  All callers changed.
      * indent.c (Fvertical_motion):
      * window.c (window_scroll_pixel_based, Frecenter):
        Don't pass `need_y_p' to `move_it_by_lines'.
      e4cc2dfc
  2. 30 Mar, 2011 5 commits
  3. 29 Mar, 2011 4 commits
  4. 28 Mar, 2011 3 commits
    • Juanma Barranquero's avatar
      src/*.c: Remove some additional unused parameters. · 461c2ab9
      Juanma Barranquero authored
      * lisp.h (multibyte_char_to_unibyte):
      * character.c (multibyte_char_to_unibyte): Remove parameter `rev_tbl',
        unused since 2002-03-01T01:16:34Z!handa@m17n.org and 2008-02-01T16:01:31Z!miles@gnu.org.
      * character.h (CHAR_TO_BYTE8):
      * cmds.c (internal_self_insert):
      * editfns.c (general_insert_function):
      * keymap.c (push_key_description):
      * search.c (Freplace_match):
      * xdisp.c (message_dolog, set_message_1): All callers changed.
      
      * coding.c (encode_designation_at_bol): Remove parameter `charbuf_end',
        unused since 2002-03-01T01:17:24Z!handa@m17n.org and 2008-02-01T16:01:31Z!miles@gnu.org.
        All callers changed.
      461c2ab9
    • Stefan Monnier's avatar
      Don't reset post-command-hook to nil upon error. · f6d62986
      Stefan Monnier authored
      * src/eval.c (enum run_hooks_condition): Remove.
      (funcall_nil, funcall_not): New functions.
      (run_hook_with_args): Call each function through a `funcall' argument.
      Remove `cond' argument, now redundant.
      (Frun_hooks, Frun_hook_with_args, Frun_hook_with_args_until_success)
      (Frun_hook_with_args_until_failure): Adjust accordingly.
      (run_hook_wrapped_funcall, Frun_hook_wrapped): New functions.
      * src/keyboard.c (safe_run_hook_funcall): New function.
      (safe_run_hooks_1, safe_run_hooks_error, safe_run_hooks): On error,
      don't set the hook to nil, but remove the offending function instead.
      (Qcommand_hook_internal): Remove, unused.
      (syms_of_keyboard): Don't initialize Qcommand_hook_internal nor define
      Vcommand_hook_internal.
      * doc/lispref/commands.texi (Command Overview): post-command-hook is not reset
      to nil any more.
      f6d62986
    • Juanma Barranquero's avatar
      src/*.c: Remove unused parameters and other warnings. · 1db5b1ad
      Juanma Barranquero authored
      * dispextern.h (string_buffer_position): Remove declaration.
      
      * print.c (strout): Remove parameter `multibyte', unused since
        1999-08-21T19:30:21Z!gerd@gnu.org.  All callers changed.
      
      * search.c (boyer_moore): Remove parameters `len', `pos' and `lim',
        never used since function introduction in 1998-02-08T21:33:56Z!rms@gnu.org.
        All callers changed.
      
      * w32.c (_wsa_errlist): Use braces for struct initializers.
      
      * xdisp.c (string_buffer_position_lim): Remove parameter `w',
        never used since function introduction in 2001-03-09T18:41:50Z!gerd@gnu.org.
        All callers changed.
        (string_buffer_position): Likewise.  Also, make static (it's never
        used outside xdisp.c).
        (cursor_row_p): Remove parameter `w', unused since
        2000-10-17T16:08:57Z!gerd@gnu.org.  All callers changed.
        (decode_mode_spec): Remove parameter `precision', introduced during
        Gerd Moellmann's rewrite at 1999-07-21T21:43:52Z!gerd@gnu.org, but never used.
        All callers changed.
      1db5b1ad
  5. 27 Mar, 2011 14 commits
  6. 26 Mar, 2011 4 commits
    • Eli Zaretskii's avatar
      Call try_scrolling even if buffer's clip_changed flag is set. · 66537049
      Eli Zaretskii authored
       src/xdisp.c (redisplay_window): Don't check buffer's clip_changed
       flag as a prerequisite for invoking try_scrolling.  (Bug#6671)
      66537049
    • Paul Eggert's avatar
      * alloc.c (garbage_collect): Don't assume stack size fits in int. · dd3f25f7
      Paul Eggert authored
      (stack_copy_size): Now size_t, not int.
      (stack_copy, stack_copy_size): Define only if MAX_SAVE_STACK > 0.
      dd3f25f7
    • Juanma Barranquero's avatar
      src/*.c: Fix more warnings about unused parameters. · 59eb0929
      Juanma Barranquero authored
      * keymap.c (describe_vector): Remove parameters `indices' and
        `char_table_depth', unused since 2002-03-01T01:43:26Z!handa@m17n.org.
        (describe_map, Fdescribe_vector): Adjust calls to `describe_vector'.
      
      * w32.c (read_unc_volume): Use parameter `henum', instead of
        global variable `wget_enum_handle'.
      59eb0929
    • Juanma Barranquero's avatar
      src/*.h: Remove unused parameters and functions. · f868cd8a
      Juanma Barranquero authored
      * keyboard.h (timer_check, show_help_echo): Remove unused parameters.
      
      * keyboard.c (timer_check): Remove parameter `do_it_now',
        unused since 1996-04-12T06:01:29Z!rms@gnu.org.
        (show_help_echo): Remove parameter `ok_to_overwrite_keystroke_echo',
        unused since 2008-04-19T19:30:53Z!monnier@iro.umontreal.ca.
      
      * keyboard.c (read_char):
      * w32menu.c (w32_menu_display_help):
      * xmenu.c (show_help_event, menu_help_callback):
        Adjust calls to `show_help_echo'.
      
      * gtkutil.c (xg_maybe_add_timer):
      * keyboard.c (readable_events):
      * process.c (wait_reading_process_output):
      * xmenu.c (x_menu_wait_for_event): Adjust calls to `timer_check'.
      
      * insdel.c (adjust_markers_gap_motion):
        Remove; no-op since 1998-01-02T21:29:48Z!rms@gnu.org.
        (gap_left, gap_right): Don't call it.
      f868cd8a
  7. 25 Mar, 2011 3 commits
    • Chong Yidong's avatar
      Discard changes to buffer->clip_changed incurred during fontification. · 2ecf6fdb
      Chong Yidong authored
      * src/xdisp.c (handle_fontified_prop): Discard changes to clip_changed
      incurred during fontification.
      2ecf6fdb
    • Juanma Barranquero's avatar
      src/*.c: Remove unused function parameters. · 6b1f9ba4
      Juanma Barranquero authored
      * buffer.c (defvar_per_buffer): Remove unused parameter `doc'.
        (DEFVAR_PER_BUFFER): Don't pass it.
      * dispnew.c (row_equal_p, add_row_entry): Remove unused parameter `w'.
        (scrolling_window): Don't pass it.
      6b1f9ba4
    • Juanma Barranquero's avatar
      nt/*.c, src/*.c: Remove unused variables. · 0f4a96b5
      Juanma Barranquero authored
      * nt/addpm.c (main): Remove unused variable `retval'.
      * nt/preprep.c (main): Remove unused variable `ptr'.
      * src/dispextern.h (glyph_matric): Use #if GLYPH_DEBUG, not #ifdef.
      * src/fileio.c (check_executable) [DOS_NT]: Remove unused variables `len'
        and `suffix'.
        (Fset_file_selinux_context) [HAVE_LIBSELINUX]: Move here declaration
        of variables specific to SELinux and computation of `encoded_absname'.
      * src/image.c (XPutPixel): Remove unused variable `height'.
      * src/keyboard.c (make_lispy_event): Remove unused variable `hpos'.
      * src/unexw32.c (get_section_info): Remove unused variable `section'.
      * src/w32.c (stat): Remove unused variables `drive_root' and `devtype'.
        (system_process_attributes): Remove unused variable `sess'.
        (sys_read): Remove unused variable `err'.
      * src/w32fns.c (top): Wrap variables with #if GLYPH_DEBUG, not #ifdef.
        (w32_wnd_proc): Remove unused variable `isdead'.
        (unwind_create_frame): Use #if GLYPH_DEBUG, not #ifdef.
        (Fx_server_max_request_size): Remove unused variable `dpyinfo'.
        (x_create_tip_frame): Remove unused variable `tem'.
      * src/w32inevt.c (w32_console_read_socket): Remove unused variable `no_events'.
      * src/w32term.c (x_draw_composite_glyph_string_foreground):
        Remove unused variable `width'.
      0f4a96b5
  8. 24 Mar, 2011 4 commits
  9. 23 Mar, 2011 2 commits
    • Juanma Barranquero's avatar
      w32: Fix some warnings reported by -Wall -Wextra. · 9c88f339
      Juanma Barranquero authored
      * lib-src/ntlib.c: Include <ctype.h>.
      
      * nt/cmdproxy.c: Include <ctype.h>.
        (make_absolute): Remove unused variable `i'.
      
      * src/w32fns.c (x_set_menu_bar_lines): Remove unused variable `olines'.
        (w32_wnd_proc): Pass NULL to Windows API, not uninitialized buffer.
        (Fx_close_connection): Remove unused variable `i'.
      
      * src/w32font.c (w32font_draw): Return number of glyphs.
        (w32font_open_internal): Remove unused variable `i'.
        (w32font_driver): Add missing initializer.
      
      * src/w32menu.c (utf8to16): Remove unused variable `utf16'.
        (fill_in_menu): Remove unused variable `items_added'.
      
      * src/w32term.c (last_mouse_press_frame): Remove static global variable.
        (w32_clip_to_row): Remove unused variable `f'.
        (x_delete_terminal): Remove unused variable `i'.
      
      * src/w32uniscribe.c (uniscribe_shape): Remove unused variable `nclusters'.
        (NOTHING): Remove unused static global variable.
        (uniscribe_check_otf): Remove unused variable `table'.
        (uniscribe_font_driver): Add missing initializers.
      9c88f339
    • Julien Danjou's avatar
      dee091a3