* image.c (x_edge_detection): Remove unnecessary cast that

gcc -Wbad-function-cast diagnoses.

* image.c (xpm_load): Redo to avoid "discards qualifiers" gcc warning.

* image.c (DIM, HAVE_STDLIB_H_1): Remove unused macros.

* image.c (clear_image_cache): Now static.

2011-03-11  Paul Eggert  <eggert@cs.ucla.edu>

Fix some minor problems found by GCC 4.5.2's static checks.

Merge: Minor changes for problems found by GCC 4.5.2's static checks.

Add bug#s.

* fringe.c: Add comment.

* fringe.c (update_window_fringes): Mark locals as initialized

lisp/help-fns.el (describe-variable): Don't complete keywords.

* .gdbinit (pwinx, xbuffer): Fix access to buffer name.

Fix package.el handling of version numbers like 1.0pre6.

* lisp/emacs-lisp/package.el (package-version-join): Impose a standard
string representation for pre/alpha/beta version lists.
(package-unpack-single): Standardize the directory name by passing
it through package-version-join.

Fix package-strip-rcs-id to be more robust.

See http://lists.gnu.org/archive/html/emacs-devel/2011-03/msg00396.html

* lisp/emacs-lisp/package.el (package-strip-rcs-id): Accept any version
string that does not signal an error in version-to-list.

* fringe.c (update_window_fringes): Initialize top_row_ends_at_zv_p and

bot_row_ends_at_zv_p, with a FIXME comment.

auth-source.el (auth-source-read-char-choice): Remove `dropdown-list'.
 (auth-source-pick-first-password): New convenience function.

* simple.el (delete-trailing-whitespace): Return nil for the
benefit of `write-file-functions'.

* lisp/vc/vc-hg.el (vc-hg-pull, vc-hg-merge-branch): Use vc-hg-program.

Add `vc-git-program' option; suggested on emacs-devel.

* lisp/vc/vc-git.el (vc-git-program): New option.
(vc-git-branches, vc-git-pull, vc-git-merge-branch, vc-git-command)
(vc-git--call): Use it.

* lisp/eshell/esh-util.el (eshell-condition-case): Doc fix.

Fix non-ASCII characters in User and ELisp manuals.

 doc/lispref/modes.texi (Operator Precedence Grammars): Don't use characters
 outside ISO-8859-1.
 doc/emacs/search.texi (Regexp Example):
 doc/emacs/mule.texi (International Chars):
 doc/emacs/building.texi (External Lisp): Don't use characters outside
 ISO-8859-1.

cus-edit tweak for bug#2298.

* lisp/cus-edit.el (Custom-newline): If no button at point, look for a
subgroup button at start-of-line.

* lisp/mail/rmail.el (rmail-msgend, rmail-msgbeg): Doc fixes.

Inhibit mouse-avoidance if cursor-type is nil (Bug#8209).

* lisp/avoid.el (mouse-avoidance-ignore-p): Do not move the cursor if
`cursor-type' is nil.

* fringe.c (destroy_fringe_bitmap, init_fringe_bitmap): Now static.

* alloc.c (mark_fringe_data): Move decl from here ...

* lisp.h (mark_fringe_data) [HAVE_WINDOW_SYSTEM]: ... to here,
to check its interface.
(init_fringe_once): Do not declare unless HAVE_WINDOW_SYSTEM.

* xfns.c (Fx_create_frame, x_create_tip_frame, Fx_show_tip):

(Fx_backspace_delete_keys_p):
Use them to avoid shadowing, and rename vars to avoid shadowing.
(x_decode_color, x_set_name, x_window): Now static.

* fontset.c (FONTSET_SPEC, FONTSET_REPERTORY, RFONT_DEF_REPERTORY): Remove; unused.

* fontset.c (fontset_font): Mark local as initialized.

* fontset.c (Fset_fontset_font): Rename local to avoid shadowing.

* fontset.c (free_realized_fontset): Now static.

* xrdb.c: Include "xterm.h", to check x_load_resources's interface.

* xselect.c (x_own_selection, Fx_disown_selection_internal): Rename locals

to avoid shadowing.
(x_handle_dnd_message): Remove local to avoid shadowing.

* xselect.c (TRACE3) [!defined TRACE_SELECTION]: Remove; not used.

* xselect.c (x_disown_buffer_selections): Remove; not used.

* lisp.h (GCPRO1_VAR, UNGCPRO_VAR): New macros,

so that the caller can use some name other than gcpro1.
(GCPRO1, UNGCPRO): Reimplement in terms of the new macros.
(Fx_create_frame, x_create_tip_frame, Fx_show_tip):
(Fx_backspace_delete_keys_p): Rename locals to avoid shadowing.
Some of these renamings use the new GCPRO1_VAR and UNGCPRO_VAR
macros.

* xfns.c: (x_real_positions, xg_set_icon_from_xpm_data, x_create_tip_frame):

Remove unused locals.

* xfns.c (Fx_file_dialog, Fx_select_font): Fix pointer signedness.

* xfns.c (Fx_create_frame): Add braces to silence GCC warning.

* xfns.c (x_decode_color, x_set_name, x_window): Now static.

* calc/calc.el (calc-mode-map): Don't bind "C-_" to `calc-missing-key'.

* allout.el Summary: Change so yank of distinctive-bullet items preserves
the existing header prefix, rebulleting it if necessary, rather than
replacing it.  This is necessary for proper operation of cooperative addons
like allout-widgets.

(allout-make-topic-prefix) (allout-rebullet-heading): Change SOLICIT arg to
INSTEAD, and interpret additionally a string value as alternate bullet to
be used, instead of prompting the user for a bullet character.

Fix last commit.

* xterm.c (x_connection_closed): Tell GCC not to suggest NO_RETURN.

* xterm.c (x_delete_glyphs, x_ins_del_lines): Mark with NO_RETURN.

Allow building manuals with Texinfo before v4.12.

 doc/lispref/intro.texi (Acknowledgements): Convert to ISO-8859-1 encoding.
 doc/lispref/makefile.w32-in (MAKEINFO_OPTS): Add --enable-encoding.
 doc/lispref/elisp.texi: Use ISO-8859-1 in @documentencoding.
 doc/emacs/ack.texi (Acknowledgments): Convert to ISO-8859-1 encoding.  Use
 Texinfo @-commands for non Latin-1 characters.
 doc/emacs/emacs.texi: Use ISO-8859-1 in @documentencoding.
 doc/emacs/makefile.w32-in (MAKEINFO_OPTS): Add --enable-encoding.

* src/search.c (compile_pattern_1): Remove unused argument.
  (compile_pattern): Don't pass it.

Fix a typo in last commit of ChangeLog file.

Fix bug #8210 with index entries for ".emacs".

 doc/emacs/custom.texi (Init File): Add index entries for ".emacs".

auth.texi (Help for developers): Untabify.

Merge changes made in Gnus trunk.

auth-source.el (auth-source-read-char-choice): New function to read a character choice using `dropdown-list', `read-char-choice', or `read-char'.  It appends "[a/b/c] " to the prompt if the choices were '(?a ?b ?c).  The `dropdown-list' support is disabled for now.  Use `eval-when-compile' to load `dropdown-list'.
 (auth-source-netrc-saver): Use it.
nnimap.el (nnimap-credentials): Keep the :save-function as the third parameter in the credentials.
 (nnimap-open-connection-1): Use it after a successful login.
 (nnimap-credentials): Add IMAP-specific user and password prompt.
auth-source.el (auth-source-search): Add :require parameter, taking a list.  Document it and the :save-function return token.  Pass :require down.  Change the CREATED message from a warning to a debug statement.
 (auth-source-search-backends): Pass :require down.
 (auth-source-netrc-search): Pass :require down.
 (auth-source-netrc-parse): Use :require, if it's given, as a filter.  Change save prompt to indicate all modifications saved here are deletions.
 (auth-source-netrc-create): Take user login name as default in user prompt.  Move all the save functionality to a lexically bound function under the :save-function token in the returned list.  Set up clearer default prompts for user, host, port, and secret.
 (auth-source-netrc-saver): New function, intended to be wrapped for :save-function.

* net/tramp-sh.el (tramp-do-copy-or-rename-file-out-of-band): Do
not use `tramp-file-name-port', because this returns also
`tramp-default-port'.

* lisp/net/rcirc.el (rcirc-handler-001): Remove useless
with-rcirc-process-buffer.
(rcirc-check-auth-status): Swap arguments to string-match.

* xterm.c: (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.

* shell.el (shell-mode): Set comint-input-ring-size from HISTSIZE.  (Bug#7889)

Fix previous gdb-mi fix...

Fix previous gdb-mi change.

gdb-mi addition for bug#7889.

* lisp/progmodes/gdb-mi.el (gdb): Improve 2010-12-08 change.
Check for GDBHISTFILE, HISTSIZE, etc.

* xterm.c (XEMBED_VERSION, xembed_set_info): Don't define unless needed.

(x_fatal_error_signal): Remove; not used.

* xterm.c (x_iconify_frame, x_free_frame_resources): Don't declare locals

unless they are used.

* xterm.c: Don't declare local "event" unless it's used.

* xterm.c (XTread_socket): Don't define label "out" unless it's used.

* xterm.c: (x_shift_glyphs_for_insert, XTflash, XTring_bell):

(x_calc_absolute_position): Now static.

* xterm.h (x_mouse_leave): New decl.

* xterm.c (x_copy_dpy_color, x_focus_on_frame, x_unfocus_frame):

Remove unused functions.

* xfaces.c (STRDUPA, LSTRDUPA, FONT_POINT_SIZE_QUANTUM): Remove; not used.

(LFACEP): Define only if XASSERTS, as it's not needed otherwise.

* xfaces.c (Fx_list_fonts): Mark variable that gcc -Wuninitialized

does not deduce is never used uninitialized.

* xfaces.c (free_realized_faces_for_fontset): Remove; not used.

* xfaces.c (tty_defined_color, merge_face_heights): Now static.

* term.c (encode_terminal_code): Mark vars for gcc -Wuninitialized.

* xfaces.c (clear_face_cache, Fx_list_fonts, Fface_font): Rename

or move locals to avoid shadowing.

* terminal.c (store_terminal_param): Now static.

* term.c, tparam.h (tgetflag, tgetnum, tgetstr): Move the const to the .h file

* alloc.c (mark_ttys): Move decl from here ...

* lisp.h (mark_ttys): ... to here, so that it's checked against defn.

* term.c (term_mouse_position): Rename local to avoid shadowing.

* 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 (tgetflag, tgetnum, tgetstr): Redefine to use const pointers.

* 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.

Add missing piece of previous doc/ change.

* doc/lispref/elisp.texi: Set documentencoding.

* term.c (encode_terminal_code): Remove unused local

* term.c (encode_terminal_code): Now static.

* cm.c (cmgoto): Mark variables that gcc -Wuninitialized does not

deduce are never used uninitialized.

* bidi.c (BIDI_BOB): Remove unused macro.

Use condition-case-no-debug in package.el.

* lisp/emacs-lisp/package.el (package-refresh-contents)
(package-menu-execute): Use condition-case-no-debug.

Merge changes made in Gnus trunk.

message.texi (Message Buffers): Update default value of message-generate-new-buffers.
shr.el (shr-table-horizontal-line): Change the defaults for the table lines to be spaces instead.
sieve-manage.el (sieve-sasl-auth): Create auth-info if not found.
 (sieve-sasl-auth): Check that auth-source-search did return something, or just return an empty string.
gnus-start.el (gnus-group-change-level): Allow putting foreign groups onto the list of killed groups, too.  This makes killed nnimap groups, for instance, more reliably not reappear.
nnimap.el (nnimap-request-thread): Don't bug out when we can't find the parent.

* simple.el (shell-command-to-string): Use `process-file'.

* emacs-lisp/package.el (package-tar-file-info): Handle also
remote files.

* emacs-lisp/package-x.el (package-upload-buffer-internal): Use
`equal' for upload base check.

Convert some names in texinfo files to UTF-8.

* doc/emacs/Makefile.in (MAKEINFO_OPTS): Add --enable-encoding.
* doc/emacs/emacs.texi (Acknowledgments):
* doc/emacs/ack.texi (Acknowledgments): Names to UTF-8.

* doc/lispref/Makefile.in (MAKEINFO_OPTS): Add --enable-encoding.
* doc/lispref/intro.texi (Acknowledgements): Names to UTF-8.

Updates for compiling with Gtk+ 3.0 (--with-x-toolkit=gtk3).

* configure.in: Require 3.0 for --with-gtk3.  Add HAVE_GTK3.

* doc/emacs/xresources.texi (GTK resources): ~/.emacs.d/gtkrc does not work
for Gtk+ 3.

* src/gtkutil.c: Include gtkx.h if HAVE_GTK3.  If ! HAVE_GTK3, define
gdk_window_get_screen, gdk_window_get_geometry,
gdk_x11_window_lookup_for_display and GDK_KEY_g.
(xg_set_screen): Use DEFAULT_GDK_DISPLAY.
(xg_get_pixbuf_from_pixmap): New function.
(xg_get_pixbuf_from_pix_and_mask): Change parameters from GdkPixmap
to Pixmap, take frame as parameter, remove GdkColormap parameter.
Call xg_get_pixbuf_from_pixmap instead of
gdk_pixbuf_get_from_drawable.
(xg_get_image_for_pixmap): Do not make GdkPixmaps, call
xg_get_pixbuf_from_pix_and_mask with Pixmap parameters instead.
(xg_check_special_colors): Use GtkStyleContext and its functions
for HAVE_GTK3.
(xg_prepare_tooltip, xg_hide_tooltip): Call gdk_window_get_screen.
(xg_prepare_tooltip, create_dialog, menubar_map_cb)
(xg_update_frame_menubar, xg_tool_bar_detach_callback)
(xg_tool_bar_attach_callback, xg_update_tool_bar_sizes): Call
gtk_widget_get_preferred_size.
(xg_frame_resized): gdk_window_get_geometry only takes 5
parameters.
(xg_win_to_widget, xg_event_is_for_menubar): Call
gdk_x11_window_lookup_for_display.
(xg_set_widget_bg): New function.
(delete_cb): New function.
(xg_create_frame_widgets): connect delete-event to delete_cb.
Call xg_set_widget_bg.  Only set backgrund pixmap for ! HAVE_GTK3
(xg_set_background_color): Call xg_set_widget_bg.
(xg_set_frame_icon): Call xg_get_pixbuf_from_pix_and_mask.
(xg_create_scroll_bar): vadj is a GtkAdjustment for HAVE_GTK3.
Only call gtk_range_set_update_policy if ! HAVE_GTK3.
(xg_make_tool_item): Only connect xg_tool_bar_item_expose_callback
if ! HAVE_GTK3.
(update_frame_tool_bar): Call gtk_widget_hide.
(xg_initialize): Use GDK_KEY_g.

* src/xmenu.c (menu_position_func): Call gtk_widget_get_preferred_size.

* src/xsmfns.c (gdk_set_sm_client_id): Define to gdk_set_sm_client_id
if ! HAVE_GTK3
(x_session_initialize): Call gdk_x11_set_sm_client_id.

* src/xterm.c (XFillRectangle): Use cairo routines for HAVE_GTK3.
(x_term_init): Disable Xinput(2) with GDK_CORE_DEVICE_EVENTS.
Load ~/emacs.d/gtkrc only for ! HAVE_GTK3.

* src/xterm.h (DEFAULT_GDK_DISPLAY): New define.
(GDK_WINDOW_XID, gtk_widget_get_preferred_size): New defines
for ! HAVE_GTK3.
(GTK_WIDGET_TO_X_WIN): Use GDK_WINDOW_XID.

* dispextern.h (bidi_dump_cached_states): Likewise.

* bidi.c (bidi_check_type): Now static, since it's not used elsewhere.

* chartab.c (ASET_RANGE, GET_SUB_CHAR_TABLE): Remove unused macros.

* chartab.c: (sub_char_table_ref_and_range, char_table_ref_and_range):

Rename locals to avoid shadowing.

* chartab.c (copy_sub_char_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.

* category.c (copy_category_table): Now static, since it's not used elsewhere.

* category.c (hash_get_category_set): Remove unused local var.

* coding.c (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.

* coding.c (decode_coding_emacs_mule): Mark variables that gcc

-Wuninitialized does not deduce are never used uninitialized.

* coding.c: (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.

* coding.c: (decode_coding_object, encode_coding_object, detect_coding_system):

Mark variables that gcc -Wuninitialized does not deduce are never
used uninitialized.

* coding.c (decode_coding_iso_2022): Add "default: abort ();" as a safety check.