* fileio.c (file_name_as_directory): Use const pointers when appropriate.

(Fexpand_file_name): Likewise.  In particular, newdir might
point at constant storage, so make it a const pointer.

* fileio.c (file_name_as_directory, directory_file_name):

(barf_or_query_if_file_exists, auto_save_error, auto_save_1):
Now static.

* minibuf.c (Ftry_completion, Fall_completions): Rename or remove locals

to avoid shadowing.

* minibuf.c (choose_minibuf_frame_1): Now static.

* marker.c (bytepos_to_charpos): Remove; unused.

* lisp.h (verify_bytepos, count_markers): New decls,

so that gcc does not warn that these functions aren't declared.

* insdel.c (copy_text): Remove unused local var.

* insdel.c (CHECK_MARKERS): Redo to avoid gcc -Wempty-body diagnostic.

* insdel.c (check_markers, make_gap_larger, make_gap_smaller):

(reset_var_on_error, Fcombine_after_change_execute_1): Now static.

* filelock.c (lock_file_1): Rename local to avoid shadowing.

* filelock.c (within_one_second): Now static.

* buffer.c (fix_overlays_before): Mark locals as initialized.

(fix_start_end_in_overlays): Likewise.  This function should be
simplified by using pointers-to-pointers, but that's a different
matter.

* buffer.c: (Fkill_buffer, record_buffer, Fbury_buffer, Fset_buffer_multibyte):

(report_overlay_modification): Rename locals to avoid shadowing.

* buffer.c (switch_to_buffer_1): Now static.

* sysdep.c (MAXPATHLEN): Define only if needed.

* sysdep.c (sys_subshell): Mark local as volatile if checking for lint,

to suppress a gcc -Wclobbered warning that does not seem to be right.

* sysdep.c: Fix pointer signedness issue.

* sysdep.c (system_process_attributes): Rename vars to avoid shadowing.

* process.c (serial_open, serial_configure): Move decls from here ...

* systty.h: ... to here, so that they can be checked.

* fns.c (get_random, seed_random): Move extern decls from here ...

* lisp.h: ... to here, so that they can be checked.

* sysdep.c (wait_for_termination_signal): Remove; unused.

* sysdep.c (reset_io): Now static.

* keymap.c: (get_keymap, access_keymap, Fdefine_key, Fwhere_is_internal):

(describe_map_tree):
Rename locals to avoid shadowing.

* keymap.c (DENSE_TABLE_SIZE): Remove; unused.

* keymap.c (Fwhere_is_internal): Don't test CONSP (sequences) unnecessarily.

* keymap.c (keymap_parent, keymap_memberp, map_keymap_internal):

(copy_keymap_item, append_key, push_text_char_description):
Now static.

* keyboard.c (read_char): Mark local as initialized.

* keyboard.c (Fexit_recursive_edit, Fabort_recursive_edit): Mark with NO_RETURN.

* keyboard.c (read_key_sequence): Mark locals as initialized.

* keyboard.c: (read_char, kbd_buffer_get_event, make_lispy_position):

(make_lispy_event, make_lispy_movement, apply_modifiers):
(decode_keyboard_code, tty_read_avail_input, menu_bar_items):
(parse_tool_bar_item, read_key_sequence, Fread_key_sequence):
(Fread_key_sequence_vector): Rename locals to avoid shadowing.

* keyboard.h (mark_kboards): Move decl here ...

* alloc.c (mark_kboards): ... from here.

* keyboard.c: Declare functions static if they are not used elsewhere.

(echo_char, echo_dash, cmd_error, top_level_2):
(poll_for_input, handle_async_input): Now static.

* keyboard.h (make_ctrl_char): New decl.

* lisp.h (force_auto_save_soon): New decl.

* emacs.c (main): Add casts to avoid warnings

if GCC considers string literals to be constants.

* emacs.c (DEFINE_DUMMY_FUNCTION): New macro.

(__do_global_ctors, __do_global_ctors_aux, __do_global_dtors, __main):
Use it.

* emacs.c (init_cmdargs): Rename local to avoid shadowing.

Rewrite to avoid "/*" in comment.

* lisp.h (fatal_error_signal): Add decl, since it's exported.

* dbusbind.c (Fdbus_register_service, Fdbus_register_method): Remove unused vars.

* dbusbind.c (Fdbus_init_bus): Add cast to avoid warning

if GCC considers string literals to be constants.

* dbusbind.c: Pointer signedness fixes.

(xd_signature, xd_append_arg, xd_initialize):
(Fdbus_call_method, Fdbus_call_method_asynchronously):
(Fdbus_method_return_internal, Fdbus_method_error_internal):
(Fdbus_send_signal, xd_read_message_1, Fdbus_register_service):
(Fdbus_register_signal): Use SSDATA when the context wants char *.

Fix BUF_* macros to handle indirect buffers properly (Bug#8219).

* buffer.h (BUF_BEGV, BUF_BEGV_BYTE, BUF_ZV, BUF_ZV_BYTE, BUF_PT)
(BUF_PT_BYTE): Rewrite to handle indirect buffers (Bug#8219).
These macros can no longer be used for assignment.

* buffer.c (Fget_buffer_create, Fmake_indirect_buffer): Assign
struct members directly, instead of using BUF_BEGV etc.
(record_buffer_markers, fetch_buffer_markers): New functions for
recording and fetching special buffer markers.
(set_buffer_internal_1, set_buffer_temp): Use them.

* lread.c (unreadchar): Use SET_BUF_PT_BOTH.

* insdel.c (adjust_point): Use SET_BUF_PT_BOTH.

* intervals.c (temp_set_point_both): Use SET_BUF_PT_BOTH.
(get_local_map): Use SET_BUF_BEGV_BOTH and SET_BUF_ZV_BOTH.

* xdisp.c (hscroll_window_tree):
(reconsider_clip_changes): Use PT instead of BUF_PT.

Add `auth-source-search' integration for LDAP searches.

* net/ldap.el (ldap-search-internal): Add `auth-source-search'
integration for LDAP parameters.  The host, base, user or binddn,
and secret tokens can be specified in a netrc file, for instance.
This is optional because an `auth-source' parameter must be
specified in the search attributes.

auth-source.el (auth-source-netrc-create): Show the default in the prompt when prompting for token creation.

MS-Windows followup for 2011-03-13T06:43:00Z!eggert@cs.ucla.edu.

 src/makefile.w32-in ($(BLD)/editfns.$(O)): Depend on
 $(EMACS_ROOT)/lib/intprops.h.

Update for gnulib, adding Solaris 9+10 fixes.

Merge from mainline.

* gtkutil.c (gdk_window_get_screen): Make it clear that this macro is needed

only if USE_GTK_TOOLTIP.

* gtkutil.c: (get_utf8_string, xg_get_file_with_chooser):

Rename locals to avoid shadowing.
(create_dialog): Move locals to avoid shadowing.

* gtkutil.c (int_gtk_range_get_value): New function, which avoids a diagnostic

from gcc -Wbad-function-cast.
(xg_set_toolkit_scroll_bar_thumb): Use it.
(xg_tool_bar_callback, xg_tool_item_stale_p): Rewrite to avoid
diagnostic from gcc -Wbad-function-cast.

* gtkutil.c (xg_free_frame_widgets): Make it clear that a local variable is

needed only if USE_GTK_TOOLTIP.

* gtkutil.c (xg_get_pixbuf_from_pixmap): Add cast from char *

to unsigned char * to avoid compiler diagnostic.

* xgselect.c (xg_select): Remove unused var.

* image.c (my_png_error, my_error_exit): Mark with NO_RETURN.

* image.c (four_corners_best): Mark locals as initialized.

(gif_load): Initialize transparent_p to zero (Bug#8238).
Mark another local as initialized.

Improve quality of tests for time stamp overflow.

* editfns.c (lisp_time_argument): Check for time stamp overflow.

Merge changes made in Gnus trunk.

auth.texi (Help for developers): Update docs to explain that the :save-function will only run the first time.
auth-source.el (auth-source-format-prompt): Always convert the value to a string to avoid evaluating non-string arguments.
 (auth-source-netrc-create): Offer default properly, not as initial content in `read-string'.
 (auth-source-netrc-saver): Use a cache keyed by file name and MD5 hash of line to determine if we've been run before.  If so, don't run again, but print a trivial message to indicate the cache was hit instead.

admin/admin.el: Add some code for deploying web manuals.

lisp/help.el (describe-mode): Link to the mode's definition.

Fixes: debbugs:8185

* editfns.c (hi_time): Do not overparenthesize.

Merge from emacs-23; up to 2010-06-03T22:16:02Z!dann@ics.uci.edu

Some more html rules for doc/misc/Makefile.in.

* doc/misc/Makefile.in (emacs-faq.html): Fix some more cross-refs.
(emacs-faq.text): New target.
(clean): Add emacs-faq.

Fix typos.

lisp/vc/vc-bzr.el (vc-bzr-state): Handle bzr 2.3.0 (follow-up to bug#8170).

Sync with Tramp 2.2.1.

* trampver.texi: Update release number.

Sync with Tramp 2.2.1.

* net/tramp-sh.el (tramp-methods): Exchange "%k" marker with options.

* net/trampver.el: Update release number.

* net/tramp.el (tramp-progress-reporter-update): Use
`tramp-compat-funcall'.
(tramp-handle-start-file-process): Use `tramp-compat-process-get'.
(tramp-handle-insert-file-contents): Make `file-remote-p' call
compatible.
(tramp-open-connection-setup-interactive-shell): Use
`tramp-compat-process-put'.

* net/tramp-compat.el (tramp-compat-process-get)
(tramp-compat-process-put): New defuns.

* net/trampver.el: Update release number.

Please do not merge with the trunk.

Remove arch-tag.

* trampver.texi: Update release number.

src/ChangeLog: Fix last entry for deps.mk.

Make termcap.c implementations consistent with tparam.h prototypes.

 src/termcap.c [MSDOS]: Include "msdos.h.
 (find_capability, tgetnum, tgetflag, tgetstr, tputs, tgetent):
 Constify `char *' arguments and their references according to
 prototypes in tparam.h.
 src/deps.mk (termcap.o): Depend on tparam.h and msdos.h.

Fix compilation warnings on MS-DOS due to constifying of pointers in xmenu.c.

 src/msdos.c (XMenuAddPane): 3rd argument is `const char *' now.
 Adapt all references accordingly.
 src/msdos.h (XMenuAddPane): 3rd argument is `const char *' now.

Merge changes made in Gnus trunk.

gnus-sync.el (gnus-sync-install-hooks, gnus-sync-unload-hook): Don't install `gnus-sync-read' to any hooks by default.  It's buggy.  The user will have to run `gnus-sync-read' manually and wait for Cloudy Gnus.
mm-uu.el (mm-uu-type-alist): Add support for diff starting with "=== modified file".

Improve quality of tests for time stamp overflow.  For example,

without this patch (encode-time 0 0 0 1 1 1152921504606846976)
returns the obviously-bogus value (-948597 62170) on my RHEL 5.5
x86-64 host.  With it, it reports time overflow.
* deps.mk (editfns.o): Depend on ../lib/intprops.h.
* editfns.c: Include limits.h and intprops.h.
(TIME_T_MIN, TIME_T_MAX): New macros.
(time_overflow): Move earlier, to before first use.
(hi_time, lo_time): New functions, for an accurate test for
out-of-range times.
(Fcurrent_time, Fget_internal_run_time, make_time): Use them.
(Fget_internal_run_time): Don't assume time_t fits in int.
(make_time): Use list2 instead of Fcons twice.
(Fdecode_time): More accurate test for out-of-range times.
(check_tm_member): New function.
(Fencode_time): Use it, to test for out-of-range times.

* lisp/progmodes/compile.el (compilation--previous-directory): Fix up
various nil/dead-marker mismatches.
(compilation-directory-properties, compilation-error-properties):
Don't call it at a position past the one we're about to change.

Fixes: debbugs:8014

* lisp/emacs-lisp/bytecomp.el (byte-compile-make-obsolete-variable):
Disable obsolescence warnings in the file that declares it.

* lisp/ebuff-menu.el (electric-buffer-menu-mode-map): Move initialization
into declaration.  Remove redundant and harmful binding.

admin/notes/elpa: Make Bzr checkout directions more explicit.

* editfns.c (time_overflow): New function, refactoring common code.

(Fformat_time_string, Fdecode_time, Fencode_time):
(Fcurrent_time_string): Use it.

Move 'make_time' to be next to its inverse 'lisp_time_argument'.

* dired.c (make_time): Move to ...
* editfns.c (make_time): ... here.
* systime.h: Note the move.

* allout-widgets.el (allout-widgets-tally) Initialize allout-widgets-tally
as a hash table rather than nil to prevent mode-line redisplay warnings.
Also, clarify the module description and fix a comment typoo (sic).

Bump version to 23.3.50

* buffer.c (syms_of_buffer): Remove obsolete comment.

Backport 2011-03-08T01:52:20Z!lekktu@gmail.com from trunk.

* src/w32xfns.c (select_palette): Check success of RealizePalette against
  GDI_ERROR, not zero.

Backport 2011-03-01T03:59:31Z!lekktu@gmail.com from trunk.

*lisp/emacs-lisp/cl-macs.el (lexical-let*): Fix argument name in docstring.

Backport 2011-03-11T00:33:57Z!lekktu@gmail.com from trunk.

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

Fix comparisons of file ownership on MS-Windows for the Administrator user.

 lisp/files.el (file-ownership-preserved-p): Pass `integer' as an
 explicit 2nd argument to `file-attributes'.  If the file's owner
 is the Administrators group on Windows, and the current user is
 Administrator, consider that a match.
 lisp/server.el (server-ensure-safe-dir): Consider server directory
 safe on MS-Windows if its owner is the Administrators group while
 the current Emacs user is Administrator.  Use `=' to compare
 numerical UIDs, since they could be integers or floats.

Fix bug #8221 with description of how HOME is set on Windows.

 doc/emacs/msdog.texi (Windows HOME): Fix the wording to clarify how
 Emacs sets HOME on Windows and where it looks for init files.

Fix MS-Windows build broken by 2011-03-11T07:24:21Z!eggert@cs.ucla.edu.

 src/term.c (encode_terminal_code): Now external again, used by
 w32console.c and msdos.c.
 src/termhooks.h (encode_terminal_code): Declare prototype.
 src/msdos.c (encode_terminal_code): Don't declare prototype.
 src/makefile.w32-in ($(BLD)/term.$(O), ($(BLD)/tparam.$(O)): Depend
 on $(SRC)/tparam.h, see 2011-03-11T07:24:21Z!eggert@cs.ucla.edu.

Fix previous change.

Add Makefile rules for generating html version of Emacs FAQ.

* doc/misc/Makefile.in (HTML_TARGETS): New.
(clean): Delete $HTML_TARGETS.
(emacs-faq.html): New, for use with the gnu.org Emacs webpage.

* image.c: (clear_image_cache, xbm_read_bitmap_data, x_detect_edges):

(jpeg_load, gif_load): Rename locals to avoid shadowing.

* image.c (gif_load): Fix pointer signedness.

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