* xdisp.c (display_mode_element): Don't assume strlen fits in int.
[bpt/emacs.git] / src / ChangeLog
index 6562df0..8c53034 100644 (file)
@@ -1,4 +1,284 @@
-2011-06-13  Paul Eggert  <eggert@cs.ucla.edu>
+2011-06-21  Paul Eggert  <eggert@cs.ucla.edu>
+
+       * xdisp.c (message_log_check_duplicate): Return intmax_t,
+       not unsigned long, as we prefer signed integers.  All callers changed.
+       Detect integer overflow in repeat count.
+       (message_dolog): Don't assume print length fits in 39 bytes.
+       (display_mode_element): Don't assume strlen fits in int.
+
+       * termcap.c: Don't assume sizes fit in int and never overflow.
+       (struct termcap_buffer, tgetent): Use ptrdiff_t, not int, for sizes.
+       (gobble_line): Check for size-calculation overflow.
+
+       * minibuf.c (Fread_buffer):
+       * lread.c (intern, intern_c_string):
+       * image.c (xpm_scan) [HAVE_NS && !HAVE_XPM]:
+       Don't assume string length fits in int.
+
+       * keyboard.c (parse_tool_bar_item):
+       * gtkutil.c (style_changed_cb): Avoid need for strlen.
+
+       * font.c: Don't assume string length fits in int.
+       (font_parse_xlfd, font_parse_fcname, font_unparse_fcname):
+       Use ptrdiff_t, not int.
+       (font_intern_prop): Don't assume string length fits in int.
+       Don't assume integer property fits in fixnum.
+       * font.h (font_intern_prop): 2nd arg is now ptrdiff_t, not int.
+
+       * filelock.c: Fix some buffer overrun and integer overflow issues.
+       (get_boot_time): Don't assume gzip command string fits in 100 bytes.
+       Reformulate so as not to need the command string.
+       Invoke gzip -cd rather than gunzip, as it's more portable.
+       (lock_info_type, lock_file_1, lock_file):
+       Don't assume pid_t and time_t fit in unsigned long.
+       (LOCK_PID_MAX): Remove; we now use more-reliable bounds.
+       (current_lock_owner): Prefer signed type for sizes.
+       Use memcpy, not strncpy, where memcpy is what is really wanted.
+       Don't assume (via atoi) that time_t and pid_t fit in int.
+       Check for time_t and/or pid_t out of range, e.g., via a network share.
+       Don't alloca where an auto var works fine.
+
+       * fileio.c: Fix some integer overflow issues.
+       (file_name_as_directory, Fexpand_file_name, Fsubstitute_in_file_name):
+       Don't assume string length fits in int.
+       (directory_file_name): Don't assume string length fits in long.
+       (make_temp_name): Don't assume pid fits in int, or that its print
+       length is less than 20.
+
+       * data.c (Fsubr_name): Rewrite to avoid a strlen call.
+
+       * coding.c (make_subsidiaries): Don't assume string length fits in int.
+
+       * callproc.c (child_setup): Rewrite to avoid two strlen calls.
+
+       * process.c (Fformat_network_address): Use EMACS_INT, not EMACS_UINT.
+       We prefer signed integers, even for size calculations.
+
+       * emacs.c: Don't assume string length fits in 'int'.
+       (DEFINE_DUMMY_FUNCTION, sort_args): Use ptrdiff_t, not int.
+       (main): Don't invoke strlen when not needed.
+
+       * dbusbind.c (XD_ERROR): Don't arbitrarily truncate string.
+       (XD_DEBUG_MESSAGE): Don't waste a byte.
+
+       * callproc.c (getenv_internal_1, getenv_internal)
+       (Fgetenv_internal):
+       * buffer.c (init_buffer): Don't assume string length fits in 'int'.
+
+       * lread.c (invalid_syntax): Omit length argument.
+       All uses changed.  This doesn't fix a bug, but it simplifies the
+       code away from its former Hollerith-constant appearance, and it's
+       one less 'int' to worry about when looking at integer-overflow issues.
+       (string_to_number): Simplify 2011-04-26 change by invoking xsignal1.
+
+       * lisp.h (DEFUN): Remove bogus use of sizeof (struct Lisp_Subr).
+       This didn't break anything, but it didn't help either.
+       It's confusing to put a bogus integer in a place where the actual
+       value does not matter.
+       (LIST_END_P): Remove unused macro and its bogus comment.
+       (make_fixnum_or_float): Remove unnecessary cast to EMACS_INT.
+
+       * lisp.h (union Lisp_Object.i): EMACS_INT, not EMACS_UINT.
+       This is for consistency with the ordinary, non-USE_LISP_UNION_TYPE,
+       implementation.
+       (struct Lisp_Bool_Vector.size): EMACS_INT, not EMACS_UINT.
+       We prefer signed types, and the value cannot exceed the EMACS_INT
+       range anyway (because otherwise the length would not be representable).
+       (XSET) [USE_LISP_UNION_TYPE]: Use uintptr_t and intptr_t,
+       not EMACS_UINT and EMACS_INT, when converting pointer to integer.
+       This avoids a GCC warning when WIDE_EMACS_INT.
+
+       * indent.c (sane_tab_width): New function.
+       (current_column, scan_for_column, Findent_to, position_indentation)
+       (compute_motion): Use it.  This is just for clarity.
+       (Fcompute_motion): Don't assume hscroll and tab offset fit in int.
+
+       * image.c (xbm_image_p): Don't assume stated width, height fit in int.
+
+       * lisp.h (lint_assume): New macro.
+       * composite.c (composition_gstring_put_cache):
+       * ftfont.c (ftfont_shape_by_flt): Use it to pacify GCC 4.6.0.
+
+       * editfns.c, insdel.c:
+       Omit unnecessary forward decls, to simplify future changes.
+
+       * ftfont.c (ftfont_shape_by_flt): Use signed integers for lengths.
+
+       * font.c (Ffont_shape_gstring): Don't assume glyph len fits in 'int'.
+
+       * fns.c (Ffillarray): Don't assume bool vector size fits in 'int'.
+       Use much-faster test for byte-length change.
+       Don't assume string byte-length fits in 'int'.
+       Check that character arg fits in 'int'.
+       (mapcar1): Declare byte as byte, for clarity.
+
+       * alloc.c (Fmake_bool_vector): Avoid unnecessary multiplication.
+
+       * fns.c (concat): Catch string overflow earlier.
+       Do not rely on integer wraparound.
+
+       * dispextern.h (struct it.overlay_strings_charpos)
+       (struct it.selective): Now EMACS_INT, not int.
+       * xdisp.c (forward_to_next_line_start)
+       (back_to_previous_visible_line_start)
+       (reseat_at_next_visible_line_start, next_element_from_buffer):
+       Don't arbitrarily truncate the value of 'selective' to int.
+
+       * xdisp.c (init_iterator): Use XINT, not XFASTINT; it might be < 0.
+
+       * composite.c: Don't truncate sizes to 'int'.
+       (composition_gstring_p, composition_reseat_it)
+       (composition_adjust_point): Use EMACS_INT, not int.
+       (get_composition_id, composition_gstring_put_cache): Use EMACS_INT,
+       not EMACS_UINT, for indexes.
+
+       * category.h (CATEGORY_SET_P): Remove unnecessary cast to EMACS_INT.
+
+       * buffer.c: Include <verify.h>.
+       (struct sortvec.priority, struct sortstr.priority):
+       Now EMACS_INT, not int.
+       (compare_overlays, cmp_for_strings): Avoid subtraction overflow.
+       (struct sortstr.size, record_overlay_string)
+       (struct sortstrlist.size, struct sortlist.used):
+       Don't truncate size to int.
+       (record_overlay_string): Check for size-calculation overflow.
+       (init_buffer_once): Check at compile-time, not run-time.
+
+2011-06-20  Deniz Dogan  <deniz@dogan.se>
+
+       * process.c (Fset_process_buffer): Clarify return value in
+       docstring.
+
+2011-06-18  Chong Yidong  <cyd@stupidchicken.com>
+
+       * dispnew.c (add_window_display_history): Use BVAR.
+
+       * xdisp.c (debug_method_add): Use BVAR.
+       (check_window_end, dump_glyph_matrix, dump_glyph)
+       (dump_glyph_row, dump_glyph_string): Convert arglist to ANSI C.
+
+       * xfaces.c (check_lface_attrs, check_lface, dump_realized_face):
+       Likewise.
+
+       * xfns.c (Fx_create_frame, x_create_tip_frame): Delay image cache
+       check till after the cache is created in init_frame_faces.
+
+2011-06-17  Stefan Monnier  <monnier@iro.umontreal.ca>
+
+       * fns.c (Fsafe_length): Yet another int/Lisp_Object mixup.
+
+2011-06-16  Paul Eggert  <eggert@cs.ucla.edu>
+
+       * lisp.h: Include <limits.h>, for INT_MAX, LONG_MAX, LLONG_MAX.
+       Without this, prin1 mishandles Lisp_Misc_Save_Value printing on
+       hosts with pre-C99 libraries, because pD is wrongly defined to "t".
+
+       Improve buffer-overflow checking (Bug#8873).
+       * fileio.c (Finsert_file_contents):
+       * insdel.c (insert_from_buffer_1, replace_range, replace_range_2):
+       Remove the old (too-loose) buffer overflow checks.
+       They weren't needed, since make_gap checks for buffer overflow.
+       * insdel.c (make_gap_larger): Catch buffer overflows that were missed.
+       The old code merely checked for Emacs fixnum overflow, and relied
+       on undefined (wraparound) behavior.  The new code avoids undefined
+       behavior, and also checks for ptrdiff_t and/or size_t overflow.
+
+       * editfns.c (Finsert_char): Don't dump core with very negative counts.
+       Tune.  Don't use wider integers than needed.  Don't use alloca.
+       Use a bigger 'string' buffer.  Rewrite to avoid 'n > 0' test.
+
+       * insdel.c (replace_range): Fix buf overflow when insbytes < outgoing.
+
+       * insdel.c, lisp.h (buffer_overflow): New function.
+       (insert_from_buffer_1, replace_range, replace_range_2):
+       * insdel.c (make_gap_larger):
+       * editfns.c (Finsert_char):
+       * fileio.c (Finsert_file_contents): Use it, to normalize wording.
+
+       * buffer.h (BUF_BYTES_MAX): Cast to ptrdiff_t so that it's signed.
+
+2011-06-15  Paul Eggert  <eggert@cs.ucla.edu>
+
+       Integer overflow and signedness fixes (Bug#8873).
+
+       * ccl.c (ASCENDING_ORDER): New macro, to work around GCC bug 43772.
+       (GET_CCL_RANGE, IN_INT_RANGE): Use it.
+
+       * fileio.c: Don't assume EMACS_INT fits in off_t.
+       (emacs_lseek): New static function.
+       (Finsert_file_contents, Fwrite_region): Use it.
+       Use SEEK_SET, SEEK_CUR, SEEK_END as appropriate.
+
+       * fns.c (Fload_average): Don't assume 100 * load average fits in int.
+
+       * fns.c: Don't overflow int when computing a list length.
+       * fns.c (QUIT_COUNT_HEURISTIC): New constant.
+       (Flength, Fsafe_length): Use EMACS_INT, not int, to avoid unwanted
+       truncation on 64-bit hosts.  Check for QUIT every
+       QUIT_COUNT_HEURISTIC entries rather than every other entry; that's
+       faster and is responsive enough.
+       (Flength): Report an error instead of overflowing an integer.
+       (Fsafe_length): Return a float if the value is not representable
+       as a fixnum.  This shouldn't happen except in contrived situations.
+       (Fnthcdr, Fsort): Don't assume list length fits in int.
+       (Fcopy_sequence): Don't assume vector length fits in int.
+
+       * alloc.c: Check that resized vectors' lengths fit in fixnums.
+       (header_size, word_size): New constants.
+       (allocate_vectorlike): Don't check size overflow here.
+       (allocate_vector): Check it here instead, since this is the only
+       caller of allocate_vectorlike that could cause overflow.
+       Check that the new vector's length is representable as a fixnum.
+
+       * fns.c (next_almost_prime): Don't return a multiple of 3 or 5.
+       The previous code was bogus.  For example, next_almost_prime (32)
+       returned 39, which is undesirable as it is a multiple of 3; and
+       next_almost_prime (24) returned 25, which is a multiple of 5 so
+       why was the code bothering to check for multiples of 7?
+
+       * bytecode.c (exec_byte_code): Use ptrdiff_t, not int, for vector length.
+
+       * eval.c, doprnt.c (SIZE_MAX): Remove; inttypes.h defines this now.
+
+       Variadic C functions now count arguments with ptrdiff_t.
+       This partly undoes my 2011-03-30 change, which replaced int with size_t.
+       Back then I didn't know that the Emacs coding style prefers signed int.
+       Also, in the meantime I found a few more instances where arguments
+       were being counted with int, which may truncate counts on 64-bit
+       machines, or EMACS_INT, which may be unnecessarily wide.
+       * lisp.h (struct Lisp_Subr.function.aMANY)
+       (DEFUN_ARGS_MANY, internal_condition_case_n, safe_call):
+       Arg counts are now ptrdiff_t, not size_t.
+       All variadic functions and their callers changed accordingly.
+       (struct gcpro.nvars): Now size_t, not size_t.  All uses changed.
+       * bytecode.c (exec_byte_code): Check maxdepth for overflow,
+       to avoid potential buffer overrun.  Don't assume arg counts fit in 'int'.
+       * callint.c (Fcall_interactively): Check arg count for overflow,
+       to avoid potential buffer overrun.  Use signed char, not 'int',
+       for 'varies' array, so that we needn't bother to check its size
+       calculation for overflow.
+       * editfns.c (Fformat): Use ptrdiff_t, not EMACS_INT, to count args.
+       * eval.c (apply_lambda):
+       * fns.c (Fmapconcat): Use XFASTINT, not XINT, to get args length.
+       (struct textprop_rec.argnum): Now ptrdiff_t, not int.  All uses changed.
+       (mapconcat): Use ptrdiff_t, not int and EMACS_INT, to count args.
+
+       * callint.c (Fcall_interactively): Don't use index var as event count.
+
+       * vm-limit.c (check_memory_limits): Fix incorrect extern function decls.
+       * mem-limits.h (SIZE): Remove; no longer used.
+
+       * xterm.c (x_alloc_nearest_color_1): Prefer int to long when int works.
+
+       Remove unnecessary casts.
+       * xterm.c (x_term_init):
+       * xfns.c (x_set_border_pixel):
+       * widget.c (create_frame_gcs): Remove casts to unsigned long etc.
+       These aren't needed now that we assume ANSI C.
+
+       * sound.c (Fplay_sound_internal): Remove cast to unsigned long.
+       It's more likely to cause problems (due to unsigned overflow)
+       than to cure them.
 
        * dired.c (Ffile_attributes): Don't use 32-bit hack on 64-bit hosts.
 
        GLYPH_CODE_FACE returns EMACS_INT, not int.
        * dispextern.h (merge_faces):
        * xfaces.c (merge_faces):
-       * xdisp.c (get_next_display_element):
-       (next_element_from_display_vector): Don't assume EMACS_INT fits in int.
+       * xdisp.c (get_next_display_element, next_element_from_display_vector):
+       Don't assume EMACS_INT fits in int.
 
        * character.h (CHAR_VALID_P): Remove unused parameter.
        * fontset.c, lisp.h, xdisp.c: All uses changed.
        arguments are no wider than unsigned, as a compile-time check
        to prevent future regressions in this area.
        * data.c (Faset):
-       * editfns.c (Fchar_to_string, general_insert_function, Finsert_char):
+       * editfns.c (Fchar_to_string, general_insert_function, Finsert_char)
        (Fsubst_char_in_region):
        * fns.c (concat):
        * xdisp.c (decode_mode_spec_coding):
        characters.  Without this test, these functions did the wrong
        thing with wildly out-of-range values on 64-bit hosts.
 
-2011-06-12  Paul Eggert  <eggert@cs.ucla.edu>
-
        Remove incorrect casts to 'unsigned' that lose info on 64-bit hosts.
        These casts should not be needed on 32-bit hosts, either.
        * keyboard.c (read_char):
        This fixes comparison bugs on 64-bit hosts.
        (ASCII_CHAR_P): Use it.
        * casefiddle.c (casify_object):
-       * character.h (ASCII_BYTE_P, CHAR_VALID_P):
+       * character.h (ASCII_BYTE_P, CHAR_VALID_P)
        (SINGLE_BYTE_CHAR_P, CHAR_STRING):
        * composite.h (COMPOSITION_ENCODE_RULE_VALID):
        * dispextern.h (FACE_FROM_ID):
        * keyboard.c (read_char): Use UNSIGNED_CMP.
 
-2011-06-11  Paul Eggert  <eggert@cs.ucla.edu>
-
        * xmenu.c (dialog_selection_callback) [!USE_GTK]: Cast to intptr_t,
        not to EMACS_INT, to avoid GCC warning.
 
        The cast incorrectly truncated 64-bit byte offsets to 32 bits, and
        isn't needed on 32-bit machines.
 
-       * buffer.c (Fgenerate_new_buffer_name): Use EMACS_INT for count, not int.
+       * buffer.c (Fgenerate_new_buffer_name):
+       Use EMACS_INT for count, not int.
        (advance_to_char_boundary): Return EMACS_INT, not int.
 
        * data.c (Qcompiled_function): Now static.
        * alloc.c: Use EMACS_INT, not int, to count objects.
        (total_conses, total_markers, total_symbols, total_vector_size)
        (total_free_conses, total_free_markers, total_free_symbols)
-       (total_free_floats, total_floats, total_free_intervals, total_intervals)
-       (total_strings, total_free_strings):
+       (total_free_floats, total_floats, total_free_intervals)
+       (total_intervals, total_strings, total_free_strings):
        Now EMACS_INT, not int.  All uses changed.
        (Fgarbage_collect): Compute overall total using a double, so that
        integer overflow is less likely to be a problem.  Check for overflow
 
        * alloc.c (Fmake_string): Check for out-of-range init.
 
+2011-06-15  Stefan Monnier  <monnier@iro.umontreal.ca>
+
+       * eval.c (Fdefvaralias): Also mark the target as variable-special-p.
+
+2011-06-14  Jan Djärv  <jan.h.d@swipnet.se>
+
+       * xfns.c (x_set_scroll_bar_default_width): Remove argument to
+       xg_get_default_scrollbar_width.
+
+       * gtkutil.c: Include emacsgtkfixed.h if HAVE_GTK3.
+       (int_gtk_range_get_value): Move to the scroll bar part of the file.
+       (style_changed_cb): Call update_theme_scrollbar_width and call
+       x_set_scroll_bar_default_width and xg_frame_set_char_size for
+       all frames (Bug#8505).
+       (xg_create_frame_widgets): Call emacs_fixed_new if HAVE_GTK3 (Bug#8505).
+       Call gtk_window_set_resizable if HAVE_GTK3.
+       (x_wm_set_size_hint): Call emacs_fixed_set_min_size with min width
+       and height if HAVE_GTK3 (Bug#8505).
+       (scroll_bar_width_for_theme): New variable.
+       (update_theme_scrollbar_width): New function.
+       (xg_get_default_scrollbar_width): Move code to
+       update_theme_scrollbar_width, just return scroll_bar_width_for_theme.
+       (xg_initialize): Call update_theme_scrollbar_width.
+
+       * gtkutil.h (xg_get_default_scrollbar_width): Remove argument.
+
+       * emacsgtkfixed.c, emacsgtkfixed.h: New files.
+
+2011-06-12  Martin Rudalics  <rudalics@gmx.at>
+
+       * frame.c (make_frame): Call other_buffer_safely instead of
+       other_buffer.
+
+       * window.c (temp_output_buffer_show): Call display_buffer with
+       second argument Vtemp_buffer_show_specifiers and reset latter
+       immediately after the call.
+       (Vtemp_buffer_show_specifiers): New variable.
+       (auto_window_vscroll_p, next_screen_context_lines)
+       (Vscroll_preserve_screen_position): Remove leading asterisks from
+       doc-strings.
+
+2011-06-12  Paul Eggert  <eggert@cs.ucla.edu>
+
+       Fix minor problems found by GCC 4.6.0 static checking.
+       * buffer.c (Qclone_number): Remove for now, as it's unused.
+       (record_buffer, Funrecord_buffer): Rename local to avoid shadowing.
+       (record_buffer): Remove unused local.
+       * frame.c (other_visible_frames, frame_buffer_list): Now static.
+       (set_frame_buffer_list): Remove; unused.
+       * frame.h (other_visible_frames): Remove decl.
+       * keyboard.h (menu_items_inuse): Declare only if USE_GTK || USE_MOTIF.
+       * lisp.h (frame_buffer_list, set_frame_buffer_list): Remove decls.
+       (add_gpm_wait_descriptor, delete_gpm_wait_descriptor): Declare only
+       if HAVE_GPM.
+       * menu.c (menu_items_inuse): Now static unless USE_GTK || USE_MOTIF.
+       * process.c (add_gpm_wait_descriptor, delete_gpm_wait_descriptor):
+       Define only if HAVE_GPM.
+       * widget.c (EmacsFrameResize, emacsFrameClassRec): Now static.
+       (update_hints_inhibit): Remove; never set.  All uses removed.
+       * widgetprv.h (emacsFrameClassRec): Remove decl.
+       * window.c (delete_deletable_window): Now returns void, since it
+       wasn't returning anything.
+       (compare_window_configurations): Remove unused locals.
+       * xfns.c (x_set_scroll_bar_default_width): Remove unused locals.
+       * xmenu.c (x_menu_set_in_use): Define only if USE_GTK || USE_MOTIF.
+       (dialog_selection_callback) [!USE_GTK]: Prefer intptr_t for integers
+       the same widths as pointers.  This follows up on the 2011-05-06 patch.
+       * xterm.c (x_alloc_lighter_color_for_widget): Define only if USE_LUCID.
+       * xterm.h: Likewise.
+       (x_menu_set_in_use): Declare only if USE_GTK || USE_MOTIF.
+
+2011-06-12  Juanma Barranquero  <lekktu@gmail.com>
+
+       * makefile.w32-in: Update dependencies.
+       (LISP_H): Add lib/intprops.h.
+
+2011-06-11  Chong Yidong  <cyd@stupidchicken.com>
+
+       * image.c (gif_load): Add animation frame delay to the metadata.
+       (syms_of_image): Use DEFSYM.  New symbol `delay'.
+
+2011-06-11  Martin Rudalics  <rudalics@gmx.at>
+
+       * window.c (delete_deletable_window): Re-add.
+       (Fset_window_configuration): Rewrite to handle dead buffers and
+       consequently deletable windows.
+       (window_tree, Fwindow_tree): Remove.  Supply functionality in
+       window.el.
+       (compare_window_configurations): Simplify code.
+
+2011-06-11  Andreas Schwab  <schwab@linux-m68k.org>
+
+       * image.c (imagemagick_load_image): Fix type mismatch.
+       (Fimagemagick_types): Likewise.
+
+       * window.h (replace_buffer_in_windows): Declare.
+
+2011-06-11  Martin Rudalics  <rudalics@gmx.at>
+
+       * buffer.c: New Lisp objects Qbuffer_list_update_hook and
+       Qclone_number.  Remove external declaration of Qdelete_window.
+       (Fbuffer_list): Rewrite doc-string.  Minor restructuring of
+       code.
+       (Fget_buffer_create, Fmake_indirect_buffer, Frename_buffer): Run
+       Qbuffer_list_update_hook if allowed.
+       (Fother_buffer): Rewrite doc-string.  Major rewrite for new
+       buffer list implementation.
+       (other_buffer_safely): New function.
+       (Fkill_buffer): Replace call to replace_buffer_in_all_windows by
+       calls to replace_buffer_in_windows and
+       replace_buffer_in_windows_safely.  Run Qbuffer_list_update_hook
+       if allowed.
+       (record_buffer): Inhibit quitting and rewrite using quittable
+       functions.  Run Qbuffer_list_update_hook if allowed.
+       (Frecord_buffer, Funrecord_buffer): New functions.
+       (switch_to_buffer_1, Fswitch_to_buffer): Remove.  Move
+       switch-to-buffer to window.el.
+       (bury-buffer): Move to window.el.
+       (Vbuffer_list_update_hook): New variable.
+
+       * lisp.h (other_buffer_safely): Add prototype in buffer.c
+       section.
+
+       * window.h (resize_frame_windows): Move up in code.
+       (Fwindow_frame): Remove EXFUN.
+       (replace_buffer_in_all_windows): Remove prototype.
+       (replace_buffer_in_windows_safely): Add prototype.
+
+       * window.c: Declare Qdelete_window static again.  Move down
+       declaration of select_count.
+       (Fnext_window, Fprevious_window): Rewrite doc-strings.
+       (Fother_window): Move to window.el.
+       (window_loop): Remove DELETE_BUFFER_WINDOWS and UNSHOW_BUFFER
+       cases.  Add REPLACE_BUFFER_IN_WINDOWS_SAFELY case.
+       (Fdelete_windows_on, Freplace_buffer_in_windows): Move to
+       window.el.
+       (replace_buffer_in_windows): Implement by calling
+       Qreplace_buffer_in_windows.
+       (replace_buffer_in_all_windows): Remove with some functionality
+       moved into replace_buffer_in_windows_safely.
+       (replace_buffer_in_windows_safely): New function.
+       (select_window_norecord, select_frame_norecord): Move in front
+       of run_window_configuration_change_hook.  Remove now obsolete
+       declarations.
+       (Fset_window_buffer): Rewrite doc-string.  Call
+       Qrecord_window_buffer.
+       (keys_of_window): Move binding for other-window to window.el.
+
 2011-06-11  Chong Yidong  <cyd@stupidchicken.com>
 
        * dispextern.h (struct image): Replace data member, whose int_val
 2011-05-18  Christoph Scholtes  <cschol2112@googlemail.com>
 
        * menu.c: Include limits.h (fixes the MS-Windows build broken by
-       revision 104625).
+       2011-06-18T18:49:19Z!cyd@stupidchicken.com).
 
 2011-05-18  Paul Eggert  <eggert@cs.ucla.edu>