Some more tinkering with Bug#16051.
[bpt/emacs.git] / src / ChangeLog
index d2ea150..053a498 100644 (file)
@@ -1,5 +1,304 @@
+2013-12-26  Martin Rudalics  <rudalics@gmx.at>
+
+       Some more tinkering with Bug#16051.
+       * window.c (resize_frame_windows): Don't let the size of the
+       root window drop below the frame's default character size.
+       Never ever delete any subwindows - let the window manager do the
+       clipping.
+
+       * w32fns.c (x_set_tool_bar_lines): Rewrite calculation of number
+       of toolbar lines needed when they exceed the height of the root
+       window.
+       (unwind_create_frame_1): New function.
+       (Fx_create_frame): Generally inhibit calling the window
+       configuration change hook here.  Remove extra call to
+       change_frame_size - it's not needed when we don't run the
+       configuration change hook.
+
+2013-12-26  Paul Eggert  <eggert@cs.ucla.edu>
+
+       Fix core dumps with gcc -fsanitize=address and GNU/Linux.
+       On my Fedora 19 platform the core dumps were so big that
+       my desktop became nearly catatonic.
+       * alloc.c (no_sanitize_memcpy) [MAX_SAVE_STACK > 0]: New function.
+       (Fgarbage_collect) [MAX_SAVE_STACK > 0]: Use it.
+       (USE_ALIGNED_MALLOC): Do not define if addresses are sanitized.
+       (mark_memory): Use ATTRIBUTE_NO_SANITIZE_ADDRESS rather than
+       a clang-only syntax.
+       * conf_post.h (__has_feature): New macro, if not already defined.
+       (ADDRESS_SANITIZER, ADDRESS_SANITIZER_WORKAROUND)
+       (ATTRIBUTE_NO_SANITIZE_ADDRESS): New macros.
+
+2013-12-25  Eli Zaretskii  <eliz@gnu.org>
+
+       * w32fns.c (Fw32_shell_execute): Make DOCUMENT absolute only if it
+       is a file name.  (Bug#16252)
+
+2013-12-25  Chong Yidong  <cyd@gnu.org>
+
+       * keyboard.c (Voverriding_terminal_local_map):
+       (Voverriding_local_map): Doc fix.
+
+       * keymap.c (Vemulation_mode_map_alists): Doc fix.
+
+2013-12-24  Eli Zaretskii  <eliz@gnu.org>
+
+       * w32fns.c (Fw32_shell_execute): Ensure DOCUMENT is an absolute
+       file name when it is submitted to ShellExecute.  Simplify code.
+       Don't test DOCUMENT for being a string, as that is enforced by
+       CHECK_STRING.  Doc fix.
+
+2013-12-23  Eli Zaretskii  <eliz@gnu.org>
+
+       * xdisp.c (tool_bar_height): Use WINDOW_PIXEL_WIDTH to set up the
+       iterator X limits, not FRAME_TOTAL_COLS, for consistency with what
+       redisplay_tool_bar does.  Improve and fix commentary.
+       (hscroll_window_tree): Don't assume w->cursor.vpos is within the
+       limits of the glyph matrices.  (Bug#16051)
+       (redisplay_tool_bar): Modify the tool-bar-lines frame parameter
+       only when the new size is different from the old one, and the new
+       size can be achieved given the frame height.
+
+2013-12-23  Jan Djärv  <jan.h.d@swipnet.se>
+
+       * conf_post.h: Use unsigned it for bool_bf if GNUSTEP (Bug#16210).
+
+2013-12-23  Glenn Morris  <rgm@gnu.org>
+
+       * lread.c (Fload): Mention load-prefer-newer in doc.
+
+2013-12-22  Martin Rudalics  <rudalics@gmx.at>
+
+       Handle Bug#16207 by being more restrictive when running hooks.
+       * window.c (unwind_change_frame): New function.
+       (Fset_window_configuration): Don't run configuration change hook
+       while the frame configuration is unsafe.  Call select_window
+       twice.
+
+2013-12-22  Xue Fuqiao  <xfq.free@gmail.com>
+
+       * lread.c (syms_of_lread) <load_prefer_newer>: Doc fix.
+
+2013-12-21  Jan Djärv  <jan.h.d@swipnet.se>
+
+       * nsterm.h: Declare EmacsColor category.
+
+       * nsterm.m (NSColor): Implement EmacsColor category.
+       (ns_get_color): Use colorUsingDefaultColorSpace.
+       (ns_get_color, ns_term_init): Use colorForEmacsRed.
+
+       * nsfns.m (Fxw_color_values): Use colorUsingDefaultColorSpace.
+
+2013-12-21  Eli Zaretskii  <eliz@gnu.org>
+
+       * image.c (fn_png_longjmp) [WINDOWSNT]: Mark the function as
+       having the PNG_NORETURN attribute, to avoid compiler warning in
+       my_png_error.
+
+2013-12-21  YAMAMOTO Mitsuharu  <mituharu@math.s.chiba-u.ac.jp>
+
+       * w32term.h (struct scroll_bar): Remove member `fringe_extended_p'.
+
+       * w32term.c (w32_draw_fringe_bitmap, x_scroll_run): Remove code for
+       fringe background extension.
+       (x_scroll_bar_create): Remove variables `sb_left' and `sb_width',
+       because they are now always the same as `left' and `width',
+       respectively.  Remove code for the case that `width' and
+       `sb_width' are different.
+
+2013-12-20  Martin Rudalics  <rudalics@gmx.at>
+
+       Remove scroll_bar_actual_width from frames.
+       * frame.h (struct frame): Remove scroll_bar_actual_width slot.
+       * frame.c (Fscroll_bar_width): Return scroll bar area width.
+       (x_figure_window_size):
+       * nsterm.m (x_set_window_size):
+       * widget.c (set_frame_size):
+       * w32term.c (x_set_window_size):
+       * xterm.c (x_set_window_size, x_set_window_size_1): Don't set
+       scroll_bar_actual_width.
+
+       Convert scroll_bar members to integers on Windows.
+       * w32term.h (struct scroll_bar): Convert top, left, width,
+       height, start, end and dragging to integers.
+       * w32fns.c (w32_createscrollbar): Remove XINT conversions for
+       scroll_bar members.
+       * w32term.c (w32_set_scroll_bar_thumb)
+       (w32_scroll_bar_handle_click): Remove XINT conversions for
+       scroll_bar members.  Treat bar->dragging as integer.
+       (x_scroll_bar_create): Call ALLOCATE_PSEUDOVECTOR with "top" as
+       first element.  Remove XINT conversions for scroll_bar members.
+       (w32_set_vertical_scroll_bar, x_scroll_bar_report_motion):
+       Remove XINT conversions for scroll_bar members.
+
+       Fix assignment for new window total sizes.
+       * window.c (Fwindow_resize_apply_total): Assign values for
+       minibuffer window.
+
+2013-12-20  Chong Yidong  <cyd@gnu.org>
+
+       * textprop.c (Fadd_face_text_property): Doc fix.  Rename `appendp'
+       argument to `append'.
+
+2013-12-19  Eli Zaretskii  <eliz@gnu.org>
+
+       * xdisp.c (extend_face_to_end_of_line): Use default face, not the
+       current text face, for extending the face of the display margins.
+       (Bug#16192)
+
+       * casefiddle.c (Fupcase_word, Fdowncase_word, Fcapitalize_word):
+       Doc fix.  (Bug#16190)
+
+2013-12-19  Jan Djärv  <jan.h.d@swipnet.se>
+
+       * nsterm.h (KEY_NS_DRAG_FILE, KEY_NS_DRAG_COLOR, KEY_NS_DRAG_TEXT):
+       Remove.
+
+       * nsterm.m (Qfile, Qurl): New.
+       (EV_MODIFIERS2): New macro.
+       (EV_MODIFIERS): Use EV_MODIFIERS2.
+       (ns_term_init): Remove font and color from DND, does not work on
+       newer OSX, and other ports don't have them.
+       (performDragOperation:): Handle modifiers used during drag.
+       Use DRAG_N_DROP_EVENT instead of NS specific events (Bug#8051).
+       Remove global Lisp variables used to communicate with ns-win.el.
+       Remove font and color handling.
+       (syms_of_nsterm): Defsym Qfile and Qurl.
+
+2013-12-19  Anders Lindgren <andlind@gmail.com>
+
+       * nsterm.m (NSTRACE_SIZE, NSTRACE_RECT): New macros.
+       (ns_constrain_all_frames, x_set_offset): Remove assignment to
+       dont_constrain.
+       (updateFrameSize:, windowWillResize:toSize:): Add trace.
+       (constrainFrameRect): Remove special case nr_screens == 1.
+       Don't constrain size to size of view.
+
+       * nsterm.h (ns_output): Remove dont_constrain.
+
+2013-12-19  Anders Lindgren <andlind@gmail.com>
+
+       * nsterm.m (mouseDown:): Generate HORIZ_WHEEL_EVENT.
+
+2013-12-18  Paul Eggert  <eggert@cs.ucla.edu>
+
+       Minor fixes for recent openp changes.
+       * lisp.h (GCPRO7): New macro.
+       * lread.c (openp): Use bool for boolean; all callers changed.
+       Protect save_string from GC.  Don't assume that file descriptors
+       are nonzero.  Redo save_mtime comparison to avoid bogus GCC
+       warning about uninitialized variable.
+
+2013-12-18  Eli Zaretskii  <eliz@gnu.org>
+
+       * w32fns.c (emacs_abort): Use intptr_t as argument of
+       INT_BUFSIZE_BOUND, to avoid compiler warnings.
+
+2013-12-18  Glenn Morris  <rgm@gnu.org>
+
+       * lread.c (Fload): Pass load_prefer_newer to openp.
+       Don't bother checking mtime if openp already did it.
+       (openp): Add `newer' argument, to check all suffixes
+       and find the newest file.
+       (syms_of_lread) <load_prefer_newer>: New option.  (Bug#2061)
+       * callproc.c (call_process):
+       * charset.c (load_charset_map_from_file):
+       * emacs.c (init_cmdargs):
+       * image.c (x_create_bitmap_from_file, x_find_image_file):
+       * lisp.h (openp):
+       * lread.c (Flocate_file_internal):
+       * process.c (Fformat_network_address):
+       * sound.c (Fplay_sound_internal):
+       * w32.c (check_windows_init_file):
+       * w32proc.c (sys_spawnve): Update for new arg spec of openp.
+
+       * emacs.c (standard_args) [HAVE_NS]: Remove -disable-font-backend.
+
+2013-12-17  Eli Zaretskii  <eliz@gnu.org>
+
+       * w32.c (getloadavg): Don't index samples[] array with negative
+       indices.  Recover from wall-clock time being set backwards.
+
+       * w32term.c (w32_initialize): Declare the argument of
+       set_user_model as const.
+
+       * w32menu.c <MessageBoxW_Proc>: Fix argument declarations.
+       (w32_menu_show): Constify some arguments passed to MessageBox.
+
+       * w32uniscribe.c (uniscribe_font_driver): Use LISP_INITIALLY_ZERO
+       to initialize Lisp objects.
+
+       * w32font.c (w32font_driver): Use LISP_INITIALLY_ZERO to
+       initialize Lisp objects.
+
+       * frame.c (x_set_frame_parameters) [HAVE_X_WINDOWS]: Declare and
+       use variables used only on X under that condition.
+
+       * fileio.c (Fcopy_file) [!WINDOWSNT]: Don't declare on Windows
+       variables not used there.
+
+2013-12-16  Paul Eggert  <eggert@cs.ucla.edu>
+
+       Fix problems with CANNOT_DUMP and EMACSLOADPATH.
+       * lread.c (init_lread): If CANNOT_DUMP, we can't be dumping.
+
+2013-12-16  Eli Zaretskii  <eliz@gnu.org>
+
+       * xdisp.c (Fmove_point_visually): Fix subtle bugs in the fallback
+       code, revealed in presence of R2L characters, character
+       compositions, and display vectors.  A better fix for Bug#16148.
+       (extend_face_to_end_of_line): Don't reference tool_bar_window in
+       GTK and NS builds, they don't have this member of struct frame.
+
+       * dispextern.h (struct composition_it): Correct a comment for the
+       'width' member.
+
+2013-12-16  Paul Eggert  <eggert@cs.ucla.edu>
+
+       * font.h (valid_font_driver) [!ENABLE_CHECKING]: Define a dummy.
+       This prevents a compilation error on C compilers that do not
+       default functions to return 'int' if not declared.  Also, add
+       INLINE_HEADER_BEGIN and INLINE_HEADER_END to this include file,
+       since it now uses inline functions.
+
+2013-12-16  Eli Zaretskii  <eliz@gnu.org>
+
+       * xdisp.c (extend_face_to_end_of_line): Don't fill background of
+       display margins on mode line, header line, and in the frame's
+       tool-bar window.  (Bug#16165)
+
+2013-12-16  Andreas Schwab  <schwab@suse.de>
+
+       * gnutls.c (Fgnutls_boot): Properly check Flistp return value.
+
+2013-12-16  Teodor Zlatanov  <tzz@lifelogs.com>
+
+       * gnutls.c (Fgnutls_boot): Use `Flistp' instead of
+       `CHECK_LIST_CONS`.
+
+2013-12-16  Martin Rudalics  <rudalics@gmx.at>
+
+       * w32term.c (w32_enable_frame_resize_hack): Default to 1.
+
+2013-12-16  Dmitry Antipov  <dmantipov@yandex.ru>
+
+       * font.c (valid_font_driver) [ENABLE_CHECKING]: New function
+       intended to find bogus pointers in font objects (Bug#16140).
+       * font.h (valid_font_driver) [ENABLE_CHECKING]: Add prototype.
+       * alloc.c (cleanup_vector): Use valid_font_driver in eassert.
+       (compact_font_cache_entry, compact_font_caches) [!HAVE_NTGUI]:
+       Disable for MS-Windows due to Bug#15876; apparently this
+       requires more or less substantial changes in fontset code.
+       * xfont.c (xfont_close):
+       * xftfont.c (xftfont_close): Call x_display_info_for_display
+       to check whether 'Display *' is valid (Bug#16093 and probably
+       Bug#16069).
+
 2013-12-15  Eli Zaretskii  <eliz@gnu.org>
 
+       * fileio.c (Fexpand_file_name) [WINDOWSNT]: Fix conditionals.
+       Reported by Juanma Barranquero <lekktu@gmail.com>.
+
        * process.c (Fprocess_send_eof): Don't crash if someone tries to
        open a pty on MS-Windows.  (Bug#16152)