1. 14 Apr, 2011 2 commits
  2. 13 Apr, 2011 10 commits
    • Paul Eggert's avatar
      * image.c: Make symbols static if they're not exported. · cd44d2eb
      Paul Eggert authored
      * dispextern.h (x_create_bitmap_from_xpm_data): Do not declare
      if USE_GTK.
      * image.c (x_create_bitmap_from_xpm_data): Do not define if USE_GTK.
      (xpm_color_cache, ct_table, ct_colors_allocated): Now static.
    • Paul Eggert's avatar
      * fringe.c (standard_bitmaps): Now static. · ad9a7a06
      Paul Eggert authored
      (max_used_fringe_bitmap): Now static, unless HAVE_NS.
    • Paul Eggert's avatar
      * frame.c: Make symbols static if they're not exported. · 81626931
      Paul Eggert authored
      (x_report_frame_params, make_terminal_frame): Now static.
      (get_frame_param): Now static, unless HAVE_NS.
      (x_fullscreen_adjust): Define if WINDOWSNT, not if HAVE_WINDOW_SYSTEM.
      (x_get_resource_string): Remove; not used.
      * frame.h (make_terminal_frame, x_report_frame_params):
      (x_get_resource_string); Remove decls.
      (x_fullscreen_adjust): Declare only if WINDOWSNT.
      * lisp.h (get_frame_param): Declare only if HAVE_NS.
    • Paul Eggert's avatar
      * font.c, fontset.c: Make symbols static if they're not exported. · 239f9db9
      Paul Eggert authored
      * dispextern.h (FACE_SUITABLE_FOR_ASCII_CHAR_P): New macro.
      * font.c (font_close_object): Now static.
      * font.h (font_close_object): Remove.
      * fontset.c (FONTSET_OBJLIST): Remove.
      (free_realized_fontset) #if-0 the body, which does nothing.
      (face_suitable_for_char_p): #if-0, as it's never called.
      * fontset.h (face_suitable_for_char_p): Remove decl.
      * xfaces.c (face_at_string_position): Use
      since 0 is always ASCII.
    • Paul Eggert's avatar
      * fns.c (weak_hash_tables): Now static. · dfcf3579
      Paul Eggert authored
    • Paul Eggert's avatar
      * fileio.c: Make symbols static if they're not exported. · 5045092b
      Paul Eggert authored
      (auto_saving, auto_save_mode_bits, auto_save_error_occurred):
      (Vwrite_region_annotation_buffers): Now static.
    • Paul Eggert's avatar
      * eval.c: Make symbols static if they're not exported. · 57a96f5c
      Paul Eggert authored
      (backtrace_list, lisp_eval_depth, when_entered_debugger): Now static.
      * lisp.h (backtrace_list): Remove decl.
    • Paul Eggert's avatar
      * emacs.c: Make symbols static if they're not exported. · 35f08c38
      Paul Eggert authored
      (malloc_state_ptr, malloc_using_checking, syms_of_emacs):
      (fatal_error_code, fatal_error_signal_hook, standard_args):
      Now static.
      (fatal_error_signal): Now static, unless FLOAT_CATCH_SIGKILL.
      (DEFINE_DUMMY_FUNCTION): Mark function as externally visible.
      (__CTOR_LIST__, __DTOR_LIST__): Now externally visible.
      * lisp.h (fatal_error_signal_hook): Remove decl.
      (fatal_error_signal): Declare only if FLOAT_CATCH_SIGKILL.
    • Paul Eggert's avatar
      * editfns.c: Move a (normally-unused) function to its only use. · f44bd759
      Paul Eggert authored
      * editfns.c, lisp.h (get_operating_system_release): Remove.
      * process.c (init_process) [DARWIN_OS]: Do it inline, as it is not
      worth the hassle of breaking this out.
    • Paul Eggert's avatar
      * xterm.c: Make symbols static if they're not exported. · b532497d
      Paul Eggert authored
      (x_raise_frame, x_lower_frame, x_wm_set_window_state):
      (x_wm_set_icon_pixmap, x_initialize, XTread_socket_fake_io_error):
      (x_destroy_window, x_delete_display):
      Now static.
      (x_dispatch_event): Now static if ! (USE_MOTIF || USE_X_TOOLKIT).
      (x_mouse_leave): Remove; unused.
      * xterm.h (x_display_info_for_name, x_raise_frame, x_lower_frame):
      (x_destroy_window, x_wm_set_window_state, x_wm_set_icon_pixmap):
      (x_delete_display, x_initialize, x_set_border_pixel, x_screen_planes):
      Remove decls.
      (x_mouse_leave): Declare only if WINDOWSNT.
      (x_dispatch_event): Declare only if USE_MOTIF or USE_X_TOOLKIT.
      (xic_create_fontsetname): Declare only if HAVE_X_WINDOWS &&
  3. 12 Apr, 2011 7 commits
    • Paul Eggert's avatar
      * ftxfont.c: Make symbols static if they're not exported. · 1675728f
      Paul Eggert authored
      (ftxfont_driver): Export only if !defined HAVE_XFT && def8ined
      * font.h (ftxfont_driver): Likewise.
    • Paul Eggert's avatar
      * xfns.c: Make symbols static if they're not exported. · e4cebfca
      Paul Eggert authored
      (x_last_font_name, x_display_info_for_name):
      (x_set_foreground_color, x_set_background_color, x_set_mouse_color):
      (x_set_cursor_color, x_set_border_pixel, x_set_border_color):
      (x_set_cursor_type, x_set_icon_type, x_set_icon_name):
      (x_set_scroll_bar_foreground, x_set_scroll_bar_background):
      (x_explicitly_set_name, x_set_title, xic_defaut_fontset, tip_timer):
      (last_show_tip_args): Now static.
      (xic_defaut_fontset, xic_create_fontsetname): Define only if
      defined HAVE_X_WINDOWS && defined USE_X_TOOLKIT
      (x_screen_planes): Remove; unused.
      * dispextern.h (x_screen_planes): Remove decl.
    • Paul Eggert's avatar
      * dispnew.c: Make symbols static if they're not exported. · 5bf46f05
      Paul Eggert authored
      * dispextern.h (redraw_garbaged_frames, scrolling):
      (increment_row_positions): Remove.
      * dispnew.c (new_glyph_matrix, increment_row_positions, scrolling):
      (delayed_size_change, glyph_matrix_count, glyph_pool_count):
      Now static.
      (redraw_garbaged_frames): Remove; unused.
    • Paul Eggert's avatar
      * xfaces.c: Make symbols static if they're not exported. · 435f4c28
      Paul Eggert authored
      * dispextern.h (ascii_face_of_lisp_face, free_realized_face):
      Remove decls.
      * xterm.h (defined_color): Remove decls.
      (x_free_dpy_colors): Declare only if USE_X_TOOLKIT.
      * xfaces.c (tty_suppress_bold_inverse_default_colors_p):
      (menu_face_changed_default, defined_color, free_realized_face):
      (x_free_dpy_colors): Define only if USE_X_TOOLKIT.
      (ascii_face_of_lisp_face): Remove; unused.
    • Paul Eggert's avatar
      * xdisp.c: Make symbols static if they're not exported. · 8524aef3
      Paul Eggert authored
      * dispextern.h (scratch_glyph_row, window_box_edges):
      (glyph_to_pixel_coords, set_cursor_from_row):
      (get_next_display_element, set_iterator_to_next):
      (highlight_trailing_whitespace, frame_to_window_pixel_xy):
      (show_mouse_face): Remove decls
      * frame.h (message_buf_print): Likewise.
      * lisp.h (pop_message, set_message, check_point_in_composition):
      * xterm.h (set_vertical_scroll_bar): Likewise.
      * xdisp.c (list_of_error, Vmessage_stack, line_number_displayed):
      (message_buf_print, scratch_glyph_row, displayed_buffer):
      (set_iterator_to_next, pop_message, set_message, set_cursor_from_row):
      (get_next_display_element, show_mouse_face, window_box_edges):
      (frame_to_window_pixel_xy, check_point_in_composition):
      (set_vertical_scroll_bar, highlight_trailing_whitespace): Now static.
      (glyph_to_pixel_coords): Remove; unused.
    • Paul Eggert's avatar
      * dired.c (file_name_completion): Now static. · 16390cd2
      Paul Eggert authored
      * dbusbind.c (xd_in_read_queued_messages): Now static.
    • Paul Eggert's avatar
      * lisp.h (circular_list_error, FOREACH): Remove; unused. · a25f4dfa
      Paul Eggert authored
      * data.c (circular_list_error): Remove.
  4. 11 Apr, 2011 15 commits
    • Paul Eggert's avatar
      * commands.h (last_point_position, last_point_position_buffer): · 14a9c8df
      Paul Eggert authored
      (last_point_position_window): Remove decls.
      * keyboard.c: Make these variables static.
    • Paul Eggert's avatar
      * coding.h (coding, code_convert_region, encode_coding_gap): Remove · 74ab6df5
      Paul Eggert authored
      * coding.c (Vsjis_coding_system, Vbig5_coding_system):
      (iso_code_class, detect_coding, code_convert_region): Now static.
      (encode_coding_gap): Remove; unused.
    • Paul Eggert's avatar
    • Paul Eggert's avatar
      * charset.h (charset_iso_8859_1): Remove decl. · a2cb4e63
      Paul Eggert authored
      * charset.c (charset_iso_8859_1, charset_emacs, map_charset_for_dump):
      Now static.
    • Paul Eggert's avatar
      * ccl.h (check_ccl_update, Vccl_program_table): Remove decls. · 127198fd
      Paul Eggert authored
      * ccl.c (Vccl_program_table): Now static.
      (check_ccl_update): Remove; unused.
    • Paul Eggert's avatar
      * category.c (SET_CATEGORY_SET, set_category_set): Move here. · d85b608f
      Paul Eggert authored
      * category.h: ... from here.
      * category.c (check_category_table, set_category_set): Now static.
    • Paul Eggert's avatar
      * casetab.c (Vascii_upcase_table, Vascii_eqv_table): Now static. · 31cd66f3
      Paul Eggert authored
      * lisp.h: Remove these decls.
    • Paul Eggert's avatar
    • Paul Eggert's avatar
      * bidi.c (bidi_dump_cached_states): Mark as externally visible, · e78aecca
      Paul Eggert authored
      so that it's not optimized away.
      (bidi_ignore_explicit_marks_for_paragraph_level): Likewise.
      * dispextern.h (bidi_dump_cached_states): Remove, since it's
      exported only to the debugger.
    • Paul Eggert's avatar
      * atimer.c (alarm_signal_handler, run_all_atimers): Now static. · e192d7d3
      Paul Eggert authored
      * atimer.h (run_all_atimers): Removed; not exported.
    • Paul Eggert's avatar
      font.c: Make copy_font_spec and merge_font_spec ordinary C functions. · 92470028
      Paul Eggert authored
      * font.c (copy_font_spec): Rename from Fcopy_font_spec, since it
      was inaccessible from Lisp.
      (merge_font_spec): Likewise, renaming from Fmerge_font_spec.
      * font.c, font.h, fontset.c, xfaces.c, xfont.c: Change all uses.
    • Paul Eggert's avatar
      alloc.c: Import and export fewer symbols, and remove unused items. · 244ed907
      Paul Eggert authored
      * lisp.h (suppress_checking, die): Declare only if ENABLE_CHECKING
      is defined.
      (suppress_checking): Add EXTERNALLY_VISIBLE attribute, so that
      it's not optimized away by whole-program optimization.
      (message_enable_multibyte, free_misc): Remove.
      (catchlist, handlerlist, mark_backtrace):
      Declare only if BYTE_MARK_STACK.
      (mark_byte_stack): Likewise, fixing a ifdef-vs-if typo.
      * alloc.c (pure): Export only if VIRT_ADDR_VARIES is defined.
      (message_enable_multibyte): Remove decl.
      (free_misc, interval_free_list, float_block, float_block_index):
      (n_float_blocks, float_free_list, cons_block, cons_block_index):
      (cons_free_list, last_marked_index):
      Now static.
      (suppress_checking, die): Define only if ENABLE_CHECKING is defined.
      * eval.c (catchlist, handlerlist): Export only if BYTE_MARK_STACK.
      (mark_backtrace): Define only if BYTE_MARK_STACK.
      * xdisp.c (message_enable_multibyte): Now static.
    • Paul Eggert's avatar
      Declare Lisp_Object Q* variables to be 'static' if not exproted. · 955cbe7b
      Paul Eggert authored
      This makes it easier for human readers (and static analyzers)
      to see whether these variables are used from other modules.
      * alloc.c, buffer.c, bytecode.c, callint.c, casetab.c, category.c:
      * ccl.c, character.c, charset.c, cmds.c, coding.c, composite.c:
      * data.c, dbusbind.c, dired.c, editfns.c, eval.c, fileio.c, fns.c:
      * font.c, frame.c, fringe.c, ftfont.c, image.c, keyboard.c, keymap.c:
      * lread.c, macros.c, minibuf.c, print.c, process.c, search.c:
      * sound.c, syntax.c, textprop.c, window.c, xdisp.c, xfaces.c, xfns.c:
      * xmenu.c, xselect.c:
      Declare Q* vars static if they are not used in other modules.
      * ccl.h, character.h, charset.h, coding.h, composite.h, font.h:
      * frame.h, intervals.h, keyboard.h, lisp.h, process.h, syntax.h:
      Remove decls of unexported vars.
      * keyboard.h (EVENT_HEAD_UNMODIFIED): Remove now-unused macro.
    • Paul Eggert's avatar
      * lisp.h (DEFINE_FUNC): Make sname 'static'. · 95c82688
      Paul Eggert authored
    • Paul Eggert's avatar
      Make Emacs functions such as Fatom 'static' by default. · 16a97296
      Paul Eggert authored
      This makes it easier for human readers (and static analyzers)
      to see whether these functions can be called from other modules.
      DEFUN now defines a static function.  To make the function external
      so that it can be used in other C modules, use the new macro DEFUE.
      * lisp.h (DEFINE_FUNC): New macro, with the old contents of DEFUN.
      (DEFUN): Rewrite in terms of DEFINE_FUNC.  It now generates a
      static function definition.  Use DEFUE if you want an extern one.
      (DEFUE, INFUN): New macros.
      (Funibyte_char_to_multibyte, Fsyntax_table_p, Finit_image_library):
      (Feval_region, Fbacktrace, Ffetch_bytecode, Fswitch_to_buffer):
      (Ffile_executable_p, Fmake_symbolic_link, Fcommand_execute):
      (Fget_process, Fdocumentation_property, Fbyte_code, Ffile_attributes):
      Remove decls, since these functions are now static.
      (Funintern, Fget_internal_run_time): New decls, since these functions
      were already external.
      * alloc.c, buffer.c, callint.c, callproc.c, casefiddle.c, casetab.c:
      * ccl.c, character.c, chartab.c, cmds.c, coding.c, data.c, dispnew.c:
      * doc.c, editfns.c, emacs.c, eval.c, fileio.c, filelock.c, floatfns.c:
      * fns.c, font.c, fontset.c, frame.c, image.c, indent.c:
      * keyboard.c, keymap.c, lread.c:
      * macros.c, marker.c, menu.c, minibuf.c, print.c, process.c, search.c:
      * syntax.c, term.c, terminal.c, textprop.c, undo.c:
      * window.c, xdisp.c, xfaces.c, xfns.c, xmenu.c, xsettings.c:
      Mark functions with DEFUE instead of DEFUN,
      if they are used in other modules.
      * buffer.c (Fset_buffer_major_mode, Fdelete_overlay): New forward
      decls for now-static functions.
      * buffer.h (Fdelete_overlay): Remove decl.
      * callproc.c (Fgetenv_internal): Mark as internal.
      * composite.c (Fremove_list_of_text_properties): Remove decl.
      (Fcomposition_get_gstring): New forward static decl.
      * composite.h (Fcomposite_get_gstring): Remove decl.
      * dired.c (Ffile_attributes): New forward static decl.
      * doc.c (Fdocumntation_property): New forward static decl.
      * eval.c (Ffetch_bytecode): New forward static decl.
      (Funintern): Remove extern decl; now in .h file where it belongs.
      * fileio.c (Fmake_symbolic_link): New forward static decl.
      * image.c (Finit_image_library): New forward static decl.
      * insdel.c (Fcombine_after_change_execute): Make forward decl static.
      * intervals.h (Fprevious_property_change):
      (Fremove_list_of_text_properties): Remove decls.
      * keyboard.c (Fthis_command_keys): Remove decl.
      (Fcommand_execute): New forward static decl.
      * keymap.c (Flookup_key): New forward static decl.
      (Fcopy_keymap): Now static.
      * keymap.h (Flookup_key): Remove decl.
      * process.c (Fget_process): New forward static decl.
      (Fprocess_datagram_address): Mark as internal.
      * syntax.c (Fsyntax_table_p): New forward static decl.
      (skip_chars): Remove duplicate decl.
      * textprop.c (Fprevious_property_change): New forward static decl.
      * window.c (Fset_window_fringes, Fset_window_scroll_bars):
      Now internal.
      (Fset_window_margins, Fset_window_vscroll): New forward static decls.
      * window.h (Fset_window_vscroll, Fset_window_margins): Remove decls.
  5. 10 Apr, 2011 6 commits