ChangeLog 270 KB
Newer Older
1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24
2013-10-29  Stefan Monnier  <monnier@iro.umontreal.ca>

	* xdisp.c (prepare_menu_bars): Call Vpre_redisplay_function.
	(syms_of_xdisp): Declare pre-redisplay-function.
	(markpos_of_region): Remove function.
	(init_iterator, compute_stop_pos, handle_face_prop)
	(face_before_or_after_it_pos, reseat_to_string)
	(get_next_display_element, window_buffer_changed)
	(redisplay_internal, try_cursor_movement, redisplay_window)
	(try_window_reusing_current_matrix, try_window_id, display_line)
	(note_mode_line_or_margin_highlight, note_mouse_highlight)
	(display_string, mouse_face_from_buffer_pos): Remove region handling.
	* window.h (struct window): Remove field `region_showing'.
	* dispextern.h (struct it): Remove region_beg/end_charpos.
	(face_at_buffer_position, face_for_overlay_string)
	(face_at_string_position): Update prototypes.
	* xfaces.c (face_at_buffer_position, face_for_overlay_string)
	(face_at_string_position): Remove `region_beg' and `region_end' args.
	* fontset.c (Finternal_char_font):
	* font.c (font_at, font_range): Adjust calls accordingly.
	* insdel.c (Qregion_extract_function): New var.
	(syms_of_insdel): Initialize it.
	(prepare_to_modify_buffer_1): Use it.

25 26 27 28 29 30 31 32 33 34 35 36 37 38 39
2013-10-29  Dmitry Antipov  <dmantipov@yandex.ru>

	Prefer 'unsigned long' to 'long unsigned int' and 'unsigned long int'.
	* ftxfont.c (ftxfont_get_gcs):
	* gtkutil.c (xg_set_widget_bg, xg_set_background_color):
	* xfaces.c (x_free_colors, x_free_dpy_colors)
	(x_create_gc, unload_color):
	* xselect.c (x_property_data_to_lisp):
	* xsettings.c (parse_settings):
	* xterm.c (x_copy_color, x_alloc_lighter_color, x_setup_relief_color)
	(get_bits_and_offset): Adjust definition.
	* frame.c (XParseGeometry): Adjust locals.
	* lisp.h (toplevel): Adjust EMACS_UINT type definition.
	* regex.h (toplevel): Likewise for reg_syntax_t.

40 41 42 43
2013-10-29  Stefan Monnier  <monnier@iro.umontreal.ca>

	* eval.c (run_hook_with_args): Use FUNCTIONP.

44 45 46 47 48 49 50 51 52 53 54 55 56 57 58
2013-10-29  Dmitry Antipov  <dmantipov@yandex.ru>

	* xterm.h (struct x_output): For 'black_relief' and 'white_relief'
	fields, drop 'allocated_p' member and use -1 for uninitialized value.
	* w32term.h (struct w32_output): Similarly but do not use -1 because...
	* xfaces.c (unload_color) [HAVE_X_WINDOWS]: ...this function is a no-op
	on MS-Windows anyway.
	(free_face_colors): Define only if HAVE_X_WINDOWS and...
	(free_realized_face): ...adjust user.
	* xfns.c (Fx_create_frame, x_create_tip_frame): Initialize black and
	white relief pixels to -1.
	* xterm.c (x_setup_relief_color, x_free_frame_resources): Adjust users.
	* w32term.c (w32_setup_relief_color, x_free_frame_resources): Likewise.
	* dispextern.h (unload_color): Move prototype under HAVE_X_WINDOWS.

59 60 61 62 63 64
2013-10-28  Paul Eggert  <eggert@cs.ucla.edu>

	* dispextern.h, image.c (x_bitmap_height, x_bitmap_width): Now static.
	* xfaces.c (load_pixmap): Omit last two args, which are always NULL
	in practice now.  All callers changed.

65 66 67 68 69 70 71 72 73 74 75
2013-10-28  Dmitry Antipov  <dmantipov@yandex.ru>

	* dispextern.h (struct face): Use bitfields for 'underline_type'
	and 'box' members.  Remove set-but-unused members 'pixmap_w' and
	'pixmap_h'.  If not HAVE_WINDOW_SYSTEM, also remove dummy
	'stipple' member.  Move 'lface' member up to help...
	* xfaces.c (make_realized_face): ...this function to find and
	clear just the members that need clearing.
	(load_face_colors, realize_x_face):
	* xdisp.c (extend_face_to_end_of_line): Adjust user.

76 77 78 79 80 81 82
2013-10-27  Dmitry Antipov  <dmantipov@yandex.ru>

	* xftfont.c (struct xftfont_info): Remove set-but-unused
	'screen' member.
	(xftfont_open): Adjust user.
	(xftfont_get_colors): Remove useless prototype.

83 84 85 86 87
2013-10-26  Eli Zaretskii  <eliz@gnu.org>

	* emacs.c (Fdump_emacs): Encode FILENAME and SYMFILE arguments
	before passing them to 'unexec'.  (Bug#15260)

88 89 90 91
2013-10-26  Xue Fuqiao  <xfq.free@gmail.com>

	* fringe.c (set_fringe_bitmap_face): Add usage note from lispref.

92 93
2013-10-25  Eli Zaretskii  <eliz@gnu.org>

94 95 96
	* w32uniscribe.c (uniscribe_close): Adjust the argument list to
	the changed signature of the font driver's 'close' method.

97 98 99 100 101 102 103 104
	* w32font.h (w32font_close): Adjust the prototype to the change in
	function definition.

	* w32font.c (w32font_close): Reintroduce deleted declaration of i.

	* w32uniscribe.c (uniscribe_close): Adapt the call to
	w32font_close to its new prototype.

105 106 107 108 109 110 111 112 113 114 115
2013-10-25  Dmitry Antipov  <dmantipov@yandex.ru>

	Omit unused frame argument of font API's close function.
	* font.h (struct font): Drop frame argument.  Adjust comment.
	* font.c (font_clear_cache, font_close_object): Adjust users.
	* ftfont.c (ftfont_close):
	* ftxfont.c (ftxfont_close):
	* macfont.m (macfont_close):
	* nsfont.m (nsfont_close):
	* w32font.c (w32font_close):
	* xfont.c (xfont_close):
116 117 118 119 120 121 122 123 124
	* xftfont.c (xftfont_close): Adjust driver-specific close functions,
	tweak comments and make functions safe if called more than once for
	the same font object.

	Perform font-specific cleanup when font object is swept by GC.  See
	http://lists.gnu.org/archive/html/emacs-devel/2013-10/msg00740.html.
	* alloc.c (cleanup_vector): New function.
	(sweep_vector): Call it for each reclaimed vector object.
	* font.h (struct font): Adjust comment.
125

126 127 128 129
2013-10-24  Glenn Morris  <rgm@gnu.org>

	* Makefile.in (abs_top_srcdir): New, set by configure.

130 131
2013-10-23  Dmitry Antipov  <dmantipov@yandex.ru>

Dmitry Antipov's avatar
Dmitry Antipov committed
132
	Adjust recent font-related changes to fix bug#15686.
133
	* alloc.c (mark_object) [HAVE_WINDOW_SYSTEM]: If marked frame
Dmitry Antipov's avatar
Dmitry Antipov committed
134
	is a live window system frame, mark its default font too.
135

136 137 138 139 140 141 142
2013-10-23  Glenn Morris  <rgm@gnu.org>

	* Makefile.in (RUN_TEMACS): Make relative (again).
	($(leimdir)/leim-list.el, .el.elc, $(lispsource)/loaddefs.el)
	(bootstrap-emacs$(EXEEXT)):
	Quote entities that might contain whitespace.

143 144 145 146 147 148
2013-10-23  Paul Eggert  <eggert@cs.ucla.edu>

	Port to Solaris 10 and its bundled GCC.
	Problem reported by Timothy C. Burt.
	* floatfns.c (isfinite, isnan): Redefine unconditionally.

149 150 151 152 153 154 155 156 157 158 159 160 161
2013-10-21  Dmitry Antipov  <dmantipov@yandex.ru>

	Do not allow font caches to grow too large.
	* alloc.c (compact_font_cache_entry, compact_font_caches):
	New functions or stub if not HAVE_WINDOW_SYSTEM.
	(compact_undo_list): Factor out from Fgarbage_collect.
	Add comment.
	(mark_face_cache): Mark face font.  Move down to avoid
	extra prototypes.
	(mark_terminals): Do not mark font cache here.
	(Fgarbage_collect): Call compaction functions described
	above.  Adjust comment.

Jan Djärv's avatar
Jan Djärv committed
162 163
2013-10-20  Jan Djärv  <jan.h.d@swipnet.se>

164 165 166
	* emacs.c (main): On Cocoa, if GUI session and 0 is not a tty,
	chdir to HOME (bug#15607).

Jan Djärv's avatar
Jan Djärv committed
167 168
	* nsterm.m (Qcocoa, Qgnustep): New variables.
	(syms_of_nsterm): Defsym Qcocoa, Qgnustep.  Fprovide appropriate one.
169
	(ns_get_color): Make selection color work for GNUStep also.
Jan Djärv's avatar
Jan Djärv committed
170

171 172 173 174 175 176 177 178
2013-10-18  Eli Zaretskii  <eliz@gnu.org>

	* keyboard.c (make_lispy_event): Remove GPM-specific code that
	handles mouse clicks.  Instead, let GPM use the same code as all
	the other mice use.  See the discussion starting at
	http://lists.gnu.org/archive/html/emacs-devel/2013-10/msg00521.html
	for the details of the problem with the menu bar this fixes.

179 180 181 182 183 184 185 186 187 188 189 190 191 192 193 194 195 196 197 198 199 200 201 202
2013-10-18  Dmitry Antipov  <dmantipov@yandex.ru>

	Remove port-specific display name lists to avoid extra
	complexity and data duplication with display info lists.
	* xterm.h (x_display_name_list): Remove declaration.
	* xterm.c (x_display_name_list): Remove.
	(x_term_init, x_delete_display, syms_of_xterm): Adjust users.
	* xfns.c (x_display_info_for_name, Fx_display_list):
	Likewise.  Use x_display_list where appropriate.
	* w32term.h (w32_display_name_list): Remove declaration.
	* w32term.c (w32_display_name_list): Remove.
	(w32_initialize_display_info, x_delete_display, syms_of_w32term):
	Adjust users.
	* w32fns.c (x_display_info_for_name, Fx_display_list):
	Likewise.  Use x_display_list where appropriate.
	* nsterm.h (ns_display_name_list): Remove declaration.
	* nsterm.m (ns_display_name_list): Remove.
	(ns_term_init, syms_of_nsterm): Adjust users.
	* nsfns.m (ns_display_info_for_name, Fx_display_list):
	Likewise.  Use x_display_list where appropriate.
	* termhooks.h (TERMINAL_FONT_CACHE): New macro.
	* alloc.c (toplevel) [HAVE_WINDOW_SYSTEM]: Include TERM_HEADER.
	(mark_terminals): Mark per-terminal font cache.

203 204 205 206 207 208 209
2013-10-17  Barry O'Reilly  <gundaetiapo@gmail.com>

	Don't run timers in input-pending-p.  Its new check-timers param
	provides the prior behavior. (Bug#15045).
	* src/keyboard.c (Finput_pending_p): Accept optional check-timers
	param.

Paul Eggert's avatar
Paul Eggert committed
210 211
2013-10-17  Paul Eggert  <eggert@cs.ucla.edu>

212 213 214 215 216 217 218 219 220 221 222 223 224 225 226 227 228 229 230 231 232 233 234 235 236 237 238 239 240 241
	Make some functions static in non-Microsoft builds.
	On my platform (Fedora 19 x86-64), this shrinks the
	Emacs executable (text+data) by 0.25%.
	* dispextern.h (erase_phys_cursor) [!WINDOWSNT]:
	(load_color) [!MSDOS]:
	* gnutls.h (emacs_gnutls_transport_set_errno) [!WINDOWSNT]:
	* keyboard.h (make_ctrl_char) [!WINDOWSNT]:
	* lisp.h (check_existing):
	* process.h (conv_sockaddr_to_lisp, network_interface_list)
	(network_interface_info) [!WINDOWSNT]:
	* termhooks.h (encode_terminal_code) [!WINDOWSNT]:
	Remove extern decls.
	* fileio.c (check_existing):
	* keyboard.c (make_ctrl_char) [!WINDOWSNT]:
	* process.c (conv_sockaddr_to_lisp, network_interface_list)
	(network_interface_info) [!WINDOWSNT]:
	* term.c (encode_terminal_code) [!WINDOWSNT]:
	* xdisp.c (erase_phys_cursor) [!WINDOWSNT]:
	* xfaces.c (load_color) [!MSDOS]:
	Now static.
	* fileio.c (check_existing, check_executable, check_writable):
	* process.c (network_interface_list, network_interface_info):
	Move earlier, so that we don't need forward decls.
	* gnutls.c (fn_gnutls_transport_set_errno)
	(emacs_gnutls_transport_set_errno) [!WINDOWNT]:
	Remove; unused.
	* w32.c (init_environment): Use faccessat rather than
	check_existing, partly for consistency with the rest of the code
	in this file, partly so that check_existing can be static.

242 243 244 245 246
	Make VALMASK visible to GDB even if clang is used (Bug#15574).
	* emacs.c (MAIN_PROGRAM): New macro.
	* lisp.h (DEFINE_GDB_SYMBOL_BEGIN, DEFINE_GDB_SYMBOL_END): New macros.
	(ARRAY_MARK_FLAG, PSEUDOVECTOR_FLAG, VALMASK): Use them.

Paul Eggert's avatar
Paul Eggert committed
247 248 249 250 251 252 253 254
	bool vector int width fixes
	* data.c (bool_vector_spare_mask, Fbool_vector_count_matches)
	(Fbool_vector_count_matches_at):
	Use EMACS_INT, not ptrdiff_t, to record bit counts, as a bit count
	can exceed PTRDIFF_MAX, at least in theory.
	(Fbool_vector_count_matches_at):
	Use int, not ptrdiff_t, to record a value that can't exceed INT_MAX.

255 256 257 258 259
2013-10-16  Paul Eggert  <eggert@cs.ucla.edu>

	* process.h (conv_sockaddr_to_lisp): New decl, for newly-extern func.
	(struct sockaddr): Add forward decl, for platforms that lack it.

260 261 262 263 264
2013-10-16  Jan Djärv  <jan.h.d@swipnet.se>

	* nsselect.m (ns_string_from_pasteboard): Remove Fquit, just return
	Qnil (Bug#15628).

265 266
2013-10-16  Eli Zaretskii  <eliz@gnu.org>

267 268 269 270 271 272
	* w32.c (network_interface_get_info, network_interface_list)
	(network_interface_info): New functions.  (Bug#15610)
	(GetAdaptersInfo_Proc): New typedef.
	(get_adapters_info): New wrapper function.
	(globals_of_w32): Initialize g_b_init_get_adapters_info.

273 274
	* process.h (network_interface_list, network_interface_info):
	New prototypes.
275 276 277 278 279 280 281 282

	* process.c (conv_sockaddr_to_lisp): Now externally-visible.
	(Fnetwork_interface_list, Fnetwork_interface_info): Define for
	all systems.  Return non-nil for systems that HAVE_NET_IF_H and
	for WINDOWSNT.  Doc fix.
	(syms_of_process): Defsubr Snetwork_interface_list and
	Snetwork_interface_info unconditionally.

283 284 285
	* menu.c (have_boxes): Fix redundant simulation of radio buttons
	in NS GUI sessions.  (Bug#15629)

286 287 288 289
2013-10-16  Dmitry Antipov  <dmantipov@yandex.ru>

	* fns.c (Fstring_as_unibyte): Use xlispstrdup.

290 291 292 293
2013-10-15  Paul Eggert  <eggert@cs.ucla.edu>

	* print.c (print_object): Print " ..." when truncating bool vectors.

294 295 296 297 298 299 300
2013-10-15  Eli Zaretskii  <eliz@gnu.org>

	* w32inevt.c (do_mouse_event): Support mouse wheel and all the 5
	standard mouse buttons.

	* termhooks.h (struct input_event): Remove incorrect commentary.

301 302
2013-10-15  Paul Eggert  <eggert@cs.ucla.edu>

303 304 305 306 307 308 309
	Disallow bool vector operations on mixed-length operands.
	The old behavior left garbage in the result vector sometimes,
	and didn't seem to be useful.
	* data.c (Qwrong_length_argument): New static var.
	(wrong_length_argument): New function.
	(bool_vector_binop_driver): Check that args agree in length.

310 311
	* keyboard.c, keyboard.h (all_kboards): Now static.

312 313 314 315 316
2013-10-15  Xue Fuqiao  <xfq.free@gmail.com>

	* buffer.c (syms_of_buffer) <buffer-invisibility-spec>: Add usage
	note from the lispref.

317 318 319 320 321 322 323
2013-10-15  Dmitry Antipov  <dmantipov@yandex.ru>

	* nsterm.h (struct ns_display_info): Remove set-but-unused
	member image_cache (image caches are per-terminal anyway).
	(FRAME_X_IMAGE_CACHE): Remove.
	* nsterm.m (ns_initialize_display_info): Adjust user.

324 325
2013-10-14  Eli Zaretskii  <eliz@gnu.org>

326 327 328 329
	* w32proc.c: Include mingw_time.h.

	* w32.c: Include mingw_time.h.

330 331 332 333 334 335 336 337 338 339 340 341 342 343
	Implement scrolling of TTY menus when the screen is too short.

	* term.c (tty_menu_display): Accept an additional argument, the
	menu item from which to start displaying the menu.  Account for
	the value of Y when limiting the menu to the number of available
	screen lines.
	(mi_result): New enumeration.
	(read_menu_input): Return enumerated value.  When the y coordinate
	hits min_y or max_y, return scroll indication instead of wrapping
	around the menu.
	(tty_menu_activate): Handle the scrolling indications from
	read_menu_input.  Compute the first menu item to display and pass
	it to tty_menu_display.

344 345 346 347 348 349 350 351 352 353 354 355 356 357 358 359 360 361 362
2013-10-14  Dmitry Antipov  <dmantipov@yandex.ru>

	* termhooks.h (FRAME_MUST_WRITE_SPACES, FRAME_LINE_INS_DEL_OK)
	(FRAME_CHAR_INS_DEL_OK, FRAME_SCROLL_REGION_OK)
	(FRAME_SCROLL_REGION_COST, FRAME_MEMORY_BELOW_FRAME):
	Adjust to match the change described below.
	(struct terminal): Move must_write_spaces, line_ins_del_ok,
	char_ins_del_ok, scroll_region_ok, scroll_region_cost and
	memory_below_frame members to...
	* termchar.h (struct tty_display_info): ...here because they're
	relevant only on TTYs.  Prefer unsigned bitfield where appropriate.
	* term.c (init_tty):
	* nsterm.m (ns_create_terminal):
	* w32term.c (w32_create_terminal):
	* xterm.c (x_create_terminal): Adjust users.
	* dispnew.c (line_hash_code, line_draw_cost): Pass frame arg
	to filter out non-TTY frames.  Adjust comment.
	(scrolling): Adjust user.  Prefer eassert for debugging check.

363 364 365 366 367 368 369 370 371 372 373
2013-10-14  Dmitry Antipov  <dmantipov@yandex.ru>

	* xfaces.c (PT_PER_INCH): Remove unused macro.
	* termhooks.h (struct terminal): Remove set-but-unused
	member fast_clear_end_of_line.
	(FRAME_FAST_CLEAR_END_OF_LINE): Remove.
	* nsterm.m (ns_create_terminal):
	* term.c (init_tty):
	* w32term.c (w32_create_terminal):
	* xterm.c (x_create_terminal): Adjust users.

374 375 376 377 378 379 380 381 382
2013-10-14  Paul Eggert  <eggert@cs.ucla.edu>

	* lisp.h (bool_vector_size): New function.
	All uses of XBOOL_VECTOR (x)->size changed to bool_vector_size (x).
	* data.c (bool_vector_spare_mask, bool_vector_binop_driver)
	(Fbool_vector_not, Fbool_vector_count_matches_at):
	Remove uses of 'eassume' that should no longer be needed,
	because they are subsumed by the 'eassume' in bool_vector_size.

383 384 385 386 387 388
2013-10-12  Eli Zaretskii  <eliz@gnu.org>

	* image.c (GIFLIB_MAJOR, GIFLIB_MINOR, GIFLIB_RELEASE): Move back
	after inclusion of gif_lib.h, thus fixing compiler warnings caused
	by 2013-10-10T19:15:33Z!eggert@cs.ucla.edu.

389 390 391 392 393 394 395 396 397 398
2013-10-11  Eli Zaretskii  <eliz@gnu.org>

	* xdisp.c (deep_copy_glyph_row): Handle the case that FROM and TO
	have different dimensions.  (Bug#15575)

	* dispnew.c (fill_up_frame_row_with_spaces): Now has external
	visibility.

	* dispextern.h (fill_up_frame_row_with_spaces): Add prototype.

399 400 401 402 403
2013-10-11  Dmitry Antipov  <dmantipov@yandex.ru>

	* term.c (tty_menu_show): Never return with unbalanced
	specpdl.  Use eassert for debugging check.  Adjust style.

404 405 406 407 408 409
2013-10-11  Eli Zaretskii  <eliz@gnu.org>

	* term.c (read_menu_input): Make selection of menu items
	cyclical.  Suggested by Dmitry Antipov <dmantipov@yandex.ru>.
	(tty_menu_activate): Fix off-by-one error when computing max_y.

410 411 412 413 414
2013-10-11  Teodor Zlatanov  <tzz@lifelogs.com>

	* gnutls.c (gnutls_audit_log_function): Add function for GnuTLS
	audit logging (only used with GnuTLS 3.x) and enable it.

415 416 417 418 419 420 421
2013-10-11  Dmitry Antipov  <dmantipov@yandex.ru>

	* dispnew.c (redraw_frame): Remove useless #ifdef because
	FRAME_MSDOS_P is a compile-time zero everywhere except MS-DOS.
	Also, move TTY fflush to...
	* term.c (tty_update_end): ...this function.

422 423 424 425 426 427 428 429 430 431 432 433
2013-10-11  Eli Zaretskii  <eliz@gnu.org>

	* xdisp.c (display_tty_menu_item): Make sure we never write beyond
	the end of the frame's glyph matrix.  (Bug#15575)

	* term.c (tty_menu_display): Don't move cursor while overwriting
	frame's glyphs with menu items.  Limit the number of items
	displayed to what can be shown on the available screen lines,
	excluding the echo area.
	(tty_menu_activate): Limit the Y coordinate allowed by
	read_menu_input to the last screen line used for menu display.

Paul Eggert's avatar
Paul Eggert committed
434 435 436 437 438 439 440 441 442 443 444 445 446 447 448 449 450 451 452 453 454
2013-10-11  Paul Eggert  <eggert@cs.ucla.edu>

	* lisp.h (eassume): New macro.
	Also, include <verify.h>, for 'assume'.
	* alloc.c (bool_vector_payload_bytes, Fmake_bool_vector)
	(vroundup, vector_nbytes):
	* data.c (bool_vector_spare_mask, bool_vector_binop_driver)
	(Fbool_vector_not, Fbool_vector_count_matches)
	(Fbool_vector_count_matches_at):
	Use eassume, not eassert.
	* casetab.c (set_identity, shuffle):
	* composite.c (composition_gstring_put_cache):
	* dispnew.c (update_frame_1):
	* ftfont.c (ftfont_shape_by_flt):
	* image.c (gif_load):
	* intervals.c (offset_intervals):
	* macfont.m (macfont_shape):
	Remove calls to 'assume' that are no longer needed, because
	--enable-gcc-warnings no longer generates bogus warnings
	when these calls are removed.

455 456 457 458
2013-10-11  Dmitry Antipov  <dmantipov@yandex.ru>

	* xdisp.c (deep_copy_glyph_row): Remove unused locals.

459 460 461 462 463 464
2013-10-10  Stefan Monnier  <monnier@iro.umontreal.ca>

	* fileio.c (Fsubstitute_in_file_name): Use substitute-env-in-file-name.
	(Qsubstitute_env_in_file_name): New var.
	(syms_of_fileio): Define it.

465 466 467 468 469 470 471 472 473 474 475
2013-10-10  Eli Zaretskii  <eliz@gnu.org>

	* xdisp.c (deep_copy_glyph_row): Assert that the 'used' counts of
	FROM and TO are identical.  Copy only the glyphs of TEXT_AREA.
	(Bug#15575)

	* term.c (save_and_enable_current_matrix): Don't allocate and
	don't save margin areas.
	(restore_desired_matrix): Don't restore margin areas.
	(free_saved_screen): Don't free margin areas.

476 477 478 479 480 481
2013-10-10  Paul Eggert  <eggert@cs.ucla.edu>

	* image.c: Pacify --enable-gcc-warnings.
	(GIFLIB_MAJOR, GIFLIB_MINOR, GIFLIB_RELEASE, fn_GifErrorString):
	#define only if used.

482 483 484 485 486 487 488 489 490 491
2013-10-10  Eli Zaretskii  <eliz@gnu.org>

	* image.c (GIFLIB_MAJOR): Define to 4 if undefined.
	(GIFLIB_MINOR, GIFLIB_RELEASE): Define to zero if undefined.
	(GifErrorString) [GIFLIB_MAJOR >= 5]: Define a function pointer.
	(gif_load): For giflib v5.x and later, display the error message
	produced by giflib when its functions fail.
	(syms_of_image) <Qlibgif_version> [HAVE_NTGUI]: New DEFSYM.
	(Bug#15531)

492 493 494 495 496 497 498 499 500 501 502 503 504 505
2013-10-10  Dmitry Antipov  <dmantipov@yandex.ru>

	* keyboard.c (last_event_timestamp): Remove.  For X selection and
	GTK popup menus, it may be obtained from per-frame X display info.
	(kbd_buffer_store_event_hold, kbd_buffer_get_event)
	(process_special_events): Adjust users.
	* keyboard.h (last_event_timestamp): Remove declaration.
	* xmenu.c (xmenu_show, create_and_show_popup_menu): Lost last arg.
	Use FRAME_DISPLAY_INFO (f)->last_user_time for gtk_menu_popup.
	* menu.h (xmenu_show): Adjust prototype.
	* menu.c (Fx_popup_menu): Adjust user.
	* xselect.c (x_own_selection, x_get_foreign_selection)
	(Fx_disown_selection_internal): Use dpyinfo->last_user_time.

506 507 508 509 510 511 512 513 514 515 516 517 518 519
2013-10-10  Dmitry Antipov  <dmantipov@yandex.ru>

	* keyboard.c (init_kboard): Now static.  Add arg
	to denote window system.  Adjust comment.
	(init_keyboard): Adjust user.
	(allocate_kboard): New function.
	(syms_of_keyboard):
	* nsterm.m (ns_term_init):
	* term.c (init_tty):
	* w32term.c (w32_create_terminal):
	* xterm.c (x_term_init): Use it.
	* keyboard.h (init_kboard): Remove prototype.
	(allocate_kboard): Add prototype.

520 521 522 523 524
2013-10-10  Barry Fishman  <barry_fishman@acm.org>  (tiny change)

	* image.c (GIFLIB_MAJOR): Ensure it's defined.
	(DGifOpen, DGifOpenFileName): Handle giflib 5 syntax.  (Bug#15531)

525 526 527 528
2013-10-09  Paul Eggert  <eggert@cs.ucla.edu>

	* fns.c (sxhash_bool_vector): Fix buffer read overrun.

529 530 531 532 533 534 535
2013-10-09  Eli Zaretskii  <eliz@gnu.org>

	* term.c (tty_menu_activate): Flush the output stream after
	showing the cursor, and don't mark the frame garbaged at exit from
	the function.  Fixes redisplay glitches when moving from one menu
	to another.

536 537 538 539 540
2013-10-09  Jan Djärv  <jan.h.d@swipnet.se>

	* nsfns.m (Fns_convert_utf8_nfd_to_nfc): Check input for valid UTF-8
	or throw error (Bug#15570).

541 542
2013-10-09  Paul Eggert  <eggert@cs.ucla.edu>

543 544 545
	* intervals.c (temp_set_point_both): Move test into 'eassert',
	for speed.

546 547 548 549 550
	* lisp.h (eassert): Don't use 'assume'.
	Sometimes 'assume' wins in performance, and sometimes it loses,
	so it shouldn't be used all the time.  Perhaps we need two
	flavors of 'eassert', one for where 'assume' is far more likely
	to help or to hurt; but that can be done later.
551
	Problem reported by Dmitry Antipov in
552 553 554
	<http://lists.gnu.org/archive/html/emacs-devel/2013-10/msg00276.html>.
	Also, don't include <verify.h>; no longer needed.

Glenn Morris's avatar
Glenn Morris committed
555 556 557 558
2013-10-09  Glenn Morris  <rgm@gnu.org>

	* eval.c (Fcond): Doc tweak.

559 560 561 562 563
2013-10-09  Eli Zaretskii  <eliz@gnu.org>

	* xfaces.c (x_free_gc) [HAVE_X_WINDOWS, HAVE_NTGUI]: Don't pass
	expressions with side effects to eassert.  (Bug#15565)

564 565 566 567 568
2013-10-09  Stefan Monnier  <monnier@iro.umontreal.ca>

	* fns.c (hashfn_user_defined): Allow hash functions to return any
	Lisp_Object.

569 570 571 572 573 574 575 576 577 578 579 580 581 582 583 584 585 586 587 588 589 590 591 592 593 594
2013-10-08  Paul Eggert  <eggert@cs.ucla.edu>

	Fix minor problems found by static checking.
	* dispnew.c (save_current_matrix): Omit unnecessary casts.
	* dispnew.c (update_frame_with_menu): Mark debug local as used.
	* keyboard.c, keyboard.h (Qmouse_movement): Now static.
	* keyboard.c (read_menu_command): Remove unused local.
	* lisp.h (read_menu_command): New decl.
	* menu.c, menu.h (menu_item_width): Arg is now unsigned char *, for
	benefit of STRING_CHAR_AND_LENGTH.  All uses changed.
	Return ptrdiff_t, not int.
	* term.c (tty_menu_struct): 'allocated' member is now ptrdiff_t,
	not int, for benefit of xpalloc.
	(tty_menu_create, tty_menu_make_room): Simplify by using xzalloc
	and xpalloc.
	(have_menus_p): Remove; unused.
	(tty_menu_add_pane, tty_menu_add_selection): Change signedness of
	local char * pointer to pacify STRING_CHAR_AND_LENGTH.
	(tty_menu_add_selection, tty_menu_locate, tty_meny_destroy):
	Now static.
	(save_and_enable_current_matrix): Omit unnecessary casts.
	(read_menu_input): Omit local extern decl (now in lisp.h).
	Don't access uninitialized storage if mouse_get_xy fails.
	(tty_menu_activate): Mark local as initialized, for lint.
	(tty_menu_activate, tty_meny_show): Remove unused locals.

595 596 597 598 599 600 601 602 603 604
2013-10-08  Eli Zaretskii  <eliz@gnu.org>

	Support menus on text-mode terminals.
	* xterm.h (xw_popup_dialog): Add prototype.

	* xmenu.c (Fx_popup_dialog): Function moved to menu.c.
	(xmenu_show): Block input here, instead in Fx_popup_menu.
	(xw_popup_dialog): New function, with X-specific bits of popup
	dialogs.

605 606
	* xdisp.c (deep_copy_glyph_row, display_tty_menu_item):
	New functions.
607 608 609 610 611 612 613 614 615 616 617 618 619 620 621 622 623 624 625 626 627 628 629 630 631 632 633 634 635 636 637 638 639 640 641 642 643 644 645 646 647 648 649 650 651 652 653 654 655 656 657 658 659 660 661 662 663 664 665 666 667 668 669 670

	* window.c (Fset_window_configuration): Use run-time tests of the
	frame type instead of compile-time conditionals, when menu-bar
	lines are considered.

	* w32term.h (w32con_hide_cursor, w32con_show_cursor)
	(w32_popup_dialog): New prototypes.

	* w32menu.c (Fx_popup_dialog): Function deleted.
	(w32_popup_dialog): New function, with w32 specific bits of popup
	dialogs.  Block input here.

	* w32inevt.c (w32_console_read_socket): Minor change to add
	debugging TTY events.

	* w32fns.c (show_hourglass): If returning early because the frame
	is not a GUI frame, unblock input.

	* w32console.c (w32con_hide_cursor, w32con_show_cursor, cursorX)
	(cursorY): New functions.

	* termhooks.h (cursorX, cursorY): Prototypes of functions on
	WINDOWSNT, macros that call curX and curY elsewhere.

	* termchar.h (struct tty_display_info) <showing_menu>: New flag.

	* term.c (tty_hide_cursor, tty_show_cursor) [WINDOWSNT]: Call w32
	specific function to hide and show cursor on a text-mode terminal.
	(tty_menu_struct, struct tty_menu_state): New structures.
	(tty_menu_create, tty_menu_make_room, tty_menu_search_pane)
	(tty_menu_calc_size, mouse_get_xy, tty_menu_display)
	(have_menus_p, tty_menu_add_pane, tty_menu_add_selection)
	(tty_menu_locate, save_and_enable_current_matrix)
	(restore_desired_matrix, screen_update, read_menu_input)
	(tty_menu_activate, tty_menu_destroy, tty_menu_help_callback)
	(tty_pop_down_menu, tty_menu_last_menubar_item)
	(tty_menu_new_item_coords, tty_menu_show): New functions.
	(syms_of_term): New DEFSYMs for tty-menu-* symbols.

	* nsterm.h (ns_popup_dialog): Adjust prototype.

	* nsmenu.m (ns_menu_show): Block and unblock input here, instead
	of in x-popup-menu.
	(ns_popup_dialog): Adapt order of arguments to the other
	*_menu_show implementations.
	(Fx_popup_dialog): Function deleted.

	* msdos.c (x_set_menu_bar_lines): Delete unused function.

	* menu.h (tty_menu_show, menu_item_width): provide prototypes.

	* menu.c (have_boxes): New function.
	(single_keymap_panes): Use it instead of a compile-time
	conditional.
	(single_menu_item): Use run-time tests of the frame type instead
	of compile-time conditionals.
	(encode_menu_string): New function.
	(list_of_items, list_of_panes): Use it instead of ENCODE_STRING
	the macro, since different types of frame need different encoding
	of menu items.
	(digest_single_submenu): Use run-time tests of frame type instead
	of, or in addition to, compile-time conditionals.
	(menu_item_width, Fmenu_bar_menu_at_x_y): New functions.
	(Fx_popup_menu): Detect when the function is called from keyboard
671 672 673 674 675
	on a TTY.  Don't barf when invoked on a text-mode frame.
	Check frame type at run time, instead of compile-time conditionals for
	invoking terminal-specific menu-show functions.
	Call tty_menu_show on text-mode frames.
	(Fx_popup_dialog): Move here from xmenu.c.  Test frame types at
676 677 678 679 680 681 682 683 684 685 686 687 688 689 690 691 692 693 694 695 696 697 698 699 700 701 702 703 704 705 706 707
	run time to determine which alternative to invoke; support dialogs
	on TTYs.

	* keyboard.h <Qmouse_movement>: Declare.

	* keyboard.c <Qmouse_movement>: Now extern.
	<Qecho_keystrokes>: New static variable.
	(read_key_sequence): Accept an additional argument, a flag to
	prevent redisplay during reading of the key sequence.  All callers
	changed.
	(read_menu_command): New function.
	(read_char): When COMMANDFLAG is -2, do not redisplay and do not
	autosave.
	(toolkit_menubar_in_use): New function.
	(make_lispy_event): Use it instead of a compile-time test.

	* fns.c (Fyes_or_no_p) [HAVE_MENUS]: Don't condition on
	window-system being available.

	* editfns.c (Fmessage_box) [HAVE_MENUS]: Don't condition the call
	to x-popup-dialog on the frame type, they all now support popup
	dialogs.

	* dispnew.c (save_current_matrix): Save the margin areas.
	(restore_current_matrix): Restore margin areas.
	(update_frame_with_menu): New function.

	* dispextern.h (display_tty_menu_item, update_frame_with_menu):
	Add prototypes.

	* alloc.c (make_save_ptr): Now compiled unconditionally.

708 709 710 711 712 713 714 715 716
2013-10-08  Dmitry Antipov  <dmantipov@yandex.ru>

	* dispnew.c (set_window_update_flags): Add buffer arg.  Adjust comment.
	(redraw_frame, update_frame): Adjust users.
	* dispextern.h (set_window_update_flags): Adjust prototype.
	* xdisp.c (redisplay_internal): When updating all frames with zero
	windows_or_buffers_changed, assume that only the windows that shows
	current buffer should be really updated.

717 718 719 720 721 722 723 724 725 726 727 728 729
2013-10-08  Dmitry Antipov  <dmantipov@yandex.ru>

	Do not allocate huge temporary memory areas and objects while encoding
	for file I/O, thus reducing an enormous memory usage for large buffers.
	See http://lists.gnu.org/archive/html/emacs-devel/2013-10/msg00180.html.
	* coding.h (struct coding_system): New member raw_destination.
	* coding.c (setup_coding_system): Initialize it to zero.
	(encode_coding_object): If raw_destination is set, do not create
	dst_object.  Add comment.
	* fileio.c (toplevel): New constant E_WRITE_MAX.
	(e_write): Do not encode more than E_WRITE_MAX characters per one loop
	iteration.  Use raw_destination if E_WRITE_MAX characters is encoded.

730 731 732 733 734
2013-10-08  Jan Djärv  <jan.h.d@swipnet.se>

	* nsterm.m (windowDidExitFullScreen:):
	(toggleFullScreen:): Change NS_IMPL_COCOA to HAVE_NATIVE_FS.

735 736
2013-10-08  Paul Eggert  <eggert@cs.ucla.edu>

737 738 739 740
	Fix race where emacs aborts when sent SIGTERM (Bug#15534).
	* keyboard.c (unblock_input_to): Don't process pending signals
	if a fatal error is in progress.

741 742 743 744
	* lisp.h (bits_word, BITS_WORD_MAX): New type and macro.
	All uses of 'size_t' and 'SIZE_MAX' changed to use them, when
	they're talking about words in Lisp bool vectors.
	(BITS_PER_BITS_WORD): Rename from BITS_PER_SIZE_T.  All uses changed.
745 746 747
	* data.c (popcount_bits_word): Rename from popcount_size_t.
	(bits_word_to_host_endian): Rename from size_t_to_host_endian.
	All uses changed.
748

749 750 751 752 753 754 755 756 757 758 759 760 761 762 763 764 765 766 767
2013-10-07  Paul Eggert  <eggert@cs.ucla.edu>

	Improve support for popcount and counting trailing zeros (Bug#15550).
	* data.c: Include <count-one-bits.h>, <count-trailing-zeros.h>.
	(USE_MSC_POPCOUNT, POPCOUNT_STATIC_INLINE)
	(NEED_GENERIC_POPCOUNT, popcount_size_t_generic)
	(popcount_size_t_msc, popcount_size_t_gcc):
	Remove; now done by Gnulib.
	(popcount_size_t): Now a macro that defers to Gnulib.
	(count_trailing_zero_bits): Return int, for consistency with
	Gnulib and because Emacs prefers signed to unsigned int.
	Don't assume that size_t is either unsigned int or unsigned long
	or unsigned long long.
	(size_t_to_host_endian): Do not assume that size_t is either
	exactly 32 or exactly 64 bits wide.
	* lisp.h (BITS_PER_SIZE_T): Define consistently with BITS_PER_LONG
	etc., so that it's now an enum constant, not a macro.
	No need to assume that it's either 32 or 64.

768 769 770 771 772
2013-10-07  Jan Djärv  <jan.h.d@swipnet.se>

	* nsterm.m (windowDidEnterFullScreen:): setPresentationOptions only
	on >= 10.7.

773 774 775
2013-10-07  Dmitry Antipov  <dmantipov@yandex.ru>

	* insdel.c (insert_from_gap): Prefer ptrdiff_t to int where needed.
776
	* xdisp.c (handle_fontified_prop): Likewise.  Use bool for boolean.
777

778 779 780 781 782 783 784 785 786
2013-10-07  Paul Eggert  <eggert@cs.ucla.edu>

	emacs_read and emacs_write now use void *, not char *.
	* alloc.c (valid_pointer_p) [!WINDOWSNT]: Remove now-unnecessary cast.
	* sysdep.c (emacs_read, emacs_write, emacs_write_sig):
	Buffer arg is now void *, not char *.  This matches plain
	'read' and 'write' better, and avoids a constraint violation
	on Solaris 9 with Oracle Studio.

787 788 789 790 791 792 793
2013-10-07  Dmitry Antipov  <dmantipov@yandex.ru>

	* alloc.c (Fmake_string): For ASCII char initializer, prefer
	memset to explicit loop.  Otherwise copy largest possible chunk
	from initialized to uninitialized part, thus allowing the longer
	memcpy runs and reducing the number of loop iterations.

794 795 796 797 798 799
2013-10-06  Jan Djärv  <jan.h.d@swipnet.se>

	* nsterm.m (ns_update_begin): If native fullscreen and no toolbar,
	hide toolbar (Bug#15388).
	(windowDidEnterFullScreen:): If presentation options are zero,
	set them here (Bug#15388).
800 801
	(ns_update_auto_hide_menu_bar): Remove runtime check.
	Don't auto hide dock unless menubar is also auto hidden.
802

Xue Fuqiao's avatar
Xue Fuqiao committed
803 804 805 806
2013-10-05  Xue Fuqiao  <xfq.free@gmail.com>

	* editfns.c (message): Mention batch mode in doc string.

807 808 809 810
2013-10-05  Jan Djärv  <jan.h.d@swipnet.se>

	* nsterm.m (check_native_fs): Remove erroneous comment.

811 812 813 814 815
2013-10-04  Dmitry Antipov  <dmantipov@yandex.ru>

	* xdisp.c (redisplay_internal): Simplify because scan_for_column now
	uses find_newline instead of scan_newline and so doesn't move point.

816 817
2013-10-04  Paul Eggert  <eggert@cs.ucla.edu>

818 819 820 821 822 823 824 825 826
	Use hardware support for byteswapping on glibc x86 etc.
	On Fedora 19 x86-64, the new bswap_64 needs 1 instruction,
	whereas the old swap64 needed 30.
	* fringe.c (init_fringe_bitmap) [WORDS_BIGENDIAN]:
	* sound.c (le2hl, le2hs, be2hl) [!WINDOWSNT]:
	Use byteswap.h's macros to swap bytes.
	* lisp.h (swap16, swap32, swap64): Remove.
	All uses replaced by bswap_16, bswap_32, bswap_64.

827 828 829 830
	* bytecode.c (exec_byte_code): Use some more volatile variables
	to work around local variables getting clobbered by longjmp.
	Port to pre-C99, which doesn't allow decls after stmts.

831 832
2013-10-03  Paul Eggert  <eggert@cs.ucla.edu>

833 834 835 836 837
	* lisp.h (eassert): Assume that COND is true when optimizing.
	In other words, take on the behavior of eassert_and_assume.
	This makes Emacs 0.2% smaller on my platform (Fedora 19, x86-64).
	(eassert_and_assume): Remove.  All uses replaced by eassert.

838 839
	* xdisp.c (Qglyphless_char): Now static.

Paul Eggert's avatar
Paul Eggert committed
840 841 842 843
	Adjust to merge from gnulib.
	* conf_post.h (__has_builtin, assume): Remove; gnulib now does these.
	* lisp.h: Include <verify.h>, for 'assume'.

844 845 846 847 848 849 850 851 852
	* eval.c (clobbered_eassert): New macro.
	(internal_catch, internal_condition_case)
	(internal_condition_case_1, internal_condition_case_2)
	(internal_condition_case_n): Use it instead of eassert
	when the argument contains locals clobbered by longjmp.
	Don't use clobbered locals outside of clobbered_eassert.
	(internal_lisp_condition_case): Use a volatile variable
	to work around a local variable's getting clobbered.

853 854 855 856 857 858 859 860 861 862 863 864 865 866 867 868 869 870 871 872 873 874 875 876 877 878 879 880 881 882
2013-10-03  Stefan Monnier  <monnier@iro.umontreal.ca>

	* lisp.h (struct handler): Merge struct handler and struct catchtag.
	(PUSH_HANDLER): New macro.
	(catchlist): Remove.
	(handlerlist): Always declare.

	* eval.c (catchlist): Remove (merge with handlerlist).
	(handlerlist, lisp_eval_depth): Not static any more.
	(internal_catch, internal_condition_case, internal_condition_case_1)
	(internal_condition_case_2, internal_condition_case_n):
	Use PUSH_HANDLER.
	(unwind_to_catch, Fthrow, Fsignal): Adjust to merged
	handlerlist/catchlist.
	(internal_lisp_condition_case): Use PUSH_HANDLER.  Adjust to new
	handlerlist which can only handle a single condition-case handler at
	a time.
	(find_handler_clause): Simplify since we only a single branch here
	any more.

	* bytecode.c (BYTE_CODES): Add Bpushcatch, Bpushconditioncase
	and Bpophandler.
	(bcall0): New function.
	(exec_byte_code): Add corresponding cases.  Improve error message when
	encountering an invalid byte-code.  Let Bunwind_protect accept
	a function (rather than a list of expressions) as argument.

	* alloc.c (Fgarbage_collect): Merge scans of handlerlist and catchlist,
	and make them unconditional now that they're heap-allocated.

883 884 885 886 887
2013-10-03  Stefan Monnier  <monnier@iro.umontreal.ca>

	* charset.c (Fdecode_char, Fencode_char): Remove description of
	`restriction' arg. now that it's hidden by advertised-calling-convention.

888 889
2013-10-02  Jan Djärv  <jan.h.d@swipnet.se>

890 891
	* macfont.m (mac_ctfont_create_preferred_family_for_attributes):
	Remove unused variable (from mac-port).
892 893
	(macfont_draw): Use s->ybase for correct y position.

894 895 896 897 898 899
2013-10-02  Dmitry Antipov  <dmantipov@yandex.ru>

	* frame.h (struct frame): Drop has_minibuffer member because...
	(FRAME_HAS_MINIBUF_P): ...this macro can be implemented without it.
	* frame.c (make_frame, make_minibuffer_frame): Adjust users.

900 901 902 903 904 905 906
2013-10-02  Dmitry Antipov  <dmantipov@yandex.ru>

	* window.h (struct window): Prefer enum text_cursor_kinds to int
	for phys_cursor_type member.  Move the latter, phys_cursor_width,
	phys_cursor_ascent and phys_cursor_height under HAVE_WINDOW_SYSTEM.
	* window.c (replace_window, make_window): Adjust users.

907 908 909 910 911
2013-10-02  Dmitry Antipov  <dmantipov@yandex.ru>

	* fringe.c (toplevel): Do not use HAVE_WINDOW_SYSTEM because
	this module is never compiled otherwise.

912 913 914 915 916
2013-10-01  Alp Aker  <alp.tekin.aker@gmail.com>

	* macfont.m (macfont_draw): Use CGRectMake rather than NSMakeRect
	(Bug#15500).

917 918 919 920 921
2013-09-29  Eli Zaretskii  <eliz@gnu.org>

	* xdisp.c (get_next_display_element): Don't call face_for_font in
	a build configured --without-x.  (Bug#15484)

922 923
2013-09-29  Jan Djärv  <jan.h.d@swipnet.se>

924 925
	* window.c (calc_absolute_offset): #elif should be #elif defined.

926 927 928
	* frame.c (delete_frame): Block/unblock input to overcome race
	condition (Bug#15475).

929 930 931 932 933
2013-09-29  Andreas Politz  <politza@hochschule-trier.de>  (tiny change)

	* frame.c (delete_frame): Record selected frame only after
	calling Qdelete_frame_functions (Bug#15477).

934 935 936 937 938 939 940 941
2013-09-28  Jan Djärv  <jan.h.d@swipnet.se>

	* nsterm.m (ns_selection_color): Remove.
	(ns_get_color): Check for ns_selection_(fg|bg)_color using
	NSColor selectedText(Background)Color.  Only for COCOA.
	(ns_term_init): Remove assignment of ns_selection_color, logic
	moved to ns_get_color.

942
	* nsterm.h (NS_SELECTION_BG_COLOR_DEFAULT): Rename from
943 944 945
	NS_SELECTION_COLOR_DEFAULT.
	(NS_SELECTION_FG_COLOR_DEFAULT): New.

946 947 948 949 950
2013-09-28  Eli Zaretskii  <eliz@gnu.org>

	* xdisp.c (Fdump_tool_bar_row): Ifdef away the body if 'struct
	frame' does not have the tool_bar_window member.

951 952 953 954 955 956 957
2013-09-26  Barry O'Reilly  <gundaetiapo@gmail.com>

	Signal error when reading an empty byte-code object (Bug#15405)
	* lread.c (read1): signal error
	* alloc.c (make_byte_code): eassert header size
	(sweep_vectors): change an int to size_t

958 959
2013-09-24  Paul Eggert  <eggert@cs.ucla.edu>

960 961 962 963
	* dispnew.c (clear_glyph_row, copy_row_except_pointers): Use enums
	instead of ints, as it's the usual style for offsetof constants.  See:
	http://lists.gnu.org/archive/html/emacs-devel/2013-09/msg00478.html

964 965 966 967 968 969 970 971 972 973 974
	* data.c (POPCOUNT_STATIC_INLINE): New macro, as a hack for popcount.
	This is ugly, but it should fix the performance problem for older
	GCC versions in the short run.  I'll look into integrating the
	Gnulib module for popcount, as a better fix.
	See the thread starting in:
	http://lists.gnu.org/archive/html/emacs-devel/2013-09/msg00474.html
	(popcount_size_t_generic) [NEED_GENERIC_POPCOUNT]:
	(popcount_size_t_msc) [USE_MSC_POPCOUNT]:
	(popcount_size_t_gcc) [USE_GCC_POPCOUNT]:
	(popcount_size_t): Use it.

975 976 977 978 979
2013-09-24  Daniel Colascione  <dancol@dancol.org>

	* process.c (Fnetwork_interface_info): Fix build break due to
	vector changes.

980 981 982 983 984 985 986 987
2013-09-24  Paul Eggert  <eggert@cs.ucla.edu>

	* dispnew.c (clear_glyph_row, copy_row_except_pointers):
	Prefer signed to unsigned integers where either will do.
	No need for 'const' on locals that do not escape.
	Omit easserts with unnecessary and unportable assumptions about
	alignment.  Avoid unnecessary casts to char *.

988 989 990 991 992 993 994 995 996 997 998 999 1000
2013-09-24  Dmitry Antipov  <dmantipov@yandex.ru>

	Use union for the payload of struct Lisp_Vector.
	This helps to avoid a few glitches dictated by C's aliasing rules.
	* lisp.h (struct Lisp_Vector): Use union for next and
	contents member.  Adjust comment.  Change related users.
	* alloc.c (next_in_free_list, set_next_in_free_list): Remove.
	Related users changed.
	* buffer.c, bytecode.c, ccl.c, character.h, chartab.c, composite.c:
	* composite.h, disptab.h, fns.c, fontset.c, indent.c, keyboard.c:
	* lread.c, msdos.c, process.c, w32menu.c, window.c, xdisp.c:
	* xfaces.c, xfont.c, xmenu.c: Related users changed.

1001 1002 1003 1004 1005 1006 1007 1008 1009 1010
2013-09-24  Dmitry Antipov  <dmantipov@yandex.ru>

	Optimize glyph row clearing and copying routines.
	* dispextern.h (struct glyph_row): Change layout of struct
	glyph_row to help copy_row_except_pointers.  Adjust comment.
	* dispnew.c (null_row): Remove.
	(clear_glyph_row): Use offsetof and memset to find and clear
	just the members that need clearing.  Adjust comment.
	(copy_row_except_pointers): Likewise for copying.

1011 1012
2013-09-24  Paul Eggert  <eggert@cs.ucla.edu>

1013 1014 1015 1016 1017 1018 1019 1020 1021 1022 1023 1024 1025 1026
	Some minor cleanups of recently-added bool vector code.
	* conf_post.h (assume): Always return void.  Use lint version
	only if GCC and MSC versions don't apply.
	* conf_post.h (assume):
	* data.c (USC_MSC_POPCOUNT, count_trailing_zero_bits):
	Depend on _MSC_VER, not __MSC_VER, for consistency with
	the rest of Emacs.
	* data.c (bool_vector_spare_mask, popcount_size_t_generic)
	(popcount_size_t_msc, popcount_size_t_gcc, popcount_size_t)
	(bool_vector_binop_driver, count_trailing_zero_bits)
	(size_t_to_host_endian): Now static, not static inline;
	the latter isn't needed with modern compilers and doesn't
	work with older compilers anyway.

1027 1028
	* alloc.c (valgrind_p): Use bool for boolean.

1029 1030 1031 1032 1033 1034 1035
2013-09-23  Dmitry Antipov  <dmantipov@yandex.ru>

	* xdisp.c (noninteractive_need_newline, message_log_need_newline)
	(overlay_arrow_seen, message_enable_multibyte, line_number_displayed)
	(display_last_displayed_message_p, message_buf_print)
	(message_cleared_p, help_echo_showing_p, hourglass_shown_p):
	Use bool for boolean.
1036 1037 1038
	* dispextern.h (cancel_line, init_desired_glyphs):
	Remove ancient leftover.
	(help_echo_showing_p, hourglass_shown_p):
1039 1040
	* lisp.h (noninteractive_need_newline): Adjust declaration.

1041 1042 1043 1044 1045 1046 1047 1048 1049 1050 1051 1052
2013-09-23  Dmitry Antipov  <dmantipov@yandex.ru>

	* dispnew.c (frame_garbaged, selected_frame, last_nonminibuf_frame):
	Move to...
	* frame.c (frame_garbaged, selected_frame, last_nonminibuf_frame):
	...this file and convert the latter to static.  Adjust comment.
	(make_initial_frame):
	* window.c (init_window_once): Adjust user.
	* frame.h (last_nonminibuf_frame): Remove declaration.
	* lisp.h (selected_frame): Likewise.
	* msdos.c (the_only_display_info): Adjust comment.

1053 1054
2013-09-23  Eli Zaretskii  <eliz@gnu.org>

1055 1056 1057
	* xdisp.c (mouse_face_from_string_pos): Fix off-by-one error in
	computing the end column of mouse-highlight that comes from
	display or overlay strings.  (Bug#15437)
Eli Zaretskii's avatar
Eli Zaretskii committed
1058 1059
	(note_mouse_highlight): Adapt calculation of last argument to
	mouse_face_from_string_pos to the above change.
1060

1061 1062 1063
	* conf_post.h (__has_builtin): Define to zero, if undefined, on
	all platforms, not just for clang.

1064 1065 1066 1067 1068 1069 1070
2013-09-23  Jan Djärv  <jan.h.d@swipnet.se>

	* filelock.c (lock_file_1): Rearrange to remove compiler warning
	about excess arguments to snprintf.

	* conf_post.h(assume): Use __builtin_unreachable for clang.

1071 1072 1073 1074 1075
2013-09-23  Juanma Barranquero  <lekktu@gmail.com>

	* w32console.c (initialize_w32_display): Remove unused variable hlinfo.
	* w32term.c (w32_scroll_bar_handle_click): Remove unused variable f.

1076 1077 1078 1079
2013-09-23  Daniel Colascione  <dancol@dancol.org>

	* alloc.c (USE_VALGRIND): New macro; on by default
	when ENABLE_CHECKING.
Paul Eggert's avatar
Paul Eggert committed
1080
	(mark_maybe_object, mark_maybe_pointer)
1081 1082 1083 1084 1085
	[USE_VALGRIND]: Mark conservatively-scanned regions valid for
	valgrind purposes.
	(valgrind_p) [USE_VALGRIND]: New variable.
	(init_alloc) [USE_VALGRIND]: Initialize valgrind_p.

1086 1087
2013-09-22  Jan Djärv  <jan.h.d@swipnet.se>

1088 1089 1090 1091 1092 1093 1094 1095 1096 1097
	* process.c (wait_reading_process_output): Change int pnamelen to
	socklen_t.

	* nsterm.m (setMarkedText:selectedRange:):
	(deleteWorkingText):
	* nsmenu.m (addDisplayItemWithImage:idx:tag:helpText:enabled:):
	* nsfont.m (ns_get_covering_families, ns_findfonts): Cast NSLog
	argument to unsigned long to avoid warning.
	(nsfont_draw): Use 0.25 instead of  Fix2X (kATSItalicQDSkew).

1098 1099
	* conf_post.h (assume): Fix compiler error: x shall be cond.

1100 1101 1102 1103 1104
2013-09-22  Daniel Colascione  <dancol@dancol.org>

	* xfns.c (x_get_monitor_attributes): Suppress unused variable
	warning when compiling without a window system.

1105 1106 1107
2013-09-22  Daniel Colascione  <dancol@dancol.org>

	* data.c (Qbool_vector_p): New symbol.
Paul Eggert's avatar
Paul Eggert committed
1108 1109
	(bool_vector_spare_mask, popcount_size_t_generic)
	(popcount_size_t_msc, popcount_size_t_gcc)
1110 1111
	(popcount_size_t)
	(bool_vector_binop_driver)
Paul Eggert's avatar
Paul Eggert committed
1112
	(count_trailing_zero_bits, size_t_to_host_endian)
1113 1114
	(Fbool_vector_exclusive_or)
	(Fbool_vector_union)
Paul Eggert's avatar
Paul Eggert committed
1115 1116
	(Fbool_vector_intersection, Fbool_vector_set_difference)
	(Fbool_vector_subsetp, Fbool_vector_not)
1117 1118 1119 1120 1121 1122 1123 1124
	(Fbool_vector_count_matches)
	(Fbool_vector_count_matches_at): New functions.
	(syms_of_data): Intern new symbol, functions.
	* alloc.c (bool_vector_payload_bytes): New function.
	(Fmake_bool_vector): Instead of calling Fmake_vector,
	which performs redundant initialization and argument checking,
	just call allocate_vector ourselves.  Make sure we clear any
	terminating padding to zero.
Paul Eggert's avatar
Paul Eggert committed
1125
	(vector_nbytes, sweep_vectors): Use bool_vector_payload_bytes
1126 1127 1128
	instead of open-coding the size calculation.
	(vroundup_ct): New macro.
	(vroundup): Assume argument >= 0; invoke vroundup_ct.
Paul Eggert's avatar
Paul Eggert committed
1129
	* casetab.c (shuffle, set_identity): Change lint_assume to assume.
1130 1131
	* composite.c (composition_gstring_put_cache):
	Change lint_assume to assume.
1132 1133 1134 1135 1136 1137 1138 1139
	* conf_post.h (assume): New macro.
	(lint_assume): Remove.
	* dispnew.c (update_frame_1): Change lint_assume to assume.
	* ftfont.c (ftfont_shape_by_flt): Change lint_assume
	to assume.
	* image.c (gif_load): Change lint_assume to assume.
	* lisp.h (eassert_and_assume): New macro.
	(Qbool_vector_p): Declare.
Paul Eggert's avatar
Paul Eggert committed
1140 1141
	(CHECK_BOOL_VECTOR, ROUNDUP, BITS_PER_SIZE_T): New macros.
	(swap16, swap32, swap64): New inline functions.
1142 1143 1144 1145 1146
	* macfont.c (macfont_shape): Change lint_assume to assume.
	* ralloc.c: Rename ROUNDUP to PAGE_ROUNDUP throughout.
	* xsettings.c (parse_settings): Use new swap16 and
	swap32 from lisp.h instead of file-specific macros.

1147 1148 1149 1150 1151 1152
2013-09-22  Eli Zaretskii  <eliz@gnu.org>

	* xdisp.c (try_window_id): Don't abort if cursor row could not be
	found (which can legitimately happen when the glyph row at the
	window start is disabled in the current_matrix.  (Bug#15365)

1153 1154 1155 1156 1157 1158 1159 1160 1161 1162 1163 1164 1165 1166 1167 1168 1169 1170
2013-09-22  Paul Eggert  <eggert@cs.ucla.edu>

	Fix syntax.h bug introduced by recent INLINE change.
	syntax.h defined an extern inline function SYNTAX_ENTRY that was
	conditionally compiled one way in some modules, and a different
	way in others.  This doesn't work with extern inline functions,
	which must have the same definition in all modules, because the
	defining code might be shared across modules, depending on the
	implementation.  Symptoms reported by Martin Rudalics in:
	http://lists.gnu.org/archive/html/emacs-devel/2013-09/msg00414.html
	* regex.c, syntax.c (SYNTAX_ENTRY_VIA_PROPERTY): Remove.
	(SYNTAX, SYNTAX_ENTRY, SYNTAX_WITH_FLAGS): New macros,
	overriding the corresponding functions in syntax.h.
	* syntax.h (syntax_property_entry, syntax_property_with_flags)
	(syntax_property): New inline functions.
	(SYNTAX_ENTRY, SYNTAX_WITH_FLAGS, SYNTAX):
	Rewrite in terms of these new functions.

1171 1172 1173 1174 1175 1176
2013-09-21  Eli Zaretskii  <eliz@gnu.org>

	* dired.c (directory_files_internal): Use multibyte_chars_in_text,
	not chars_in_text, whose result depends on the multibyteness of
	the current buffer.  (Bug#15426)

Paul Eggert's avatar
Paul Eggert committed
1177 1178
2013-09-20  Paul Eggert  <eggert@cs.ucla.edu>

1179 1180 1181 1182 1183
	Port recent change to hosts where pointers aren't 'long'.
	* xterm.c (x_send_scroll_bar_event, x_scroll_bar_to_input_event):
	Don't assume that pointers are the same width as 'long'.
	Add a compile-time check that a pointer fits into two X slots.

Paul Eggert's avatar
Paul Eggert committed
1184 1185 1186 1187 1188 1189 1190 1191 1192 1193 1194 1195 1196 1197 1198 1199 1200 1201 1202 1203 1204 1205 1206 1207 1208 1209
	A simpler, centralized INLINE.
	* conf_post.h (INLINE): Define only if not already defined.
	This allows us to use a single INLINE, defined by one file
	per executable.
	* emacs.c (INLINE): Define it.
	Also, include category.h, charset.h, composite.h, dispextern.h,
	syntax.h, systime.h, so that their INLINE definitions are expanded
	properly for Emacs.
	* blockinput.h, keyboard.c (BLOCKINPUT_INLINE):
	* buffer.h, buffer.c (BUFFER_INLINE):
	* category.h, category.c (CATEGORY_INLINE):
	* character.h, character.c (CHARACTER_INLINE):
	* charset.h, charset.c (CHARSET_INLINE):
	* composite.h, composite.c (COMPOSITE_INLINE):
	* dispextern.h, dispnew.c (DISPEXTERN_INLINE):
	* frame.h, frame.c (FRAME_INLINE):
	* intervals.h, intervals.c (INTERVALS_INLINE):
	* keyboard.h, keyboard.c (KEYBOARD_INLINE):
	* lisp.h, alloc.c (LISP_INLINE):
	* process.h, process.c (PROCESS_INLINE):
	* syntax.h, syntax.c (SYNTAX_INLINE):
	* systime.h, sysdep.c (SYSTIME_INLINE):
	* termhooks.h, terminal.h (TERMHOOKS_INLINE):
	* window.h, window.c (WINDOW_INLINE):
	Remove.  All uses replaced with INLINE.

1210 1211 1212 1213 1214
2013-09-20  Dmitry Antipov  <dmantipov@yandex.ru>

	* xterm.c (handle_one_xevent): Revert part of 2013-09-17 change
	to avoid Bug#15398.

1215 1216 1217 1218 1219 1220
2013-09-19  Eli Zaretskii  <eliz@gnu.org>

	* w32reg.c (w32_get_string_resource): Make the first 2 arguments
	'const char *' to avoid compiler warnings due to similar change in
	the prototype of x_get_string_resource.

1221 1222 1223 1224 1225 1226 1227 1228 1229 1230 1231 1232 1233 1234 1235 1236 1237 1238 1239 1240 1241 1242 1243 1244 1245 1246 1247
2013-09-19  Dmitry Antipov  <dmantipov@yandex.ru>

	* xterm.h (struct x_display_info): New members last_mouse_glyph_frame,
	last_mouse_scroll_bar, last_mouse_glyph and last_mouse_movement_time,
	going to replace static variables below.  Adjust comments.
	* xterm.c (last_mouse_glyph, last_mouse_glyph_frame)
	(last_mouse_scroll_bar, last_mouse_movement_time): Remove.
	(note_mouse_movement, XTmouse_position, x_scroll_bar_note_movement)
	(x_scroll_bar_report_motion, handle_one_xevent, syms_of_xterm):
	Related users changed.
	* w32term.h (struct w32_display_info): New members last_mouse_glyph_frame,
	last_mouse_scroll_bar, last_mouse_scroll_bar_pos, last_mouse_glyph and
	last_mouse_movement_time, going to replace static variables below.
	Adjust comments.
	* w32term.c (last_mouse_glyph_frame, last_mouse_scroll_bar)
	(last_mouse_scroll_bar_pos, last_mouse_glyph, last_mouse_movement_time):
	Remove.
	(note_mouse_movement, w32_mouse_position, w32_scroll_bar_handle_click)
	(x_scroll_bar_report_motion, syms_of_w32term): Related users changed.
	* nsterm.h (struct ns_display_info): New members last_mouse_glyph,
	last_mouse_movement_time and last_mouse_scroll_bar, going to replace
	static variables below.
	* nsterm.m (last_mouse_glyph, last_mouse_movement_time)
	(last_mouse_scroll_bar): Remove.
	(note_mouse_movement, ns_mouse_position, mouseMoved, mouseEntered)
	(mouseExited): Related users changed.

1248 1249 1250 1251 1252 1253 1254 1255
2013-09-19  Dmitry Antipov  <dmantipov@yandex.ru>

	Do not use external array to process X scroll bar messages.
	* xterm.c (scroll_bar_windows, scroll_bar_windows_size): Remove.
	(x_send_scroll_bar_event): Pack window pointer into two slots
	of XClientMessageEvent if we're 64-bit.  Adjust comment.
	(x_scroll_bar_to_input_event): Unpack accordingly.

1256 1257 1258 1259 1260 1261 1262 1263
2013-09-18  Dmitry Antipov  <dmantipov@yandex.ru>

	Ifdef away recent changes which aren't relevant to NS port.
	* dispextern.h (x_mouse_grabbed, x_redo_mouse_highlight)
	[!HAVE_NS]: Declare as such.
	* frame.c (x_mouse_grabbed, x_redo_mouse_highlight)
	[!HAVE_NS]: Define as such.

1264 1265 1266 1267 1268 1269 1270 1271 1272 1273 1274 1275 1276 1277 1278 1279 1280 1281 1282 1283 1284 1285 1286
2013-09-18  Dmitry Antipov  <dmantipov@yandex.ru>

	* frame.c (x_redo_mouse_highlight): New function
	to factor out common code used in W32 and X ports.
	* dispextern.h (x_redo_mouse_highlight): Add prototype.
	* xterm.h (struct x_display_info):
	* w32term.h (struct w32_display_info):
	* nsterm.h (struct ns_display_info): New members
	last_mouse_motion_frame, last_mouse_motion_x and
	last_mouse_motion_y, going to replace static variables below.
	* xterm.c (last_mouse_motion_event, last_mouse_motion_frame)
	(redo_mouse_highlight): Remove.
	(note_mouse_movement, syms_of_xterm): Adjust user.
	(handle_one_xevent): Likewise.  Use x_redo_mouse_highlight.
	* w32term.c (last_mouse_motion_event, last_mouse_motion_frame)
	(redo_mouse_highlight): Remove.
	(note_mouse_movement, syms_of_w32term): Adjust user.
	(w32_read_socket): Likewise.  Use x_redo_mouse_highlight.
	* nsterm.m (last_mouse_motion_position, last_mouse_motion_frame):
	Remove.
	(note_mouse_movement, mouseMoved, syms_of_nsterm):
	* nsfns.m (compute_tip_xy): Adjust user.

1287 1288 1289 1290 1291 1292 1293 1294 1295 1296 1297 1298 1299 1300 1301 1302
2013-09-18  Dmitry Antipov  <dmantipov@yandex.ru>

	* frame.c (x_mouse_grabbed): New function.
	* dispextern.h (x_mouse_grabbed): Add prototype.
	(last_mouse_frame): Remove declaration.
	* xterm.h (struct x_display_info):
	* w32term.h (struct w32_display_info):
	* nsterm.h (struct ns_display_info): New member
	last_mouse_frame, going to replace...
	* xdisp.c (last_mouse_frame): ...global variable.
	(note_tool_bar_highlight):
	* w32term.c (w32_mouse_position, w32_read_socket):
	* xterm.c (XTmouse_position, handle_one_xevent):
	Use x_mouse_grabbed.
	* nsterm.m (ns_mouse_position, mouseDown): Adjust user.

1303 1304 1305 1306 1307 1308
2013-09-17  Dmitry Antipov  <dmantipov@yandex.ru>

	* w32term.c (w32_read_socket): Avoid temporary
	variables in a call to x_real_positions.
	* xterm.c (handle_one_xevent): Likewise.

1309 1310 1311 1312 1313 1314 1315 1316 1317 1318
2013-09-17  Dmitry Antipov  <dmantipov@yandex.ru>

	* frame.h (x_set_bitmap_icon) [!HAVE_NS]: New function.
	(x_icon_type): Remove prototype.
	(x_bitmap_icon) [!HAVE_NS]: Declare as such.
	* frame.c (x_icon_type): Remove.
	* w32term.c (x_make_frame_visible, x_iconify_frame):
	* xterm.c (x_make_frame_visible, x_iconify_frame):
	Use x_set_bitmap_icon to factor out common code.

1319 1320 1321 1322 1323 1324 1325 1326 1327 1328 1329 1330
2013-09-17  Dmitry Antipov  <dmantipov@yandex.ru>

	* dispextern.h (check_x_display_info, x_get_string_resource):
	Declare here just once and unify the latter.
	* frame.c (check_x_display_info, x_get_string_resource):
	* nsterm.h (check_x_display_info):
	* xrdb.c (x_get_string_resource):
	* xterm.h (check_x_display_info): Remove prototypes.
	* nsfns.m (x_get_string_resource): Likewise.  Adjust definition.
	* w32reg.c (x_get_string_resource): Likewise.
	(w32_get_rdb_resource): Adjust user.

1331 1332 1333 1334 1335 1336 1337 1338 1339 1340 1341
2013-09-17  Dmitry Antipov  <dmantipov@yandex.ru>

	* xterm.h (struct x_display_info): New member
	x_pending_autoraise_frame, going to replace...
	* xterm.c (pending_autoraise_frame): ...static variable.
	(x_new_focus_frame, XTread_socket): Adjust users.
	* w32term.h (struct w32_display_info): New member
	w32_pending_autoraise_frame, going to replace...
	* w32term.c (pending_autoraise_frame): ...global variable.
	(x_new_focus_frame, w32_read_socket): Adjust users.

1342 1343 1344 1345 1346
2013-09-17  Glenn Morris  <rgm@gnu.org>

	* xdisp.c (message_dolog): If we create *Messages*,
	switch it to messages-buffer-mode.

1347 1348 1349 1350 1351 1352 1353 1354 1355 1356 1357 1358 1359 1360 1361 1362 1363 1364 1365 1366 1367
2013-09-17  Paul Eggert  <eggert@cs.ucla.edu>

	Don't overuse 'const' in types of locals.
	* bidi.c (bidi_count_bytes):
	* gtkutil.c, gtkutil.h (xg_event_is_for_menubar)
	(xg_event_is_for_scrollbar):
	* xselect.c (x_handle_property_notify)
	(x_handle_selection_notify, x_handle_dnd_message):
	* xsettings.c, xsettings.h (xft_settings_event):
	* xterm.c (x_handle_net_wm_state, handle_one_event)
	(x_menubar_window_to_frame, x_detect_focus_change)
	(construct_mouse_click, note_mouse_movement)
	(x_scroll_bar_to_input_event, x_scroll_bar_expose)
	(x_scroll_bar_handle_click, x_scroll_bar_note_movement)
	(handle_one_xevent, x_handle_net_wm_state):
	* xterm.h (x_handle_property_notify, x_handle_selection_notify)
	(x_handle_dnd_message):
	Avoid unnecessary 'const', typically the second 'const' in
	'const foo * const arg', a 'const' that does not affect the API
	and doesn't significantly help the human reader.

1368 1369 1370 1371 1372 1373 1374
2013-09-17  Dmitry Antipov  <dmantipov@yandex.ru>

	* image.c (fn_g_type_init) [WINDOWSNT]: Define and load
	only if Glib < 2.36.0.
	(fn_g_type_init) [!WINDOWSNT]: Define only if Glib < 2.36.0.
	* xsettings.c (init_gconf, init_gsettings): Do not check
	for g_type_init.
1375 1376
	* xterm.c (handle_one_xevent): Do not call to x_clear_area
	if GTK >= 2.7.0.
1377
	(toplevel) [USE_MOTIF]: Include xlwmenu.h to pacify GCC.
1378

Jan Djärv's avatar
Jan Djärv committed
1379 1380
2013-09-16  Jan Djärv  <jan.h.d@swipnet.se>

1381 1382 1383
	* xsettings.c (init_gconf, init_gsettings): Check for Glib 2.36.0
	before calling g_type_init.

Jan Djärv's avatar
Jan Djärv committed
1384 1385 1386 1387
	* font.c (syms_of_font): Move call to syms_of_(ns|mac)font ...

	* nsterm.m (syms_of_nsterm): ... to here.

1388 1389 1390 1391 1392 1393
2013-09-16  Dmitry Antipov  <dmantipov@yandex.ru>

	* xterm.c (toolkit_scroll_bar_interaction): Use bool for boolean.
	(ignore_next_mouse_click_timeout): Use Time as X does.
	(handle_one_xevent): Avoid cast and use unsigned comparison.

1394 1395 1396 1397 1398 1399 1400 1401 1402 1403 1404 1405 1406 1407 1408 1409 1410 1411 1412 1413 1414
2013-09-16  Dmitry Antipov  <dmantipov@yandex.ru>

	Do not copy X event in handle_one_xevent except KeyPress case.
	Wnen XEvent is processed, it is unlikely to be changed except
	KeyPress case, so we can avoid copying and use const pointer to
	const data to make sure that an event is not changed elsewhere.
	* xterm.c (handle_one_xevent): Change 2nd arg to 'const XEvent *
	const' and do not create local copy except for the KeyPress event.
	Use casts to avoid a few glitches.  Adjust formatting.  Add comments.
	(SET_SAVED_BUTTON_EVENT): Remove and move the code to the only user.
	(x_handle_net_wm_state, x_menubar_window_to_frame)
	(x_detect_focus_change, construct_mouse_click, note_mouse_movement)
	(x_scroll_bar_to_input_event, x_scroll_bar_expose)
	(x_scroll_bar_handle_click, x_scroll_bar_note_movement):
	* gtkutil.c (xg_event_is_for_menubar, xg_event_is_for_scrollbar):
	* xselect.c (x_handle_property_notify, x_handle_selection_notify)
	(x_handle_dnd_message):
	* xsettings.c (xft_settings_event):
	Use 'const XEvent * const' where appropriate.
	* xterm.h, gtkutil.h, xsettngs.h: Adjust related prototypes.

1415 1416 1417 1418 1419 1420 1421 1422 1423 1424
2013-09-16  Dmitry Antipov  <dmantipov@yandex.ru>

	Fix X event waiting to handle multiple frames.
	* frame.h (struct frame) [HAVE_X_WINDOWS]: New member wait_event_type.
	* xterm.c (pending_event_wait): Remove.  Adjust users.
	(x_detect_focus_change): Pass frame arg.
	(handle_one_xevent): Find related frame early and clear per-frame
	wait_event_type only if this is an event for the relevant frame.
	(x_wait_for_event): Use per-frame wait_event_type.

1425 1426
2013-09-15  Jan Djärv  <jan.h.d@swipnet.se>

1427 1428 1429 1430 1431 1432
	* nsfns.m (Fx_create_frame): Fix font driver registration for
	GNUStep.

	* font.c (syms_of_font): Check MAC_OS_X_VERSION_MAX_ALLOWED >= 1050
	for syms_of_macfont.

1433 1434 1435 1436 1437 1438 1439 1440 1441 1442 1443 1444 1445 1446 1447 1448 1449 1450 1451 1452 1453 1454 1455 1456 1457 1458 1459 1460 1461 1462
	* nsterm.m: Include macfont.h.
	(ns_tmp_flags, ns_tmp_font): Remove.
	(ns_compute_glyph_string_overhangs): Check for driver Qns.
	(ns_draw_glyph_string): Use local variables instead of ns_tmp_flags,
	ns_tmp_font.  Call ns_draw_text_decoration here instead of nsfont.m.
	(changeFont:): Fix code style.  Check for font driver type when
	getiing font.

	* nsterm.h (FONT_DESCENT, FONT_ASCENT): Define to (f)->ascent and
	(f)->descent.

	* nsfont.m (ns_tmp_flags, ns_tmp_font): Remove.
	(nsfont_open): Set font driver type.
	Set font->ascent and font->descent.  Figure out font instead of
	ns_tmp_font, and flags instead of ns_tmp_flags.
	Fix indentation. Remove call to ns_draw_text_decoration,
	moved to nsterm.

	* nsfns.m: Include macfont.h.
	(Fx_create_frame): Register macfont driver, make a better default font.
	(Fns_popup_font_panel): Get font from macfont driver, if used.

	* macfont.m, macfont.h, maccuvs.h: New files.

	* font.h: Declare syms_of_macfont.

	* font.c (syms_of_font): Call syms_of_macfont.

	* Makefile.in (NS_OBJ, SOME_MACHINE_OBJECTS): Add macfont.o.

1463 1464 1465 1466 1467 1468 1469 1470 1471 1472 1473 1474 1475 1476
2013-09-15  Dmitry Antipov  <dmantipov@yandex.ru>

	Drop VERTICAL_SCROLL_BAR_WIDTH_TRIM.  For X, it is zero since 1999,
	and it is always zero for others, so I assume that this is an ancient
	leftover which nobody will want to change any more.
	* xterm.h, w32term.h, nsterm.h (VERTICAL_SCROLL_BAR_WIDTH_TRIM): Remove.
	(VERTICAL_SCROLL_BAR_INSIDE_WIDTH):
	* frame.c (x_set_scroll_bar_width):
	* w32fns.c (w32_createscrollbar):
	* w32term.c (w32_set_vertical_scroll_bar):
	* xfns.c (x_set_scroll_bar_default_width):
	* xterm.c (XTflash, x_scroll_bar_create, XTset_vertical_scroll_bar)
	(x_scroll_bar_expose): Related users changed.

1477 1478 1479 1480
2013-09-15  Dmitry Antipov  <dmantipov@yandex.ru>

	* xterm.h (FRAME_X_SCREEN_NUMBER): Add comment.
	(BLACK_PIX_DEFAULT, WHITE_PIX_DEFAULT): Use FRAME_X_SCREEN_NUMBER.
1481 1482
	(SCROLL_BAR_X_WIDGET, SET_SCROLL_BAR_X_WIDGET) [USE_X_TOOLKIT]:
	Define as such.
1483 1484 1485
	* frame.h (FRAME_SMALLEST_CHAR_WIDTH, FRAME_SMALLEST_FONT_HEIGHT):
	Define once here...
	* nsterm.h, w32term.h, xterm.h: ...and not here.
1486 1487
	* w32term.h (SCROLL_BAR_X_WIDGET, SET_SCROLL_BAR_X_WIDGET):
	Remove unused Xisms.
1488 1489
	* xterm.c, xfns.c (toplevel): Remove #ifdef HAVE_X_WINDOWS because
	these modules are never compiled otherwise.
1490

1491 1492
2013-09-14  Eli Zaretskii  <eliz@gnu.org>

1493
	* buffer.c (syms_of_buffer) <left-margin-width, right-margin-width>:
1494 1495
	Doc fix.  (Bug#15375)

1496 1497 1498 1499 1500 1501 1502 1503 1504 1505 1506 1507 1508 1509 1510 1511 1512 1513 1514 1515
2013-09-13  Dmitry Antipov  <dmantipov@yandex.ru>

	Unify Fx_focus_frame between all ports.
	* frame.h (x_focus_frame): New prototype.
	* xfns.c (Fx_focus_frame): Remove.
	(syms_of_xfns): Do not defsubr it.
	(x_focus_frame): X implementation.
	* nsfns.m (Fx_focus_frame): Remove.
	(syms_of_nsfns): Do not defsubr it.
	(x_focus_frame): NS implementation.
	* w32term.c (Fx_focus_frame): Remove.
	(x_focus_on_frame): Rename to...
	(x_focus_frame): W32 implementation.
	* w32term.h (x_focus_on_frame): Remove prototype.
	* w32fns.c (Fx_focus_frame): Remove.
	(syms_of_w32fns): Do not defsubr it.
	* frame.c (Fx_focus_frame): Define here.
	(syms_of_frame): Defsubr here.
	* gtkutil.c (xg_tool_bar_callback): Use x_focus_frame.

1516 1517 1518 1519 1520 1521 1522 1523 1524 1525 1526 1527 1528 1529 1530 1531 1532
2013-09-13  Dmitry Antipov  <dmantipov@yandex.ru>

	Unify FRAME_window_system_DISPLAY_INFO macros between all ports.
	All of them are replaced with FRAME_DISPLAY_INFO, defined in
	each port to reference the port-specific window system data.
	* msdos.h (FRAME_X_DISPLAY_INFO): Remove.
	(FRAME_DISPLAY_INFO): Define.
	* w32term.h (FRAME_W32_DISPLAY_INFO, FRAME_X_DISPLAY_INFO): Remove.
	(FRAME_DISPLAY_INFO): Define.  Adjust users.
	* xterm.h (FRAME_X_DISPLAY_INFO): Remove.
	(FRAME_DISPLAY_INFO): Define.  Adjust users.
	* frame.h (FRAME_RES_X, FRAME_RES_Y): Unify.
	* font.c, frame.c, gtkutil.c, image.c, menu.c, msdos.c, nsfns.m:
	* nsfont.m, nsterm.m, w32fns.c, w32font.c, w32menu.c, w32term.c:
	* w32xfns.c, widget.c, xdisp.c, xfaces.c, xfns.c, xfont.c, xmenu.c:
	* xselect.c, xterm.c: All related users changed.

1533 1534 1535 1536 1537 1538 1539 1540 1541 1542 1543
2013-09-13  Dmitry Antipov  <dmantipov@yandex.ru>

	* xterm.h (x_window_to_frame, x_any_window_to_frame)
	(x_menubar_window_to_frame): Remove prototypes.
	* xfns.c (x_window_to_frame, x_any_window_to_frame)
	(x_menubar_window_to_frame, x_top_window_to_frame):
	Move from here...
	* xterm.c (x_window_to_frame, x_any_window_to_frame)
	(x_menubar_window_to_frame, x_top_window_to_frame):
	...to here and convert all but the last to static.

1544 1545 1546 1547
2013-09-12  Eli Zaretskii  <eliz@gnu.org>

	* lisp.mk (lisp): Add w32-common-fns.elc.

1548 1549 1550 1551
2013-09-12  Xue Fuqiao  <xfq.free@gmail.com>

	* charset.c (char_charset): Document an exception for char-charset.

1552 1553 1554 1555 1556 1557
2013-09-12  Dmitry Antipov  <dmantipov@yandex.ru>

	* xterm.h (x_display_info): New field last_user_time...
	* xterm.c (toplevel): ...to replace static last_user_time.
	(handle_one_xevent, x_ewmh_activate_frame): Adjust users.

1558 1559 1560 1561 1562
2013-09-12  Dmitry Antipov  <dmantipov@yandex.ru>

	* xterm.c (x_set_scroll_bar_thumb) [USE_LUCID && !HAVE_XAW3D]: Clip
	scroll bar values to prevent thumb from disappear and update comment.

1563 1564 1565 1566
2013-09-11  Glenn Morris  <rgm@gnu.org>

	* emacs.c (usage_message): Possessive apostrophe tweak.

1567 1568
2013-09-11  Dmitry Antipov  <dmantipov@yandex.ru>

1569 1570 1571 1572 1573 1574
	* nsterm.m (syms_of_nsterm): Use Qns.
	* w32fns.c (Fx_open_connection): Remove old '#if 0' code.
	* w32term.c (w32_create_terminal, syms_of_w32term): Use Qw32.
	* xfns.c (x_display_info_for_name, Fx_open_connection):
	Remove old '#if 0' code.
	(syms_of_xfns): Use Qx.
1575 1576 1577
	* termhooks.h (fullscreen_hook): Remove the leftover.
	(struct terminal): Fix typo in comment.

1578 1579 1580 1581 1582 1583 1584 1585 1586 1587 1588 1589 1590 1591 1592 1593
2013-09-11  Dmitry Antipov  <dmantipov@yandex.ru>

	Cleaning up a few X scroll bar bits.
	* termhooks.h (enum scroll_bar_part): Add scroll_bar_nowhere member.
	* xterm.h (struct scroll_bar) [USE_TOOLKIT_SCROLL_BARS && USE_LUCID]:
	New member last_seen_part, going to replace...
	* xterm.c [USE_TOOLKIT_SCROLL_BARS]: ...global last_scroll_bar_part.
	(xt_action_hook) [USE_LUCID]: Adjust user.
	(xm_scroll_callback, xg_scroll_callback): Do not bloat with
	Lucid-specific scroll bar support.
	(xaw_jump_callback, xaw_scroll_callback): Prefer enum scroll_par_part
	to int and adjust to use last_seen_part member.
	(x_set_toolkit_scroll_bar_thumb) [USE_LUCID]: Adjust user.
	(x_scroll_bar_create) [USE_TOOLKIT_SCROLL_BARS && USE_LUCID]:
	Initialize last_seen_part.

1594 1595 1596 1597 1598
2013-09-11  Stefan Monnier  <monnier@iro.umontreal.ca>

	* insdel.c (insert_from_buffer_1): Don't mark buffer as modified when
	insert-buffer-substring an empty string.

1599 1600 1601 1602 1603
2013-09-11  Paul Eggert  <eggert@cs.ucla.edu>

	* xdisp.c (Ftool_bar_lines_needed): Declare as 'const' if ifdeffed out,
	avoiding a GCC warning.

1604 1605 1606 1607 1608 1609 1610 1611 1612 1613 1614 1615 1616 1617 1618 1619 1620 1621 1622 1623 1624 1625 1626
2013-09-11  Dmitry Antipov  <dmantipov@yandex.ru>

	Ifdef away frame tool bar code when it is not really used.
	* frame.h (struct frame) [HAVE_WINDOW_SYSTEM && !USE_GTK && !HAVE_NS]:
	Move tool_bar_window, desired_tool_bar_string, current_tool_bar_string
	and minimize_tool_bar_window_p under the above.
	(fset_current_tool_bar_string, fset_desired_tool_bar_string)
	(fset_tool_bar_window): Likewise.
	* dispnew.c (clear_current_matrices, clear_desired_matrices)
	(adjust_frame_glyphs_for_window_redisplay, free_glyphs, update_frame)
	(change_frame_size_1):
	* window.c (window_from_coordinates, Frecenter): Adjust users.
	* window.h (WINDOW_TOOL_BAR_P): Define to zero when frame tool bar
	code is not really used.
	* xdisp.c (build_desired_tool_bar_string, display_tool_bar_line)
	(tool_bar_lines_needed, MAX_FRAME_TOOL_BAR_HEIGHT, tool_bar_item_info)
	(get_tool_bar_item, handle_tool_bar_click, note_tool_bar_highlight)
	[!USE_GTK && !HAVE_NS]: Define as such.
	(Ftool_bar_lines_needed, redisplay_tool_bar, show_mouse_face)
	(note_mouse_highlight, expose_frame):
	* xfns.c (x_set_tool_bar_lines):
	* xterm.c (handle_one_xevent): Adjust users.

1627 1628 1629 1630 1631 1632 1633 1634 1635 1636 1637
2013-09-11  Paul Eggert  <eggert@cs.ucla.edu>

	Fix corruption with multiple emacsclient -t instances (Bug#15222).
	This bug was introduced by my 2013-08-26 patch, which incorrectly
	assumed that the terminfo implementation doesn't use termcap buffers.
	* term.c (init_tty) [TERMINFO]: Remove optimization, as
	these buffers apparently are used after all.
	* termchar.h (TERMCAP_BUFFER_SIZE) [TERMINFO]: Define here too.
	(struct tty_display_info): Define members termcap_term_buffer and
	termcap_strings_buffer even if TERMINFO.

Dmitry Antipov's avatar
Dmitry Antipov committed
1638 1639 1640 1641 1642 1643 1644 1645