* frame.c (Fset_frame_height, Fset_frame_width): Mention nil frame in docstring.
[bpt/emacs.git] / src / ChangeLog
index ab90682..1c4880e 100644 (file)
@@ -1,6 +1,357 @@
+2013-07-31  Dmitry Antipov  <dmantipov@yandex.ru>
+
+       * frame.c (Fmake_terminal_frame): Use store_in_alist to setup
+       frame parameters and call to Fmodify_frame_parameters just once.
+       (Fset_frame_height, Fset_frame_width): Mention nil frame in docstring.
+       (Fset_frame_size, Fset_frame_position): Use decode_live_frame
+       and mention nil frame in docstring.
+
+2013-07-31  Dmitry Antipov  <dmantipov@yandex.ru>
+
+       * frame.c (make_frame, x_set_frame_parameters): Use bool for boolean.
+       (x_figure_window_size): Likewise.  Adjust to return long.
+       (syms_of_frame): Do not DEFSYM Qterminal_live_p.
+       (toplevel): Move Qterminal_live_p to...
+       * terminal.c (toplevel): ...here, make it static, and...
+       (syms_of_terminal): ...DEFSYM here.
+       * frame.h (Qterminal_live_p): Remove declaration.
+       (make_frame, x_figure_window_size): Adjust prototype.
+       * nsfns.m (Fx_create_frame): Use long for window flags.
+
+2013-07-30  Paul Eggert  <eggert@cs.ucla.edu>
+
+       Fix tempfile bug on platforms lacking mkostemp and mkstemp (Bug#14986).
+       * callproc.c (create_temp_file) [! (HAVE_MKOSTEMP || HAVE_MKSTEMP)]:
+       Do not assume that emacs_close (INT_MAX) is a no-op.
+
+2013-07-30  Dmitry Antipov  <dmantipov@yandex.ru>
+
+       * xfaces.c (make_face_cache): For struct face_cache, prefer
+       xmalloc to xzalloc and so avoid redundant call to memset.
+       (Finternal_set_lisp_face_attribute): Fix comment typo and style.
+
+2013-07-30  Dmitry Antipov  <dmantipov@yandex.ru>
+
+       * fringe.c (draw_window_fringes, update_window_fringes)
+       (compute_fringe_widths):
+       * w32term.c (x_draw_glyph_string):
+       * window.c (candidate_window_p, Frecenter):
+       * xfaces.c (realize_basic_faces, realize_default_face)
+       (Fbitmap_space_p, Finternal_set_lisp_face_attribute)
+       (x_update_menu_appearance, face_attr_equal_p, lface_equal_p):
+       * xfns.c (x_set_cursor_color, xic_free_xfontset):
+       * xmenu.c (Fx_menu_bar_open_internal):
+       * xselect.c (x_reply_selection_request, Fx_get_atom_name):
+       * xsettings.c (xft_settings_event):
+       * xterm.c (x_draw_glyph_string, x_had_errors_p):
+       Use bool for booleans.  Adjust style and comments where
+       appropriate.
+       * dispextern.h (draw_window_fringes, update_window_fringes)
+       (compute_fringe_widths):
+       * xterm.h (x_had_errors_p): Adjust prototype.
+
+2013-07-30  Dmitry Antipov  <dmantipov@yandex.ru>
+
+       * frame.c (Fmodify_frame_parameters): Always check 2nd arg with
+       CHECK_LIST.  Rewrite the loop to avoid useless local variable.
+
+2013-07-29  Dmitry Antipov  <dmantipov@yandex.ru>
+
+       * fns.c (toplevel): Remove comment before Fsafe_length because
+       it checks for QUIT.
+
+2013-07-28  Paul Eggert  <eggert@cs.ucla.edu>
+
+       * frame.c (delete_frame): Avoid unnecessary 'this_f' test (Bug#14970).
+
+2013-07-28  Eli Zaretskii  <eliz@gnu.org>
+
+       * w32fns.c (w32_wnd_proc) <WM_IME_STARTCOMPOSITION>: Make sure the
+       frame which got the message is still alive, before dereferencing
+       its pointer.  (Bug#14970)
+
+       * frame.c (delete_frame): Test "this" frame's minibuffer window to
+       be a live window, before using it as such.  (Bug#14970)
+
+2013-07-27  Eli Zaretskii  <eliz@gnu.org>
+
+       * w32term.c (w32_read_socket) <WM_KILLFOCUS>: Call
+       w32_detect_focus_change instead of doing part of its job by hand.
+       This fixes the problem whereby FOCUS_OUT events were not sent to
+       the event queue.
+
+2013-07-26  Eli Zaretskii  <eliz@gnu.org>
+
+       * process.c (Fprocess_list): Doc fix.
+
+       * w32term.c (w32_read_socket) <WM_EMACS_PAINT>: Warn about frame
+       being re-exposed only if it didn't ask to become visible.
+       <WM_SIZE>: Under SIZE_RESTORED, only set the frame visible if it
+       was previously iconified.  (Bug#14841)
+       (x_iconify_frame): Mark the frame iconified.
+
+2013-07-26  Paul Eggert  <eggert@cs.ucla.edu>
+
+       Fix minor problems found by static checking.
+       * eval.c (get_backtrace_frame, backtrace_eval_unrewind): Now static.
+       (backtrace_eval_unrewind): ';' -> '{}' to pacify GCC.
+
+2013-07-26  Stefan Monnier  <monnier@iro.umontreal.ca>
+
+       * eval.c (set_specpdl_old_value): New function.
+       (unbind_to): Minor simplification.
+       (get_backtrace_frame): New function.
+       (Fbacktrace_frame): Use it.  Add `base' argument.
+       (backtrace_eval_unrewind, Fbacktrace_eval): New functions.
+       (syms_of_eval): Export backtrace-eval.
+       * xterm.c (x_focus_changed): Simplify.
+
+2013-07-25  Paul Eggert  <eggert@cs.ucla.edu>
+
+       * fileio.c (Finsert_file_contents): Avoid double-close (Bug#14936).
+
+2013-07-24  Eli Zaretskii  <eliz@gnu.org>
+
+       * xdisp.c (redisplay_window): Instead of moving point out of
+       scroll margin, reject the force_start method, and try scrolling
+       instead.  (Bug#14780)
+
+2013-07-24  Ken Brown  <kbrown@cornell.edu>
+
+       * alloc.c (make_save_ptr): Define if HAVE_NTGUI is defined
+       (Bug#14944).
+
+2013-07-24  Paul Eggert  <eggert@cs.ucla.edu>
+
+       * eval.c (Fprogn): Do not check that BODY is a proper list.
+       This undoes the previous change.  The check slows down the
+       interpreter, and is not needed to prevent a crash.  See
+       <http://lists.gnu.org/archive/html/emacs-devel/2013-07/msg00693.html>.
+
+2013-07-23  Glenn Morris  <rgm@gnu.org>
+
+       * Makefile.in ($(etc)/DOC, temacs$(EXEEXT)): Ensure etc/ exists.
+
+2013-07-23  Paul Eggert  <eggert@cs.ucla.edu>
+
+       Port to GNU/Linux systems with tinfo but not ncurses.
+       * dispnew.c (init_display): Depend on USE_NCURSES, not GNU_LINUX,
+       to decide whether ncurses is being used.  Without this change,
+       GCC complains about tgetent not being declared, on a system
+       that has tinfo installed but ncurses not installed.
+
+       * eval.c (Fprogn): Check that BODY is a proper list.
+
+       Tune UNEVALLED functions by using XCAR instead of Fcar, etc.
+       * data.c (Fsetq_default):
+       * eval.c (Fif, Fcond, Fprog1, Fsetq, Fquote, Ffunction, Fdefvar)
+       (Fdefconst, FletX, Flet, Fwhile, Fcatch, Funwind_protect)
+       (Fcondition_case):
+       Tune by taking advantage of the fact that ARGS is always a list
+       when a function is declared to have UNEVALLED args.
+
+       * emacsgtkfixed.c: Port to GCC 4.6.
+       GCC 4.6 complains about -Wunused-local-typedefs, introduced in 4.7.
+
+2013-07-23  Juanma Barranquero  <lekktu@gmail.com>
+
+       * callproc.c (child_setup)[!WINDOWSNT]: Move exec_errno and pid
+       here to silence compiler warnings.
+
+2013-07-22  Paul Eggert  <eggert@cs.ucla.edu>
+
+       * sysdep.c (frame) [__FreeBSD__]: #define to freebsd_frame
+       when including <sys/user.h>, to prevent Sparc/ARM machine/frame.h
+       from messing up Emacs's 'struct frame' (Bug#14923).
+
+2013-07-21  Paul Eggert  <eggert@cs.ucla.edu>
+
+       * alloc.c (make_save_ptr_ptr): Define this function.
+       It was inadvertently omitted.  It's needed only if
+       HAVE_MENUS && ! (USE_X_TOOLKIT || USE_GTK).
+
+2013-07-21  Jan Djärv  <jan.h.d@swipnet.se>
+
+       * nsterm.m (sendEvent:): Skip mouse moved if no dialog and no Emacs
+       frame have focus (Bug#14895).
+
+2013-07-21  Paul Eggert  <eggert@cs.ucla.edu>
+
+       Avoid vfork-related deadlock more cleanly.
+       * callproc.c (child_setup): When the child's exec fails, output
+       the program name, as that's more useful.  Use O_NONBLOCK to avoid
+       deadlock.
+       * process.c (create_process_1): Remove; no longer needed.
+       (create_process): Remove timer hack; no longer needed, now that
+       the child avoids deadlock.
+
+2013-07-20  Glenn Morris  <rgm@gnu.org>
+
+       * image.c (Fimage_flush): Fix doc typo.
+
+2013-07-20  Paul Eggert  <eggert@cs.ucla.edu>
+
+       Fix array bounds violation when pty allocation fails.
+       * process.c (PTY_NAME_SIZE): New constant.
+       (pty_name): Remove static variable; it's now auto.
+       (allocate_pty): Define even if !HAVE_PTYS; that's simpler.
+       Take pty_name as an arg rather than using a static variable.
+       All callers changed.
+       (create_process): Recover pty_flag from process, not from volatile local.
+       (create_pty): Stay inside array even when pty allocation fails.
+       (Fmake_serial_process): Omit unnecessary initializaiton of pty_flag.
+
+       * lread.c (Fload): Avoid initialization only when lint checking.
+       Mention that it's needed only for older GCCs.
+
+2013-07-20  Kenichi Handa  <handa@gnu.org>
+
+       * coding.c (CODING_ISO_FLAG_LEVEL_4): New macro.
+       (decode_coding_iso_2022): Check the single-shift area.  (Bug#8522)
+
+2013-07-20  Andreas Schwab  <schwab@linux-m68k.org>
+
+       * lread.c (Fload): Avoid uninitialized warning.
+
+2013-07-19  Paul Eggert  <eggert@cs.ucla.edu>
+
+       Fix some minor file descriptor leaks and related glitches.
+       * filelock.c (create_lock_file) [!O_CLOEXEC]: Use fcntl with FD_CLOEXEC.
+       (create_lock_file): Use write, not emacs_write.
+       * image.c (slurp_file, png_load_body):
+       * process.c (Fnetwork_interface_list, Fnetwork_interface_info)
+       (server_accept_connection):
+       Don't leak an fd on memory allocation failure.
+       * image.c (slurp_file): Add a cheap heuristic for growing files.
+       * xfaces.c (Fx_load_color_file): Block input around the fopen too,
+       as that's what the other routines do.  Maybe input need not be
+       blocked at all, but it's better to be consistent.
+       Avoid undefined behavior when strlen is zero.
+
+       * alloc.c (staticpro): Avoid buffer overrun on repeated calls.
+       (NSTATICS): Now a constant; doesn't need to be a macro.
+
+2013-07-19  Richard Stallman  <rms@gnu.org>
+
+       * coding.c (decode_coding_utf_8): Add simple loop for fast
+       processing of ASCII characters.
+
+2013-07-19  Paul Eggert  <eggert@cs.ucla.edu>
+
+       * conf_post.h (RE_TRANSLATE_P) [emacs]: Remove obsolete optimization.
+
+2013-07-19  Eli Zaretskii  <eliz@gnu.org>
+
+       * keyboard.c (kbd_buffer_get_event): Use Display_Info instead of
+       unportable 'struct x_display_info'.
+       (DISPLAY_LIST_INFO): Delete macro: not needed, since Display_Info
+       is a portable type.
+
+2013-07-19  Paul Eggert  <eggert@cs.ucla.edu>
+
+       * sysdep.c [GNU_LINUX]: Fix fd and memory leaks and similar issues.
+       (procfs_ttyname): Don't use uninitialized storage if emacs_fopen
+       or fscanf fails.
+       (system_process_attributes): Prefer plain char to unsigned char
+       when either will do.  Clean up properly if interrupted or if
+       memory allocations fail.  Don't assume sscanf succeeds.  Remove
+       no-longer-needed workaround to stop GCC from whining.  Read
+       command-line once, instead of multiple times.  Check read status a
+       bit more carefully.
+
+       Fix obscure porting bug with varargs functions.
+       The code assumed that int is treated like ptrdiff_t in a vararg
+       function, which is not a portable assumption.  There was a similar
+       -- though these days less likely -- porting problem with various
+       assumptions that pointers of different types all smell the same as
+       far as vararg functions is conserved.  To make this problem less
+       likely in the future, redo the API to use varargs functions.
+       * alloc.c (make_save_value): Remove this vararg function.
+       All uses changed to ...
+       (make_save_int_int_int, make_save_obj_obj_obj_obj)
+       (make_save_ptr_int, make_save_funcptr_ptr_obj, make_save_memory):
+       New functions.
+       (make_save_ptr): Rename from make_save_pointer, for consistency with
+       the above.  Define only on platforms that need it.  All uses changed.
+
+2013-07-18  Paul Eggert  <eggert@cs.ucla.edu>
+
+       * keyboard.c: Try to fix typos in previous change.
+       (DISPLAY_LIST_INFO): New macro.
+       (kbd_buffer_get_event): Do not access members that are not present
+       in X11.  Revert inadvertent change of "!=" to "=".
+
+2013-07-18  Juanma Barranquero  <lekktu@gmail.com>
+
+       * keyboard.c (kbd_buffer_get_event):
+       * w32term.c (x_focus_changed): Port FOCUS_(IN|OUT)_EVENT changes to W32.
+       Followup to 2013-07-16T11:41:06Z!jan.h.d@swipnet.se.
+
+2013-07-18  Paul Eggert  <eggert@cs.ucla.edu>
+
+       * filelock.c: Fix unlikely file descriptor leaks.
+       (get_boot_time_1): Rework to avoid using emacs_open.
+       This doesn't actually fix a leak, but is better anyway.
+       (read_lock_data): Use read, not emacs_read.
+
+       * doc.c: Fix minor memory and file descriptor leaks.
+       * doc.c (get_doc_string): Fix memory leak when doc file absent.
+       (get_doc_string, Fsnarf_documentation):
+       Fix file descriptor leak on error.
+
+       * term.c: Fix minor fdopen-related file descriptor leaks.
+       * term.c (Fresume_tty) [!MSDOS]: Close fd if fdopen (fd) fails.
+       (init_tty) [!DOS_NT]: Likewise.  Also close fd if isatty (fd) fails.
+
+       * charset.c: Fix file descriptor leaks and errno issues.
+       Include <errno.h>.
+       (load_charset_map_from_file): Don't leak file descriptor on error.
+       Use plain record_xmalloc since the allocation is larger than
+       MAX_ALLOCA; that's simpler here.  Simplify test for exhaustion
+       of entries.
+       * eval.c (record_unwind_protect_nothing):
+       * fileio.c (fclose_unwind):
+       New functions.
+       * lread.c (load_unwind): Remove.  All uses replaced by fclose_unwind.
+       The replacement doesn't block input, but that no longer seems
+       necessary.
+
+2013-07-17  Paul Eggert  <eggert@cs.ucla.edu>
+
+       * lread.c: Fix file descriptor leaks and errno issues.
+       (Fload): Close some races that leaked fds or streams when 'load'
+       was interrupted.
+       (Fload, openp): Report error number of last nontrivial failure to open.
+       ENOENT counts as trivial.
+       * eval.c (do_nothing, clear_unwind_protect, set_unwind_protect_ptr):
+       New functions.
+       * fileio.c (close_file_unwind): No need to test whether FD is nonnegative,
+       now that the function is always called with a nonnegative arg.
+       * lisp.h (set_unwind_protect_ptr, set_unwind_protect_int): Remove.
+       All uses replaced with ...
+       (clear_unwind_protect, set_unwind_protect_ptr): New decls.
+
+       A few more minor file errno-reporting bugs.
+       * callproc.c (Fcall_process):
+       * doc.c (Fsnarf_documentation):
+       * fileio.c (Frename_file, Fadd_name_to_file, Fmake_symbolic_link):
+       * process.c (set_socket_option):
+       Don't let a constructor trash errno.
+       * doc.c: Include <errno.h>.
+
+2013-07-16  Juanma Barranquero  <lekktu@gmail.com>
+
+       * w32fns.c (unwind_create_tip_frame): Fix declaration.
+
 2013-07-16  Paul Eggert  <eggert@cs.ucla.edu>
 
-       Fix bug where Emacs tries to close a file twice (Bug#14839).
+       Fix w32 bug with call-process-region (Bug#14885).
+       * callproc.c (Fcall_process_region): Pass nil, not "/dev/null",
+       to Fcall_process when the input is empty.  This simplifies the
+       code a bit.  It makes no difference on POSIXish platforms but
+       apparently it fixes a bug on w32.
+
+       Fix bug where insert-file-contents closes a file twice. (Bug#14839).
        * fileio.c (close_file_unwind): Don't close if FD is negative;
        this can happen when unwinding a zapped file descriptor.
        (Finsert_file_contents): Unwind-protect the fd before the point marker,
        initializers.
 
        Syntax cleanup, mostly replacing macros with functions.
-`      This removes the need for the syntax_temp hack.
+       This removes the need for the syntax_temp hack.
        * search.c: Include syntax.h after buffer.h, since syntax.h uses BVAR.
        * syntax.c (SYNTAX_INLINE): New macro.
        (SYNTAX_FLAGS_COMSTART_FIRST, SYNTAX_FLAGS_COMSTART_SECOND)
        (emacswrite_sig, emacs_perror): New functions.
        * xrdb.c (fatal): Don't invoke perror, since errno might be garbage.
 
-2013-07-08  Magnus Henoch <magnus.henoch@gmail.com> (tiny change).
+2013-07-08  Magnus Henoch  <magnus.henoch@gmail.com>  (tiny change).
 
        * image.c (imagemagick_load_image): Do not use MagickExportImagePixels
        on NS even if it is present.  Pixmap on NS is a void*.
 
        * floatfns.c (Flog10): Move to Lisp (marked obsolete there).
 
-2013-06-20  Rüdiger Sonderfeld <ruediger@c-plusplus.de>
+2013-06-20  Rüdiger Sonderfeld  <ruediger@c-plusplus.de>
 
        * floatfns.c (Flog) [HAVE_LOG2]: Use log2 if available and if the
        base is 2; this is more accurate.