Fix bug #8597 with setting frame background mode on w32 console.
[bpt/emacs.git] / src / ChangeLog
index e2ac232..10ca2e2 100644 (file)
@@ -1,3 +1,386 @@
+2011-05-07  Eli Zaretskii  <eliz@gnu.org>
+
+       * w32console.c (Fset_screen_color): Doc fix.
+       (Fget_screen_color): New function.
+       (syms_of_ntterm): Defsubr it.
+
+       * callproc.c (call_process_cleanup): Don't close and unlink the
+       temporary file if Fcall_process didn't create it in the first
+       place.
+       (Fcall_process): Don't create tempfile if stdout of the child
+       process will be redirected to a file specified with `:file'.
+       Don't try to re-open tempfile in that case, and set fd[0] to -1 as
+       cue to call_process_cleanup not to close that handle.
+
+2011-05-07  Ben Key  <bkey76@gmail.com>
+
+       * makefile.w32-in: The bootstrap-temacs rule now makes use of
+       one of two shell specific rules, either bootstrap-temacs-CMD or
+       bootstrap-temacs-SH.  The bootstrap-temacs-SH rule is identical
+       to the previous implementation of the bootstrap-temacs rule.
+       The bootstrap-temacs-CMD rule is similar to the previous
+       implementation of the bootstrap-temacs rule except that it
+       makes use of the ESC_CFLAGS variable instead of the CFLAGS
+       variable.
+
+       These changes, along with some changes to nt/configure.bat,
+       nt/gmake.defs, and nt/nmake.defs, are required to extend my
+       earlier fix to add support for --cflags and --ldflags options
+       that include quotes so that it works whether make uses cmd or
+       sh as the shell.
+
+2011-05-06  Michael Albinus  <michael.albinus@gmx.de>
+
+       * dbusbind.c (QCdbus_type_unix_fd): Declare static.
+       (xd_remove_watch): Don't check QCdbus_type_unix_fd for SYMBOLP, it
+       is a constant.
+       (Fdbus_init_bus, xd_read_queued_messages): Bus can be a symbol or
+       a string.  Handle both cases.
+       (Fdbus_call_method_asynchronously, Fdbus_register_signal)
+       (Fdbus_register_method): Use Qinvalid_function.
+
+2011-05-06  Juanma Barranquero  <lekktu@gmail.com>
+
+       * makefile.w32-in: Update dependencies.
+       (LISP_H): Add inttypes.h and stdin.h.
+       (PROCESS_H): Add unistd.h.
+
+2011-05-06  Eli Zaretskii  <eliz@gnu.org>
+
+       * lread.c: Include limits.h (fixes the MS-Windows build broken by
+       2011-05-06T07:13:19Z!eggert@cs.ucla.edu).
+
+2011-05-06  Paul Eggert  <eggert@cs.ucla.edu>
+
+       * image.c (Finit_image_library) [!HAVE_NTGUI]: Omit unused local.
+
+       * term.c (vfatal): Remove stray call to va_end.
+       It's not needed and the C Standard doesn't allow it here anyway.
+
+       Use C99's va_copy to avoid undefined behavior on x86-64 GNU/Linux.
+       * eval.c (verror): doprnt a copy of ap, not the original.  (Bug#8545)
+
+       * eval.c (verror): OK to create a string of up to MOST_POSITIVE_FIXNUM
+       bytes.
+
+       * term.c: Don't include <stdarg.h>, as <lisp.h> does that.
+
+       * callproc.c (Fcall_process): Use 'volatile' to avoid vfork clobbering.
+
+       * process.c (Fformat_network_address): Fix typo: args2 -> *args2.
+
+       * xmenu.c (set_frame_menubar): Fix typo: int * -> int (3 times).
+
+       * coding.c (detect_coding_charset): Fix typo: * 2 -> *4 (Bug#8601).
+
+       * charset.h (struct charset.code_space): Now has 15 elements, not 16.
+       * charset.c (Fdefine_charset_internal): Don't initialize
+       charset.code_space[15].  The value was garbage, on hosts with
+       32-bit int (Bug#8600).
+
+       * lread.c (read_integer): Be more consistent with string-to-number.
+       Use string_to_number to do the actual conversion; this avoids
+       rounding errors and fixes some other screwups.  Without this fix,
+       for example, #x1fffffffffffffff was misread as -2305843009213693952.
+       (digit_to_number): Move earlier, for benefit of read_integer.
+       Return -1 if the digit is out of range for the base, -2 if it is
+       not a digit in any supported base.  (Bug#8602)
+
+       * doprnt.c (doprnt): Support arbitrary pI values, such as "I64".
+
+       * dispnew.c (scrolling_window): Return 1 if we scrolled,
+       to match comment at start of function.  This also removes a
+       GCC warning about overflow in a 32+64-bit port.
+
+       * lisp.h (EMACS_INT, EMACS_UINT, BITS_PER_EMACS_INT, pI): Simplify.
+
+       * dbusbind.c: Do not use XPNTR on a value that may be an integer.
+       Reported by Stefan Monnier in
+       <http://lists.gnu.org/archive/html/emacs-devel/2011-04/msg00919.html>.
+       (xd_remove_watch, Fdbus_init_bus, xd_read_queued_messages): Use
+       SYMBOLP-guarded XSYMBOL, not XPNTR.
+
+       * lisp.h (EMACS_INTPTR): Remove.  All uses changed to intptr_t.
+       (EMACS_UINTPTR): Likewise, with uintptr_t.
+
+       * lisp.h: Prefer 64-bit EMACS_INT if available.
+       (EMACS_INT, EMACS_UINT, BITS_PER_EMACS_INT, pI): Define to 64-bit
+       on 32-bit hosts that have 64-bit int, so that they can access
+       large files.
+       However, temporarily disable this change unless the temporary
+       symbol WIDE_EMACS_INT is defined.
+
+       * lread.c, process.c: Do not include <inttypes.h>; lisp.h does it now.
+
+       Prefer intptr_t/uintptr_t for integers the same widths as pointers.
+       This removes an assumption that EMACS_INT and long are the same
+       width as pointers.  The assumption is true for Emacs porting targets
+       now, but we want to make other targets possible.
+       * lisp.h: Include <inttypes.h>, for INTPTR_MAX, UINTPTR_MAX.
+       (EMACS_INTPTR, EMACS_UINTPTR): New macros.
+       In the rest of the code, change types of integers that hold casted
+       pointers to EMACS_INTPTR and EMACS_UINTPTR, systematically
+       replacing EMACS_INT, long, EMACS_UINT, and unsigned long.
+       (XTYPE): Don't cast arg to EMACS_UINT; normally is not needed.
+       (XSET): Cast type of XTYPE arg to EMACS_INTPTR; it is needed here.
+       No need to cast type when ORing.
+       (XPNTR): Return a value of type EMACS_INTPTR or EMACS_UINTPTR.
+       * alloc.c (lisp_align_malloc): Remove a no-longer-needed cast.
+       * doc.c (store_function_docstring): Use EMACS_INTPTR, so as not to
+       assume EMACS_INT is the same width as char *.
+       * gtkutil.c (xg_gtk_scroll_destroy, xg_tool_bar_button_cb):
+       (xg_tool_bar_callback, xg_tool_bar_help_callback, xg_make_tool_item):
+       Remove no-longer-needed casts.
+       (xg_create_scroll_bar, xg_tool_bar_button_cb, xg_tool_bar_callback):
+       (xg_tool_bar_help_callback, xg_make_tool_item):
+       Use EMACS_INTPTR to hold an integer
+       that will be cast to void *; this can avoid a GCC warning
+       if EMACS_INT is not the same width as void *.
+       * menu.c (find_and_call_menu_selection): Remove no-longer-needed cast.
+       * xdisp.c (display_echo_area_1, resize_mini_window_1):
+       (current_message_1, set_message_1):
+       Use a local to convert to proper width without a cast.
+       * xmenu.c (dialog_selection_callback): Likewise.
+
+       * sysdep.c (get_random): Don't assume EMACS_INT is no wider than long.
+       Also, don't assume VALBITS / RAND_BITS is less than 5,
+       and don't rely on undefined behavior when shifting a 1 left into
+       the sign bit.
+       * lisp.h (get_random): Change signature to match.
+
+       * lread.c (hash_string): Use size_t, not int, for hash computation.
+       Normally we prefer signed values; but hashing is special, because
+       it's better to use unsigned division on hash table sizes so that
+       the remainder is nonnegative.  Also, size_t is the natural width
+       for hashing into memory.  The previous code used 'int', which doesn't
+       retain enough info to hash well into very large tables.
+       (oblookup, oblookup_last_bucket_number, Funintern): Likewise.
+
+       * dbusbind.c: Don't possibly lose pointer info when converting.
+       (xd_remove_watch, Fdbus_init_bus, xd_read_queued_messages):
+       Use XPNTR rather than XHASH, so that the high-order bits of
+       the pointer aren't lost when converting through void *.
+
+       * eval.c (Fautoload): Don't double-shift a pointer.
+
+       * fns.c (Frandom): Let EMACS_UINT be wider than unsigned long.
+
+2011-05-06  Juanma Barranquero  <lekktu@gmail.com>
+
+       * gnutls.c (DEF_GNUTLS_FN):
+       * image.c (DEF_IMGLIB_FN): Make function pointers static.
+
+2011-05-05  Andreas Schwab  <schwab@linux-m68k.org>
+
+       * lread.c (lisp_file_lexically_bound_p): Stop scanning at end
+       marker.  (Bug#8610)
+
+2011-05-05 Eli Zaretskii  <eliz@gnu.org>
+
+       * w32heap.c (allocate_heap) [USE_LISP_UNION_TYPE || USE_LSB_TAG]:
+       New version that can reserve upto 2GB of heap space.
+
+2011-05-05  Chong Yidong  <cyd@stupidchicken.com>
+
+       * nsfns.m (Fns_read_file_name): Doc fix (Bug#8534).
+
+2011-05-05  Teodor Zlatanov  <tzz@lifelogs.com>
+
+       * gnutls.c (fn_gnutls_certificate_set_x509_key_file): Add alias to
+       `gnutls_certificate_set_x509_key_file'.
+
+2011-05-05  Juanma Barranquero  <lekktu@gmail.com>
+
+       * makefile.w32-in ($(BLD)/image.$(O), $(BLD)/process.$(O)):
+       Update dependencies.
+
+2011-05-04  Juanma Barranquero  <lekktu@gmail.com>
+
+       * gnutls.h (emacs_gnutls_write, emacs_gnutls_read):
+       * gnutls.c (emacs_gnutls_write, emacs_gnutls_read):
+       Remove unused parameter `fildes'.
+       * process.c (read_process_output, send_process): Don't pass it.
+
+2011-05-04  Juanma Barranquero  <lekktu@gmail.com>
+
+       Fix previous change: the library cache is defined in w32.c.
+       * image.c (CACHE_IMAGE_TYPE) [!HAVE_NTGUI]: Define to noop.
+       (Finit_image_library): Wrap Vlibrary_cache on "#ifdef HAVE_NTGUI".
+
+2011-05-04  Juanma Barranquero  <lekktu@gmail.com>
+
+       Implement dynamic loading of GnuTLS on Windows.
+
+       * gnutls.h (GNUTLS_EMACS_ERROR_NOT_LOADED): New macro.
+       (emacs_gnutls_write, emacs_gnutls_read): Mark as extern.
+       (emacs_gnutls_record_check_pending, emacs_gnutls_transport_set_errno):
+       Declare.
+
+       * gnutls.c (Qgnutls_dll): Define.
+       (DEF_GNUTLS_FN, LOAD_GNUTLS_FN): New macros.
+       (gnutls_*): Declare function pointers.
+       (init_gnutls_functions): New function to initialize function pointers.
+       (emacs_gnutls_handshake, Fgnutls_error_string, Fgnutls_deinit)
+       (emacs_gnutls_global_init, Fgnutls_bye): Use function pointers.
+       (emacs_gnutls_record_check_pending, emacs_gnutls_transport_set_errno):
+       Wrappers for gnutls_record_check_pending and gnutls_transport_set_errno.
+       (emacs_gnutls_write, emacs_gnutls_read)
+       (emacs_gnutls_handle_error, Fgnutls_error_fatalp)
+       (Fgnutls_available_p): New function.
+       (Fgnutls_boot): Call Fgnutls_available_p.  Use function pointers.
+       (syms_of_gnutls) <Qgnutls_dll>: Initialize and staticpro it.
+       (syms_of_gnutls) <Sgnutls_available_p>: defsubr it.
+
+       * image.c: Include w32.h.
+       (Vimage_type_cache): Delete.
+       (syms_of_image) <Vimage_type_cache>: Don't initialize and staticpro it.
+       (CACHE_IMAGE_TYPE, Finit_image_library): Use Vlibrary_cache instead.
+       (w32_delayed_load): Move to w32.c.
+
+       * w32.h (VlibraryCache, QCloaded_from, w32_delayed_load): Declare.
+
+       * w32.c (QCloaded_from, Vlibrary_cache): Define.
+       (w32_delayed_load): Move from image.c.  When loading a library, record
+       its filename in the :loaded-from property of the library id.
+       (globals_of_w32) <QCloaded_from, Vlibrary_cache>:
+       Initialize and staticpro them.
+       (emacs_gnutls_pull, emacs_gnutls_push): Call emacs_gnutls_* functions.
+
+       * process.c: Include lisp.h before w32.h, not after.
+       (wait_reading_process_output): Call emacs_gnutls_record_check_pending
+       instead of gnutls_record_check_pending.
+
+       * callproc.c, emacs.c: Include lisp.h before w32.h, not after.
+
+2011-05-04  Teodor Zlatanov  <tzz@lifelogs.com>
+
+       * gnutls.c (Fgnutls_boot): Support :keylist and :crlfiles options
+       instead of :keyfiles.  Give GnuTLS the keylist and the CRL lists
+       as passed in.
+
+2011-05-03  Jan Djärv  <jan.h.d@swipnet.se>
+
+       * xterm.c (x_set_frame_alpha): Do not set property on anything
+       else than FRAME_X_OUTER_WINDOW (Bug#8608).
+
+2011-05-02  Juanma Barranquero  <lekktu@gmail.com>
+
+       * sysdep.c (get_tty_size) [WINDOWSNT]: Implement.  (Bug#8596)
+
+2011-05-02  Juanma Barranquero  <lekktu@gmail.com>
+
+       * gnutls.c (Qgnutls_log_level, Qgnutls_code, Qgnutls_anon)
+       (Qgnutls_x509pki, Qgnutls_e_interrupted, Qgnutls_e_again)
+       (Qgnutls_e_invalid_session, Qgnutls_e_not_ready_for_handshake)
+       (gnutls_global_initialized, Qgnutls_bootprop_priority)
+       (Qgnutls_bootprop_trustfiles, Qgnutls_bootprop_keyfiles)
+       (Qgnutls_bootprop_callbacks, Qgnutls_bootprop_loglevel)
+       (Qgnutls_bootprop_hostname, Qgnutls_bootprop_verify_flags)
+       (Qgnutls_bootprop_verify_error, Qgnutls_bootprop_verify_hostname_error)
+       (Qgnutls_bootprop_callbacks_verify): Make static.
+
+2011-05-01  Andreas Schwab  <schwab@linux-m68k.org>
+
+       * callproc.c: Indentation fixup.
+
+       * sysdep.c (wait_for_termination_1): Make static.
+       (wait_for_termination, interruptible_wait_for_termination): Move
+       after wait_for_termination_1.
+
+2011-05-01  Lars Magne Ingebrigtsen  <larsi@gnus.org>
+
+       * sysdep.c (interruptible_wait_for_termination): New function
+       which is like wait_for_termination, but allows keyboard
+       interruptions.
+
+       * callproc.c (Fcall_process): Add (:file "file") as an option for
+       the STDOUT buffer.
+       (Fcall_process_region): Ditto.
+
+2011-04-30  Eli Zaretskii  <eliz@gnu.org>
+
+       * dosfns.c (Fint86, Fdos_memget, Fdos_memput): Use `ASIZE (FOO)'
+       rather than `XVECTOR (FOO)->size'.
+
+       * process.c: Remove HAVE_INTTYPES_H condition from inclusion of
+       inttypes.h, as a gnulib replacement is used if it not available in
+       system headers.
+
+2011-04-21  Eli Zaretskii  <eliz@gnu.org>
+
+       Lift the MOST_POSITIVE_FIXNUM/4 limitation on visited files.
+       * fileio.c (Finsert_file_contents): Don't limit file size to 1/4
+       of MOST_POSITIVE_FIXNUM.  (Bug#8528)
+
+       * coding.c (coding_alloc_by_realloc): Error out if destination
+       will grow beyond MOST_POSITIVE_FIXNUM.
+       (decode_coding_emacs_mule): Abort if there isn't enough place in
+       charbuf for the composition carryover bytes.  Reserve an extra
+       space for up to 2 characters produced in a loop.
+       (decode_coding_iso_2022): Abort if there isn't enough place in
+       charbuf for the composition carryover bytes.
+
+2011-04-21  Eli Zaretskii  <eliz@gnu.org>
+
+       * doprnt.c (doprnt) [!HAVE_LONG_LONG_INT]: Error out instead of
+       aborting when %lld or %lll format is passed.
+       [!HAVE_UNSIGNED_LONG_LONG_INT]: Error out instead of aborting when
+       %llo or %llx format is passed.  (Bug#8545)
+
+       * window.c (window_scroll_line_based): Use a marker instead of
+       simple variables to record original value of point.  (Bug#7952)
+
+       * doprnt.c (doprnt): Fix the case where a multibyte sequence
+       produced by %s or %c overflows available buffer space.  (Bug#8545)
+
+2011-04-28  Paul Eggert  <eggert@cs.ucla.edu>
+
+       * doprnt.c (doprnt): Omit useless test; int overflow check (Bug#8545).
+       (SIZE_MAX): Move defn after all includes, as they might #define it.
+
+2011-04-28  Juanma Barranquero  <lekktu@gmail.com>
+
+       * w32.c (init_environment): Warn about defaulting HOME to C:\.
+
+2011-04-28  Juanma Barranquero  <lekktu@gmail.com>
+
+       * keyboard.c (Qdelayed_warnings_hook): Define.
+       (command_loop_1): Run `delayed-warnings-hook'
+       if Vdelayed_warnings_list is non-nil.
+       (syms_of_keyboard) <delayed-warnings-hook>: DEFSYM it.
+       (syms_of_keyboard) <delayed-warnings-list>: DEFVAR_LISP it.
+
+2011-04-28  Eli Zaretskii  <eliz@gnu.org>
+
+       * doprnt.c (doprnt): Don't return value smaller than the buffer
+       size if the message was truncated.  (Bug#8545).
+
+2011-04-28  Juanma Barranquero  <lekktu@gmail.com>
+
+       * w32fns.c (Fx_change_window_property, Fx_delete_window_property)
+       (Fx_window_property): #if-0 the whole functions, not just the bodies.
+
+2011-04-27  Paul Eggert  <eggert@cs.ucla.edu>
+
+       * doprnt.c (doprnt): Support "ll" length modifier, for long long.
+
+2011-04-27  Juanma Barranquero  <lekktu@gmail.com>
+
+       * makefile.w32-in: Update dependencies.
+
+2011-04-27  Eli Zaretskii  <eliz@gnu.org>
+
+       Improve `doprnt' and its usage.  (Bug#8545)
+       * doprnt.c (doprnt): Make sure `format' is never accessed beyond
+       `format_end'.  Remove support for %l as a conversion specifier.
+       Don't use xrealloc.  Improve diagnostics when the %l size modifier
+       is used.  Update the commentary.
+
+       * eval.c (verror): Simplify calculation of size_t.
+
+       * coding.c (Ffind_operation_coding_system): Fix diagnostic error
+       messages.
+
 2011-04-27  Yoshiaki Kasahara  <kasahara@nc.kyushu-u.ac.jp>  (tiny change)
 
        * buffer.c (init_buffer) [USE_MMAP_FOR_BUFFERS]: Adjust to aliasing
        Remove unused local.
        (emacs_gnutls_write): Don't use uninitialized rtnval if nbyte <= 0.
 
-       lisp.h: Fix a problem with aliasing and vector headers.  (Bug#8546)
+       lisp.h: Fix a problem with aliasing and vector headers.  (Bug#8546)
        GCC 4.6.0 optimizes based on type-based alias analysis.  For
        example, if b is of type struct buffer * and v of type struct
        Lisp_Vector *, then gcc -O2 was incorrectly assuming that &b->size
        supposed to be handshaking.  (Bug#8556)
        Reported by Paul Eggert <eggert@cs.ucla.edu>.
 
-2011-04-26  Daniel Colascione <dan.colascione@gmail.com>
+2011-04-26  Daniel Colascione  <dan.colascione@gmail.com>
 
        * lisp.h (Qdebug): List symbol.
        * eval.c (Qdebug): Restore global linkage.
        define if USE_GTK || USE_X_TOOLKIT.
        (SET_SAVED_BUTTON_EVENT): Define only if USE_X_TOOLKIT || USE_GTK.
        * xterm.h (x_dispatch_event): Extern only if USE_X_TOOLKIT.
-       * xterm.c, xterm.h (x_mouse_leave): Bring this function back, but only if
-       defined HAVE_MENUS && !defined USE_X_TOOLKIT && !defined USE_GTK.
+       * xterm.c, xterm.h (x_mouse_leave): Bring this function back, but only
+       if defined HAVE_MENUS && !defined USE_X_TOOLKIT && !defined USE_GTK.
 
        * xmenu.c (menu_help_callback): Pointer type fixes.
        Use const pointers when pointing at readonly data.  Avoid pointer