ChangeLog 615 KB
Newer Older
1 2 3 4 5
2014-12-13  Paul Eggert  <eggert@cs.ucla.edu>

	* alloc.c (XMALLOC_BASE_ALIGNMENT): Use max_align_t instead of
	a hack involving a union.

6
2014-12-13  Eli Zaretskii  <eliz@gnu.org>
Ted Zlatanov's avatar
Ted Zlatanov committed
7 8 9 10 11

	* gnutls.c (Fgnutls_available_p, syms_of_gnutls): Move
	gnutls-available-p out of the HAVE_GNUTLS conditional, and define
	them only once.

12
2014-12-13  Teodor Zlatanov  <tzz@lifelogs.com>
Ted Zlatanov's avatar
Ted Zlatanov committed
13 14 15 16 17 18 19 20 21 22

	* emacs.c (main): Always include gnutls.h and run syms_of_gnutls.

	* gnutls.h: Always declare syms_of_gnutls.

	* gnutls.c (Fgnutls_available_p syms_of_gnutls): Move later for
	clarity.  Let the availability check return Qnil when the GnuTLS
	integration is not available, instead of erroring out.
	(Bug#19346)

23
2014-12-13  Eli Zaretskii  <eliz@gnu.org>
Ted Zlatanov's avatar
Ted Zlatanov committed
24 25 26 27 28

	* xdisp.c (move_it_in_display_line_to, display_line): Don't
	disallow overflow-newline-into-fringe when word-wrap is in
	effect.  (Bug#19300)

Stefan Monnier's avatar
Stefan Monnier committed
29 30 31 32 33 34 35
2014-12-11  Stefan Monnier  <monnier@iro.umontreal.ca>

	* fileio.c: Better preserve window-points during revert (bug#19161).
	(Qget_buffer_window_list): New var.
	(get_window_points_and_markers, restore_window_points): New functions.
	(Finsert_file_contents): Use them to save and restore window-points.

36 37 38 39 40 41
2014-12-11  Dmitry Antipov  <dmantipov@yandex.ru>

	* xterm.c (x_delete_terminal): Call emacs_close for X connection
	descriptor if called from x_connection_closed and always delete
	this descriptor from keyboard waiting set (Bug#19147).

42 43 44 45 46 47 48 49 50
2014-12-10  Eli Zaretskii  <eliz@gnu.org>

	* bidi.c (BIDI_CACHE_MAX_ELTS_PER_SLOT): New macro.
	(bidi_cache_max_elts): New global variable.
	(bidi_shelve_header_size): Add the sizeof bidi_cache_max_elts.
	(bidi_cache_shrink, bidi_initialize): Reset bidi_cache_max_elts to
	its initial value.
	(bidi_cache_search): Handle overflown cache.  Improve commentary.
	(bidi_cache_ensure_space): Limit allocations to the current value
Stefan Monnier's avatar
Stefan Monnier committed
51 52
	of bidi_cache_max_elts.  Force xpalloc not to over-allocate.
	If less than a full BIDI_CACHE_CHUNK is left to the limit, decrease
53 54 55 56 57
	the increment to not exceed the limit.
	(bidi_cache_iterator_state): Now returns non-zero if succeeded to
	cache, zero otherwise (meaning the cache overflowed).  In the
	latter case, set bidi_cache_last_idx to -1.
	(bidi_peek_at_next_level): Handle overflown cache.
Stefan Monnier's avatar
Stefan Monnier committed
58
	(bidi_push_it): Increase the cache limit for iterating the new object.
59 60 61 62 63 64 65 66 67 68 69 70
	(bidi_pop_it): Decrease the cache limit back to previous value.
	(bidi_shelve_cache): Shelve the current value of the cache limit.
	(bidi_unshelve_cache): Restore the value of cache limit.
	(bidi_find_bracket_pairs): If the cache overflows while looking
	for the paired bracket, give up and let bidi_resolve_neutrals
	process the bracket as a simple neutral.  (Bug#19322)
	(bidi_find_other_level_edge): If the cache overflows, fall back on
	Plan B, which effectively stops the reordering and restarts it on
	the next character (after resetting the cache).
	(bidi_move_to_visually_next): When the cache overflows, reset it
	after processing the last cached character.

71 72 73 74 75 76 77 78 79 80 81 82 83 84
2014-12-10  Paul Eggert  <eggert@cs.ucla.edu>

	Fix glitches in gnutls.c, mostly memory-related
	* gnutls.c: Sort macro definitions by name.
	(fn_gnutls_transport_set_errno): Omit unreachable definition.
	(fn_gnutls_x509_crt_get_signature): Omit unused symbol.
	(gnutls_hex_string): Arg is now unsigned char *, to avoid a cast.
	Prefer ptrdiff_t for sizes.  Check for arithmetic overflow when
	calculating string length.  Use make_uninit_string, to avoid
	copying the string.  Cast the char, not the pointer.
	(gnutls_certificate_details): Use xmalloc and xfree, not malloc
	and free.  Work even for dates past the year 9999.  Use void *
	for buffers, to avoid casts.

85 86 87 88
2014-12-09  Andy Moreton  <andrewjmoreton@gmail.com>  (tiny change)

	* gnutls.c (gnutls_protocol_get_name): Fix a copy/paste typo.

89 90 91 92 93
2014-12-08  Lars Magne Ingebrigtsen  <larsi@gnus.org>

	* gnutls.c (Fgnutls_peer_status): Return the key exchange, cipher
	and MAC algorithms.

94 95 96 97
2014-12-08  Stefan Monnier  <monnier@iro.umontreal.ca>

	* process.c: Whitespace and line-break nitpicks.

98 99 100 101
2014-12-08  Lars Magne Ingebrigtsen  <larsi@gnus.org>

	* gnutls.c (gnutls_certificate_details): The :signature isn't
	that useful, so remove it.
102
	(gnutls_certificate_details): Clean up whitespace slightly.
103 104
	(Fgnutls_peer_status): Return the Diffie-Hellman prime bits for
	the connection.
105 106
	(gnutls_certificate_details): Put :signature-algorithm back again,
	which was removed by mistake.
107 108
	(emacs_gnutls_handle_error): Fatal errors should be on level 1, so
	that they are not messaged by default (bug#16253).
109
	(Fgnutls_error_fatalp): Doc fix (bug#18210).
110

111 112
	* gnutls.c: Add Windows specs for gnutls_dh_get_prime_bits.

Jan D's avatar
Jan D committed
113 114 115 116 117
2014-12-07  Jan Djärv  <jan.h.d@swipnet.se>

	* nsimage.m (setPixmapData): Make bmRep local so class member is not
	set (Bug#19133).

Stefan Monnier's avatar
Stefan Monnier committed
118
2014-12-05  Lee Duhem  <lee.duhem@gmail.com>  (tiny change)
119 120 121 122

	* eval.c (Fsignal): Remove duplicate test.
	(Fautoload_do_load): Fix up docstring.

Stefan Monnier's avatar
Stefan Monnier committed
123
2014-12-05  Jan Djärv  <jan.h.d@swipnet.se>
124 125 126 127 128 129 130 131 132 133 134

	* nsterm.m (represented_filename, represented_frame): New variables.
	(ns_set_represented_filename): New function.
	(sendEvent:): Set represented filename here to avoid flicker,
	related to Bug#18757.

	* nsterm.h: Declare ns_set_represented_filename.

	* nsfns.m (ns_set_name_as_filename): Don't set represented filename
	at once, call ns_set_represented_filename instead.

135 136 137 138 139 140 141 142 143 144 145 146 147 148 149 150 151 152
2014-12-05  Eli Zaretskii  <eliz@gnu.org>

	* dispextern.h (enum bidi_dir_t): Force NEUTRAL_DIR to be zero.
	(struct bidi_stack): Reduce size by using bit fields and by
	packing sos, override, and isolate_status into a single 8-bit
	byte called 'flags'.

	* bidi.c (ISOLATE_STATUS, OVERRIDE): New macros.
	(bidi_push_embedding_level): Construct flags from individual
	bits.  Adapt to changes in prev_for_neutral and next_for_neutral
	members.
	(bidi_pop_embedding_level): Use ISOLATE_STATUS.  Extract 'sos'
	from flags.  Adapt to changes in prev_for_neutral,
	next_for_neutral, and last_strong members.
	(bidi_line_init): Initialize flags to zero.
	(bidi_resolve_explicit, bidi_resolve_weak, bidi_resolve_brackets)
	(bidi_resolve_neutral): Use ISOLATE_STATUS and OVERRIDE.

153 154 155 156 157
2014-12-04  Stefan Monnier  <monnier@iro.umontreal.ca>

	* eval.c (backtrace_eval_unrewind): Rewind also the excursions.
	(Fapply): Try and simplify the control flow.

Chris Zheng's avatar
Chris Zheng committed
158 159 160 161 162 163 164
2014-12-03  Chris Zheng  <chriszheng99@gmail.com>  (tiny change)

	* gnutls.c (init_gnutls_functions, gnutls_certificate_details):
	Use gnutls_sign_get_name directly because
	gnutls_sign_algorithm_get_name is a macro on MinGW (bug#19231).
	(init_gnutls_functions): Load missing gnutls_server_name_set.

165 166 167 168 169 170 171 172 173
2014-12-02  Eli Zaretskii  <eliz@gnu.org>

	* bidi.c (bidi_find_first_overridden): New function.

	* xdisp.c (Fbidi_find_overridden_directionality): New function.
	(syms_of_xdisp): Defsubr it.

	* dispextern.h (bidi_find_first_overridden): Add prototype.

Jan Djärv's avatar
Jan Djärv committed
174 175 176 177 178
2014-12-02  Jan Djärv  <jan.h.d@swipnet.se>

	* nsimage.m (initFromSkipXBM:width:height:flip:length:): Set bmRep
	to nil after release (Bug#9133).

Paul Eggert's avatar
Paul Eggert committed
179 180 181 182 183 184 185 186 187 188 189 190 191 192 193 194 195 196
2014-11-30  Paul Eggert  <eggert@cs.ucla.edu>

	Port better to AddressSanitizer.
	These changes suffice for temacs on x86-64 with GCC 4.9.2 and
	-fsanitize=address.
	* alloc.c (valid_pointer_p) [ADDRESS_SANITIZER]:
	Return -1 or 0, as the pipe trick doesn't work.
	* alloc.c (relocatable_string_data_p, mark_object, sweep_symbols):
	* data.c (Ffset):
	* print.c (print_object):
	When a pointer-check primitive returns -1, do not assume this
	means the pointer is valid or that the underlying system has failed.
	It could just be that addresses are being sanitized so Emacs can't
	test for pointer validity.
	* lisp.h (defined_GC_CHECK_STRING_BYTES): New constant.
	(USE_STACK_STRING) [GC_CHECK_STRING_BYTES]: Now false, since the
	string validity checker doesn't work on stack-based strings.

197 198 199 200 201 202 203 204 205
2014-11-29  Paul Eggert  <eggert@cs.ucla.edu>

	Improve clarity of USE_LSB_TAG definition.
	Problem reported by Lee Duhem.  Suggestion by Andreas Schwab in:
	http://lists.gnu.org/archive/html/emacs-devel/2014-11/msg02222.html
	* lisp.h (USE_LSB_TAG): Define in terms of the (simpler)
	VAL_MAX / 2 rather than in terms of the (more complicated)
	EMACS_INT_MAX >> GCTYPEBITS, and adjust commentary to match.

Paul Eggert's avatar
Paul Eggert committed
206
2014-11-29  Eli Zaretskii  <eliz@gnu.org>
207 208 209 210 211 212

	* xdisp.c (handle_single_display_spec): When ignoring a fringe
	bitmap display, reset the ignore_overlay_strings_at_pos_p flag.
	(next_element_from_buffer): When done with overlays, reset the
	ignore_overlay_strings_at_pos_p flag.  (Bug#19201)

Paul Eggert's avatar
Paul Eggert committed
213
2014-11-29  Eli Zaretskii  <eliz@gnu.org>
214 215 216 217

	* w32fns.c (Fw32_shell_execute): Convert forward slashes in
	DOCUMENT to backslashes, if DOCUMENT is a file.  (Bug#19141)

Paul Eggert's avatar
Paul Eggert committed
218
2014-11-29  Ulrich Müller  <ulm@gentoo.org>
219 220 221 222 223

	* Makefile.in (temacs$(EXEEXT)): Use consistent flag settings
	for paxctl and setfattr. Fixes temacs startup failure with
	grsecurity/PaX enabled Linux kernel (bug#19067).

224 225
2014-11-27  Oscar Fuentes  <ofv@wanadoo.es>

226 227
	* src/w32.c, src/w32heap.c, src/w32term.c:
	Use MINGW_W64 instead of _W64.
228

229
2014-11-27  Stefan Monnier  <monnier@iro.umontreal.ca>
230 231 232

	* frame.c (Fhandle_switch_frame): Deactivate shift-region (bug#19003).

233 234 235 236 237
2014-11-26  Lars Magne Ingebrigtsen  <larsi@gnus.org>

	* gnutls.c (gnutls_ip_address_p): New function.
	(Fgnutls_boot): Only send SNI if the host name is not an IP address.

238 239 240 241
2014-11-26  Toke Høiland-Jørgensen  <toke@toke.dk>  (tiny change)

	* gnutls.c (Fgnutls_boot): Send the server name over (bug#18208).

242 243 244 245 246 247
2014-11-25  Paul Eggert  <eggert@cs.ucla.edu>

	Fix gnutls problems caught by static checking.
	* gnutls.c (Fgnutls_boot): Remove unused local.
	(Fgnutls_boot): Fix pointer signedness.

248 249 250 251 252
2014-11-25  Lars Magne Ingebrigtsen  <larsi@gnus.org>

	* gnutls.c (gnutls_certificate_details): Don't include certain
	certificate details on older gnutls versions (bug#19168).
	(syms_of_gnutls): Export gnutls_peer_status_warning_describe.
253
	(gnutls_certificate_details): Reversed check in last checkin.
254

255 256 257
2014-11-25  Teodor Zlatanov  <tzz@lifelogs.com>

	* gnutls.c (Fgnutls_peer_status): Check GNUTLS_INITSTAGE, not gnutls_p.
258 259 260
	(Fgnutls_peer_status_warning_describe): Add function to describe a
	peer verification warning symbol.
	(Fgnutls_peer_status): Use it.
261
	(Fgnutls_boot): Use it.  Minor logging fixes.
262

263 264 265 266
2014-11-24  Lars Magne Ingebrigtsen  <larsi@gnus.org>

	* gnutls.c: Fix compilation warnings given fix --enable-gcc-warnings.

267 268
2014-11-24  Glenn Morris  <rgm@gnu.org>

269 270
	* emacs.c (main) <--version>: Make use of PACKAGE_NAME.

271 272 273
	* Makefile.in (emacs.res): Use ../nt rule, rather than duplicating it.
	(ntsource, WINDRES, EMACS_MANIFEST): Remove, now unused.

274 275
2014-11-23  Lars Magne Ingebrigtsen  <larsi@gnus.org>

276
	* gnutls.c: Fix Windows function definition of
277 278
	gnutls_x509_crt_get_fingerprint.

279 280
	* gnutls.c: Fix Windows return value for gnutls_pk_algorithm_get_name.

281 282 283 284 285 286 287 288 289
	* gnutls.c (Fgnutls_boot): Save certificate for later inspection.

	* process.h: Added more fields to Lisp_Process to track
	certificate details.

	* gnutls.c (gnutls_hex_string, gnutls_certificate_details)
	(Fgnutls_peer_status): New functions to export TLS certificate
	details to Emacs Lisp.

Jan D's avatar
Jan D committed
290 291 292 293 294
2014-11-23  Jan Djärv  <jan.h.d@swipnet.se>

	* gtkutil.c (gtk_adjustment_configure): Define for Gtk+ < 2.14.
	(xg_set_toolkit_horizontal_scroll_bar_thumb): Remove wrong version
	check for gtk_adjustment_configure (Bug#19149).
Jan D's avatar
Jan D committed
295
	(gtk_adjustment_configure): Don't use 2.14 functions (Bug#19149).
Jan D's avatar
Jan D committed
296

297 298 299 300
2014-11-22  Glenn Morris  <rgm@gnu.org>

	* Makefile.in (emacs.res): Add dependency on icons/hand.cur.  Use $<.

301 302 303 304 305 306 307 308 309 310 311 312
2014-11-22  Martin Rudalics  <rudalics@gmx.at>

	Don't shrink frame when using desktop-save-mode (Bug#19048).
	* w32fns.c (x_set_tool_bar_lines): Always call
	x_change_tool_bar_height (Bug#19048).
	(x_change_tool_bar_height): Mark frame as garbaged since
	adjust_frame_size might not have done that.
	* xfns.c (x_set_tool_bar_lines): Always call
	x_change_tool_bar_height (Bug#19048).
	(x_change_tool_bar_height): Mark frame as garbaged since
	adjust_frame_size might not have done that.

313 314 315 316
2014-11-21  Ulf Jasper  <ulf.jasper@web.de>

	* xml.c (parse_region): Take care of new optional parameter
	'discard-comments' of 'libxml-parse(html|xml)-region'.
Stefan Monnier's avatar
Stefan Monnier committed
317 318
	(Flibxml_parse_html_region, Flibxml_parse_xml_region):
	New optional parameter 'discard-comments'.
319

320 321 322 323 324 325 326 327 328 329 330 331 332 333 334 335 336 337 338 339 340 341 342 343 344 345 346 347
2014-11-17  Paul Eggert  <eggert@cs.ucla.edu>

	Improve time stamp handling, and be more consistent about it.
	* editfns.c (invalid_time): New function.
	Use it instead of 'error ("Invalid time specification")'.
	(time_add, time_subtract, time_arith, Ftime_add, Ftime_less_p)
	(decode_float_time, lisp_to_timespec, lisp_time_struct):
	New functions.
	(make_time_tail, make_time): Remove.  All uses changed to use
	new functions or plain list4i.
	(disassemble_lisp_time): Return effective length if successful.
	Check that LOW is an integer, if it's combined with other components.
	(decode_time_components): Decode into struct lisp_time, not
	struct timespec, so that we can support a wide set of times
	regardless of whether time_t is signed.  Decode plain numbers
	as seconds since the Epoch, and nil as the current time.
	(lisp_time_argument, lisp_seconds_argument, Ffloat_time):
	Reimplement in terms of new functions.
	(Fencode_time): Just use list2i.
	(syms_of_editfns): Add time-add, time-subtract, time-less-p.
	* keyboard.c (decode_timer): Don't allow the new formats (floating
	point or nil) in timers.
	* systime.h (LO_TIME_BITS): New constant.  Use it everywhere in
	place of the magic number '16'.
	(struct lisp_time): New type.
	(decode_time_components): Use it.
	(lisp_to_timespec): New decl.

348 349 350 351 352 353 354 355 356 357 358 359 360 361 362 363 364 365 366
2014-11-16  Lars Magne Ingebrigtsen  <larsi@gnus.org>

	* intervals.h (INTERVAL_WRITABLE_P): Check the `inhibit-read-only'
	text property.

	* callint.c (Fcall_interactively): Pass in nil as argument to
	Fbarf_if_buffer_read_only.

	* fileio.c (Finsert_file_contents): Ditto.

	* insdel.c (prepare_to_modify_buffer_1): Pass start region in.

	* textprop.c (verify_interval_modification): Check buffer
	readedness after the last interval.

	* buffer.c (Fbarf_if_buffer_read_only): Don't raise an error if
	the text at POSITION (new optional argument) has the
	`inhibit-read-only' text property set.

Stefan Monnier's avatar
Stefan Monnier committed
367
2014-11-16  Eli Zaretskii  <eliz@gnu.org>
368 369

	* window.c (window_scroll_pixel_based): Avoid truncation/rounding
Stefan Monnier's avatar
Stefan Monnier committed
370 371
	errors in computing the number of pixels to scroll.
	Suggested by Kelly Dean <kelly@prtime.org>.  (Bug#19060)
372

Stefan Monnier's avatar
Stefan Monnier committed
373
2014-11-16  Jan Djärv  <jan.h.d@swipnet.se>
Jan D's avatar
Jan D committed
374

375 376 377
	* nsmenu.m (update_frame_tool_bar): If tool bar changes height,
	call updateFrameSize.

Jan D's avatar
Jan D committed
378 379
	* nsterm.m (setFrame:): Remove call to display (Bug#18757).

Stefan Monnier's avatar
Stefan Monnier committed
380
2014-11-16  Jan Djärv  <jan.h.d@swipnet.se>
Jan Djärv's avatar
Jan Djärv committed
381 382 383 384 385

	* nsfns.m (x_set_foreground_color, x_set_background_color)
	(x_set_cursor_color, Fxw_color_values): Block/unblock input,
	use SET_FRAME_GARBAGED instead of redraw_frame (Bug#19036).

386 387 388 389 390 391
2014-11-15  Jan Djärv  <jan.h.d@swipnet.se>

	* nsterm.m (ns_send_appdefined): Check for application defined
	event on Cocoa (Bug#18993).
	(run): Restore code before the previous 18993 fix.

David Reitter's avatar
David Reitter committed
392 393
2014-11-14  David Reitter  <david.reitter@gmail.com>

Stefan Monnier's avatar
Stefan Monnier committed
394
	* nsterm.m (run): Set timeout for event loop to prevent hang.
David Reitter's avatar
David Reitter committed
395 396
	(Bug#18993)

397 398 399 400
2014-11-14  Paul Eggert  <eggert@cs.ucla.edu>

	* .gitignore: Add emacs-[1-9]*, to ignore files like emacs-25.0.50.1.

401 402 403 404 405 406
2014-11-14  Dmitry Antipov  <dmantipov@yandex.ru>

	* xfont.c (xfont_list_pattern): Do not allocate font entity object
	for each candidate font but attempt to reuse it from previous improper
	candidate, if any.

407 408 409 410 411 412 413
2014-11-13  Paul Eggert  <eggert@cs.ucla.edu>

	Avoid undefined behavior in color table hashing.
	* image.c (CT_HASH_RGB) [COLOR_TABLE_SUPPORT]: Remove, replacing with ...
	(ct_hash_rgb) [COLOR_TABLE_SUPPORT]: New function.  All uses changed.
	This function avoids undefined behavior with signed shift overflow.

414
2014-11-10  Eli Zaretskii  <eliz@gnu.org>
415

416 417 418
	* fileio.c (Finsert_file_contents): Invalidate buffer caches also
	when the inserted text does not need decoding.  (Bug#18982)

419
2014-11-10  Jan Djärv  <jan.h.d@swipnet.se>
420 421 422 423 424 425 426 427

	* nsterm.h (EmacsScroller): judge returns bool.

	* nsterm.m (ns_set_vertical_scroll_bar): Release bar.
	(ns_judge_scroll_bars): Only set removed if judge returns true.
	(judge): Returns bool == condemned.  Remove self from window.
	(setPosition:portion:whole:): Remove raise SIGIO (Bug#18757).

428 429 430 431 432
2014-11-10  Stefan Monnier  <monnier@iro.umontreal.ca>

	* keyboard.c (command_loop_1): Record this-command in recent-keys.
	(Frecent_keys): Rewrite. and add optional `include-cmds' arg.

Jan Djärv's avatar
Jan Djärv committed
433 434 435 436 437
2014-11-09  Jan Djärv  <jan.h.d@swipnet.se>

	* nsterm.m (ns_set_vertical_scroll_bar)
	(ns_set_horizontal_scroll_bar): Cleanup merge error.

438 439
2014-11-09  Paul Eggert  <eggert@cs.ucla.edu>

440 441 442 443
	* xgselect.c (xg_select): Don't assume n_gfds is nonnegative
	merely because tmo_in_millisec is nonnegative.  The 1st call
	to g_main_context_query could succeed while the 2nd one fails.

444 445
	* frame.c (Fcan_run_window_configuration_change_hook): Return a value.

446 447
2014-11-08  Jan Djärv  <jan.h.d@swipnet.se>

448
	* nsterm.m (run): Only use non-system event loop if OSX version is
449
	exactly 10.9 (Bug#18993).
450 451
	(ns_set_vertical_scroll_bar): Don't call bar setPosition: unless
	needed (Bug#18757).
452

453
2014-11-08  Michael Albinus  <michael.albinus@gmx.de>
454 455 456 457

	* callproc.c (encode_current_directory): Support handling of file
	names prepended by "/:".  (Bug#18891)

458
2014-11-08  Alan Mackenzie  <acm@muc.de>
459 460 461

	* syntax.c (back_comment): Fix off-by-one error (bug#18022).

462
2014-11-08  Dima Kogan  <dima@secretsauce.net>
463 464 465

	* xgselect.c (xg_select): Use g_main_context_acquire (bug#18861).

466
2014-11-08  Michael Albinus  <michael.albinus@gmx.de>
467 468 469 470

	* dired.c (Ffile_attributes): Return Qnil, if Fexpand_file_name
	raises an error.  (Bug#18891)

471 472 473 474 475 476 477 478 479 480
2014-11-08  Martin Rudalics  <rudalics@gmx.at>

	* frame.c (adjust_frame_size): Call x_set_window_size only if
	f->can_x_set_window_size is set.
	(make_frame): Initialize f->can_x_set_window_size and
	f->can_run_window_configuration_change_hook.
	(Fcan_run_window_configuration_change_hook): New function.
	* frame.h (frame): Split `official' into `can_x_set_window_size'
	and `can_run_window_configuration_change_hook'.
	* nsfns.m (Fx_create_frame): Set f->can_x_set_window_size.
Stefan Monnier's avatar
Stefan Monnier committed
481 482 483 484
	* w32fns.c (Fx_create_frame, x_create_tip_frame):
	Set f->can_x_set_window_size.
	* window.c (run_window_configuration_change_hook):
	Return immediately if either f->can_x_set_window_size or
485 486 487
	f->can_run_window_configuration_change_hook are false.
	(Fset_window_configuration): Instead of f->official set
	f->can_x_set_window_size.
Stefan Monnier's avatar
Stefan Monnier committed
488 489
	* xfns.c (Fx_create_frame, x_create_tip_frame):
	Set f->can_x_set_window_size.
490

491 492 493 494 495
2014-11-08  Jan Djärv  <jan.h.d@swipnet.se>

	* nsterm.m (EmacsScroller.dealloc): Reinstate, removed at merge
	cleanup from 2014-11-01 (Bug#18972).

496 497 498 499 500
2014-11-07  Stefan Monnier  <monnier@iro.umontreal.ca>

	* keyboard.c: Call gui-set-selection instead of x-set-selection.
	* xdisp.c (window-scroll-functions): Improve docstring.

Paul Eggert's avatar
Paul Eggert committed
501 502 503 504 505 506 507 508 509
2014-11-07  Paul Eggert  <eggert@cs.ucla.edu>

	Uniquify the 'size' symbol.
	* frame.c (Qsize):
	* w32notify.c (Qsize): Remove.
	* lisp.h (Qsize): New decl.
	* lread.c (Qsize): Now extern.
	* w32notify.c (syms_of_w32notify): No need to defsym.

510 511 512 513 514 515 516 517 518 519 520 521 522 523
2014-11-07  Martin Rudalics  <rudalics@gmx.at>

	* dispnew.c (change_frame_size_1): Fix call of
	adjust_frame_size.
	* frame.c (Qsize, Qframe_position, Qframe_outer_size)
	(Qframe_inner_size, Qexternal_border_size, Qtitle_height)
	(Qmenu_bar_external, Qmenu_bar_size, Qtool_bar_external)
	(Qtool_bar_size): New constants.
	(frame_inhibit_resize, adjust_frame_size): New argument to
	handle case where frame_inhibit_implied_resize is a list.
	(Fmake_terminal_frame, Fset_frame_height, Fset_frame_width)
	(Fset_frame_size, x_set_left_fringe, x_set_right_fringe)
	(x_set_right_divider_width, x_set_bottom_divider_width)
	(x_set_vertical_scroll_bars, x_set_horizontal_scroll_bars)
524 525
	(x_set_scroll_bar_width, x_set_scroll_bar_height):
	Update callers.
526
	(frame-inhibit-implied-resize): Rewrite doc-string.
527 528
	* frame.h (frame_inhibit_resize, adjust_frame_size):
	Fix external declarations.
529 530 531 532 533 534 535 536 537 538
	(Qframe_position, Qframe_outer_size)
	(Qframe_inner_size, Qexternal_border_size, Qtitle_height)
	(Qmenu_bar_external, Qmenu_bar_size, Qtool_bar_external)
	(Qtool_bar_size): Extern them.
	* gtkutil.c (FRAME_TOTAL_PIXEL_HEIGHT, FRAME_TOTAL_PIXEL_WIDTH)
	(xg_height_or_width_changed): Remove.
	(xg_frame_set_char_size): Adjust adjust_frame_size calls.
	(menubar_map_cb, xg_update_frame_menubar, free_frame_menubar)
	(tb_size_cb, update_frame_tool_bar, free_frame_tool_bar)
	(xg_change_toolbar_position): Call adjust_frame_size directly.
539 540
	* nsfns.m (x_set_internal_border_width, Fx_create_frame):
	Fix calls of adjust_frame_size.
541
	* w32fns.c (x_set_internal_border_width, x_set_menu_bar_lines)
542
	(Fx_create_frame, x_create_tip_frame): Adjust adjust_frame_size calls.
543 544 545 546 547 548 549 550
	(x_set_tool_bar_lines, x_change_tool_bar_height): Make sure that
	frame can get resized when tool-bar-lines parameter changes from
	or to zero.
	(Fw32_frame_menu_bar_size): Return fourth value.
	(Fw32_frame_rect): Block input around system calls
	(Fx_frame_geometry): New function.
	* w32menu.c (set_frame_menubar): Adjust adjust_frame_size call.
	* w32term.c (x_new_font): Adjust adjust_frame_size call.
551 552
	* widget.c (EmacsFrameSetCharSize): Adjust frame_inhibit_resize call.
	* window.c (Fset_window_configuration): Adjust adjust_frame_size call.
553 554 555 556 557 558 559 560 561 562 563 564 565
	* xfns.c (x_set_menu_bar_lines, x_set_internal_border_width)
	(Fx_create_frame): Adjust adjust_frame_size calls.
	(x_set_tool_bar_lines, x_change_tool_bar_height): Make sure that
	frame can get resized when tool-bar-lines parameter changes from
	or to zero.
	(Fx_frame_geometry): New function.
	* xmenu.c (update_frame_menubar): On Lucid call
	adjust_frame_size with one pixel less height to avoid that
	repeatedly adding/removing the menu bar grows the frame.
	(free_frame_menubar): On Motif arrange to optionally preserve
	the old frame height when removing the menu bar.
	* xterm.c (x_new_font): Adjust adjust_frame_size call.

566 567 568 569 570 571
2014-11-03  Eli Zaretskii  <eliz@gnu.org>

	* xdisp.c (Fdump_glyph_matrix, Fdump_frame_glyph_matrix): Doc fix.
	(Fdump_frame_glyph_matrix): Don't segfault if invoked on a GUI
	frame; instead, print an error message.

572 573 574 575
2014-11-03  Jan Djärv  <jan.h.d@swipnet.se>

	* nsfns.m (ns_set_doc_edited): Check for FRAME_NS (Bug#18925).

576 577
2014-11-02  Jan Djärv  <jan.h.d@swipnet.se>

Jan Djärv's avatar
Jan Djärv committed
578 579 580
	* nsimage.m (allocInitFromFile:): Initialize bmRep.
	(dealloc): Release bmRep.

581 582 583 584 585 586 587 588 589
	* nsterm.h (EmacsImage): Remove imageListNext, refCount, reference,
	imageListSetNext, imageListNext.

	* nsimage.m (ImageList, imageListNext, imageListSetNext:)
	(reference): Remove.
	(allocInitFromFile:): Remove searching ImageList and calling
	reference (Bug#18918).
	(dealloc): Remove handling if ImageList.

Andreas Schwab's avatar
Andreas Schwab committed
590 591 592 593
2014-11-02  Andreas Schwab  <schwab@linux-m68k.org>

	* process.c (catch_child_signal): Fix incorrect assertion.

594
2014-11-01  Jan Djärv  <jan.h.d@swipnet.se>
595

Jan Djärv's avatar
Jan Djärv committed
596 597 598
	* macfont.m (macfont_draw):
	* nsterm.m (ns_mouse_position, judge): Clean up merge conflict.

599
	* macfont.m (macfont_glyph_extents): Turn off synthetic bold
600
	if force_integral_p (i.e. no antialias).
601
	(macfont_draw): Check ns_antialias_text, also turn off synthetic
602 603
	bold if no antialias (Bug#18876).

604 605 606
	* emacs.c (main): Don't chdir to $HOME on Cocoa if --chdir
	was given (Bug#18846).

607 608 609 610 611 612 613 614
	* nsterm.h (ns_set_doc_edited): Declare taking no args.

	* nsfns.m (ns_set_doc_edited): Do all logic (check frames) here
	instead of in xdisp.c, function now takes void. (Bug#18884).

	* xdisp.c (prepare_menu_bars): Remove HAVE_NS code.
	(redisplay_internal): Call ns_set_doc_edited if HAVE_NS (Bug#18884).

615 616 617 618 619 620 621 622 623 624 625 626
	* nsterm.h (EmacsScroller): Replace Lisp_Object win with
	struct window* (Bug#18889).
	Remove getMouseMotionPart.
	(ns_output): Make icon_top/left int.

	* nsfns.m (x_icon): icon_top/left is int.

	* nsterm.m (ns_mouse_position): Remove unused code.
	(initFrame:window:, dealloc): Use window instead of win.
	(getMouseMotionPart:window:x:y:): Remove, unused.
	(sendScrollEventAtLoc:fromEvent:): Make Lisp_Object win from window.

627 628 629 630 631 632 633
2014-11-01  Eli Zaretskii  <eliz@gnu.org>

	* keyboard.c (readable_events): When FLAGS include
	READABLE_EVENTS_FILTER_EVENTS, ignore BUFFER_SWITCH_EVENT events.
	This avoids returning non-nil from input-pending-p when only such
	events are in the queue.  (Bug#18856)

634 635
2014-11-01  Jan Djärv  <jan.h.d@swipnet.se>

636 637 638 639 640 641
	* nsselect.m (QCLIPBOARD, QSECONDARY, QTEXT, QFILE_NAME)
	(NXPrimaryPboard, NXSecondaryPboard): Declare static.
	(Qforeign_selection): Remove.
	(ns_get_local_selection): Identation fix.
	(syms_of_nsselect): Remove Qforeign_selection, ns-lost-selection-hooks

642 643 644 645
	* nsselect.m (ns_get_local_selection): Remove calling of
	functions in Vselection_converter_alist (Bug#18911).
	(syms_of_nsselect): Remove Vselection_converter_alist.

646
2014-10-31  Dmitry Antipov  <dmantipov@yandex.ru>
647 648 649 650

	* font.c (copy_font_spec): Redesign to avoid Fcopy_alist
	and unnecessary initialization.  Adjust comments.

651 652 653 654 655 656 657
2014-10-30  Eli Zaretskii  <eliz@gnu.org>

	* bidi.c (bidi_cache_reset_to): Invalidate bidi_cache_last_idx by
	setting it to -1.
	(bidi_find_bracket_pairs): Pass to bidi_cache_reset_to a relative
	index, not an absolute one, as that's what the function expects.

658 659 660 661 662
2014-10-30  Dmitry Antipov  <dmantipov@yandex.ru>

	* xfaces.c (Finternal_set_lisp_face_attribute): Don't try to realize
	font on an initial frame when running as a daemon (Bug#18869).

663 664
	* fontset.c (toplevel): Adjust comment to match 2014-06-19 change.

665
2014-10-30  Eli Zaretskii  <eliz@gnu.org>
666 667 668 669

	* fileio.c (Fexpand_file_name): Use make_unibyte_string, not
	build_string, when importing a home directory.  (Bug#18873)

670 671
	* dispnew.c (buffer_posn_from_coords):
	Use WINDOW_WANTS_HEADER_LINE_P, not WINDOW_WANTS_MODELINE_P, to
672 673
	account for the header-line height.  (Bug#18839)

674 675 676 677
2014-10-28  Ulf Jasper  <ulf.jasper@web.de>

	* xml.c (parse_region): Do not forget the first document child.

678 679 680 681 682 683 684 685
2014-10-25  Jan Djärv  <jan.h.d@swipnet.se>

	* nsselect.m: pasteboard_changecount is new.
	(ns_store_pb_change_count, ns_get_pb_change_count)
	(ns_get_our_change_count_for): New functions.
	(ns_string_to_pasteboard_internal): Correct comment.
	type => gtype in eassert, Call ns_store_pb_change_count.
	(Fns_own_selection_internal): Remove data, use value (Bug#18799).
686 687
	(Fns_disown_selection_internal, Fns_selection_owner_p):
	Replace Vselection_alist check, with change count check.
688 689 690 691
	(Fns_get_selection): Initialize val to Qnil.  Only get local
	selection if change counts match (Bug#18799).
	(nxatoms_of_nsselect): Initialize pasteboard_changecount.

692 693 694 695 696
2014-10-25  Noam Postavsky  <npostavs@users.sourceforget.net>

	* src/w32proc.c (create_child): If calling a quoted batch file,
	pass NULL for exe.  (Bug#18745)

697 698 699 700 701 702
2014-10-24  Eli Zaretskii  <eliz@gnu.org>

	* bidi.c (bidi_resolve_explicit, bidi_find_bracket_pairs)
	(bidi_resolve_brackets): Use end of string position rather than ZV
	when iterating over a string.  (Bug#18815)

703 704 705 706 707 708 709
2014-10-24  Martin Rudalics  <rudalics@gmx.at>

	* keyboard.c (make_lispy_position): Return coordinates also when
	on scroll bars, fringes, margins or not in a window.
	* xdisp.c (show_mouse_face): Don't change cursor face during
	mouse tracking.

710 711 712
2014-10-23  Martin Rudalics  <rudalics@gmx.at>

	* frame.c (Fset_frame_height, Fset_frame_width, Fset_frame_size)
713 714
	(frame_resize_pixelwise, frame_inhibit_implied_resize):
	Fix doc-strings (Bug#18789).
715

716 717 718 719
2014-10-23  Paul Eggert  <eggert@cs.ucla.edu>

	* Makefile.in (ACLOCAL_INPUTS): Omit unnecessary use of 'wildcard'.

720 721 722 723 724 725 726 727 728 729 730 731 732 733 734 735 736 737 738 739 740
2014-10-22  Eli Zaretskii  <eliz@gnu.org>

	Optimize redisplay of simple bracketed text.
	* bidi.c (bidi_cache_reset_to): New function.
	(bidi_cache_reset): Call it.
	(bidi_init_it, bidi_line_init): Initialize the bracket_pairing_pos
	member to -1.
	(bidi_resolve_explicit): Reset bracket_pairing_pos and
	bracket_enclosed_type only if bracket_pairing_pos's value is not
	ZV.
	(MAX_BPA_STACK): Make sure the value is signed.
	(PUSH_BPA_STACK): If the BPA stack overflows, don't bail out, but
	stop pushing values onto the stack.
	(bidi_find_bracket_pairs): If the bracketed text is only on the
	base embedding level, remove all the states cached by this
	function from the cache, and return zero, so that the brackets in
	this segment of text are processed as normal neutrals.
	(bidi_resolve_brackets): Detect the brackets that are to be
	processed as neutrals, and don't call bidi_find_bracket_pairs on
	them.  (Bug#18778)

741 742 743 744 745 746 747 748 749 750 751 752 753 754 755 756 757 758
2014-10-21  Stefan Monnier  <monnier@iro.umontreal.ca>

	* w32select.c (Fw32_selection_exists_p): Rename from
	Fx_selection_exists_p.
	(syms_of_w32select): Adjust accordingly.

	* w16select.c (Fw16_selection_exists_p): Rename from
	Fx_selection_exists_p.
	(syms_of_win16select): Adjust accordingly.

	* nsselect.m (ns_get_local_selection): Signal error rather than `quit'.
	(Fns_own_selection_internal): Tighten scoping.
	(Fns_selection_exists_p): Rename from Fx_selection_exists_p.
	(Fns_get_selection): Rename from Fx_get_selection_internal.
	(Fns_get_selection_internal, Fns_store_selection_internal):
	Remove functions.
	(syms_of_nsselect): Adjust accordingly.

759 760 761 762 763 764 765
2014-10-21  Martin Rudalics  <rudalics@gmx.at>

	* w32fns.c (Fw32_frame_menu_bar_size): New function.
	* w32term.c (x_set_window_size): Account for wrapped menu bar
	lines when setting up frame height (Bug#15174 and Bug#18720).
	(w32_add_wrapped_menu_bar_lines): New variable.

Stefan Monnier's avatar
Stefan Monnier committed
766
2014-10-21  Stefan Monnier  <monnier@iro.umontreal.ca>
767 768 769 770

	* xdisp.c (redisplay_window): Re-run pre-redisplay-function after we
	move point.

771 772
2014-10-20  Glenn Morris  <rgm@gnu.org>

773 774
	* Merge in all changes up to 24.4 release.

775 776
2014-10-19  Jan Djärv  <jan.h.d@swipnet.se>

777 778 779
	* gtkutil.c (xg_update_menubar, xg_update_menu_item): Only call
	g_object_notify for label if Gtk+ >= 2.16 (Bug#16522).

780 781 782 783 784 785
	* xterm.h (x_output): Remove net_wm_state_hidden_seen.

	* xterm.c (handle_one_xevent): Check return value from
	x_handle_net_wm_state, remove net_wm_state_hidden_seen (Bug#18722).
	(get_current_wm_state): Remove net_wm_state_hidden_seen setting.

786 787 788 789 790 791 792
	* gtkutil.c (create_dialog): Don't use gtk_dialog_get_action_area on
	Gt+ >= 3.12, or gtk_misc_set_alignment on Gtk+ >= 3.14 (Bug#18674).
	(make_widget_for_menu_item): Don't use gtk_misc_set_alignment on
	Gtk+ >= 3.14 (Bug#18674).
	(update_frame_tool_bar): Don't use  gtk_misc_set_padding on
	Gtk+ >= 3.14 (Bug#18674).

793 794 795 796 797 798 799 800 801
2014-10-18  Paul Eggert  <eggert@cs.ucla.edu>

	* gtkutil.c: Remove no-longer-used code.
	(xg_update_menubar): Remove unused local and unnecessary call to
	gtk_menu_item_get_submenu.
	(XG_TOOL_BAR_PROXY_BUTTON, xg_tool_bar_proxy_callback)
	(xg_get_tool_bar_widgets, xg_tool_bar_proxy_help_callback)
	(TOOLBAR_TOP_WIDGET): Remove; no longer used.

802 803
2014-10-18  Jan Djärv  <jan.h.d@swipnet.se>

804 805
	* xfns.c (syms_of_xfns): Remove x-gtk-whole-detached-tool-bar.

806 807 808 809 810 811 812 813 814 815
	* xterm.h (handlebox_widget): Remove.

	* xmenu.c (set_frame_menubar): Remove GTK block that calls
	xg_have_tear_offs.

	* gtkutil.h (xg_have_tear_offs): Remove declaration.

	* gtkutil.c (XG_TEXT_CANCEL, XG_TEXT_OPEN, XG_TEXT_OK): New defines
	to handle Gtk versions.
	(xg_get_file_with_chooser): Use them.
Stefan Monnier's avatar
Stefan Monnier committed
816 817
	(xg_have_tear_offs, tearoff_remove, tearoff_activate):
	Remove (create_menus): Remove teroff argument and code.
818
	Remove call to gtk_menu_set_title.
Stefan Monnier's avatar
Stefan Monnier committed
819 820
	(xg_update_menubar, xg_update_submenu): Remove tearoff code.
	Adjust args to create_menus.
821 822 823 824 825 826 827 828 829 830
	(xg_tool_bar_menu_proxy, xg_tool_bar_detach_callback)
	(xg_tool_bar_attach_callback, TOOLBAR_TOP_WIDGET): Remove.
	(xg_pack_tool_bar): Replace TOOLBAR_TOP_WIDGET, remove detach code.
	(xg_make_tool_item): Remove detach code.
	(xg_update_tool_bar_sizes): Replace TOOLBAR_TOP_WIDGET.
	(find_icon_from_name): New function.
	(update_frame_tool_bar): Remove GtkStockItem code, move to
	find_icon_from_name.  Let stock be a list of icon names to try.
	Only use gtk_image_new_from_stock on Gtk+ < 3.10.
	Replace TOOLBAR_TOP_WIDGET.
Stefan Monnier's avatar
Stefan Monnier committed
831 832
	(free_frame_tool_bar, xg_change_toolbar_position ):
	Replace TOOLBAR_TOP_WIDGET.
833 834
	(xg_initialize): Remove tearoff code.

835 836
2014-10-18  Eli Zaretskii  <eliz@gnu.org>

837 838 839 840 841 842 843
	* xterm.c (x_draw_bar_cursor, x_draw_hollow_cursor): Subtract 1
	pixel from the window's cursor glyph width, since X renders hollow
	blocks 1 pixel wider than the 'width' parameter.

	* xdisp.c (get_phys_cursor_geometry): Don't subtract 1 pixel from
	glyph->pixel_width; this is now done in xterm.c.

844 845 846 847 848 849 850 851 852 853 854 855 856 857 858 859 860 861 862 863 864 865
	Fix reordering of bracket characters in isolates.
	* bidi.c (bidi_cache_find): Rename the argument NEUTRALS_OK to
	RESOLVED_ONLY; when non-zero, return from the cache only fully
	resolved states.  All callers changed.
	(CANONICAL_EQU): New macro.
	(PUSH_BPA_STACK): Use it to push onto the BPA stack the canonical
	equivalent of the paired closing bracket character.
	(bidi_find_bracket_pairs): Set the bracket_pairing_pos member to
	the default non-negative value, to be checked later in
	bidi_resolve_brackets.  Use CANONICAL_EQU to test candidate
	characters against those pushed onto the BPA stack.
	(bidi_record_type_for_neutral): New function.
	(bidi_resolve_brackets): Record next_for_neutral and
	prev_for_neutral when embedding level gets pushed.  Force
	resolution of bracket pairs when entering a level run that was not
	yet BPA-resolved.
	(bidi_resolve_neutral): Add assertions before calling
	bidi_resolve_neutral_1.
	(bidi_level_of_next_char): Remove the code that attempted to
	resolve unresolved neutrals; that is now done by
	bidi_resolve_neutral.

866 867
	* w32select.c (owner_callback): Mark with ALIGN_STACK attribute.

868 869 870 871 872 873 874
2014-10-17  Eli Zaretskii  <eliz@gnu.org>

	* xterm.c (x_draw_hollow_cursor): Fix display of hollow cursor on
	1-pixel R2L characters.
	Reported by Dmitry Antipov <dmantipov@yandex.ru>, see
	http://lists.gnu.org/archive/html/emacs-devel/2014-10/msg00518.html.

875 876 877 878 879
2014-10-16  Eli Zaretskii  <eliz@gnu.org>

	* bidi.c (bidi_find_bracket_pairs): Avoid a loop that does nothing
	useful.

880 881 882 883 884 885 886
2014-10-15  Paul Eggert  <eggert@cs.ucla.edu>

	* bidi.c (bidi_find_bracket_pairs): Initialize local var.
	This pacifies GCC 4.9.1 with --enable-gcc-warnings.
	It's not clear to me whether the initialization is needed,
	but it can't hurt so I played it safe.

887 888 889 890
2014-10-15  Stefan Monnier  <monnier@iro.umontreal.ca>

	* lisp.mk (lisp): Add emacs-lisp/eldoc.elc.

891 892 893
2014-10-15  Eli Zaretskii  <eliz@gnu.org>

	Update the bidirectional reordering engine for Unicode 6.3 and 7.0.
894 895 896
	* bidi.c (bidi_ignore_explicit_marks_for_paragraph_level):
	Remove variable.
	(bidi_get_type): Return the isolate initiators and terminator types.
897 898 899
	(bidi_isolate_fmt_char, bidi_paired_bracket_type)
	(bidi_fetch_char_skip_isolates, find_first_strong_char)
	(bidi_find_bracket_pairs, bidi_resolve_brackets): New functions.
900
	(bidi_set_sos_type): Rename from bidi_set_sor_type and updated
901 902 903
	for the new features.
	(bidi_push_embedding_level, bidi_pop_embedding_level): Update to
	push and pop correctly for isolates.
904
	(bidi_remember_char): Modify to accept an additional argument
905 906 907
	and record the bidi type according to its value.
	(bidi_cache_iterator_state): Accept an additional argument to only
	update an existing state.  Handle the new members of struct bidi_it.
Eli Zaretskii's avatar
Eli Zaretskii committed
908
	(bidi_cache_find): Arguments changed: no longer accepts a level,
909 910 911 912 913 914 915 916 917 918 919 920 921 922 923 924 925 926 927 928 929 930 931 932 933 934 935 936 937 938 939 940 941 942 943 944 945 946 947 948 949 950 951 952 953 954 955 956
	instead accepts a flag telling it whether it is okay to return
	unresolved neutrals.
	(bidi_initialize): Initiate and staticpro the bracket-type uniprop
	table.  Initialize new isolate-related members.
	(bidi_paragraph_init): Some code factored out into
	find_first_strong_char.
	(bidi_resolve_explicit_1): Function deleted, its code incorporated
	into bidi_resolve_explicit.
	(bidi_resolve_explicit): Support the isolate initiators and
	terminator.  Fix handling of embeddings and overrides according to
	new UBA requirements.  Record information about previously seen
	characters here (moved from bidi_level_of_next_char).
	(bidi_resolve_weak): Adapt to changes in struct members.
	(FLAG_EMBEDDING_INSIDE, FLAG_OPPOSITE_INSIDE, MAX_BPA_STACK)
	(STORE_BRACKET_CHARPOS, PUSH_BPA_STACK): New macros.
	(bidi_resolve_neutral): Call bidi_resolve_brackets to handle the
	paired bracket resolution.  Handle isolate initiators and
	terminator.
	(bidi_type_of_next_char): Remove unneeded code for BN limit.
	(bidi_level_of_next_char): Move the code that records information
	about previous characters to bidi_resolve_explicit.  Fix logic of
	resolving neutrals and make sure their cache entries are updated.
	Remove now unneeded special handling of PDF level.

	* dispextern.h (struct glyph): Enlarge the width of resolved_level.
	(BIDI_MAXDEPTH): New macro, renamed from BIDI_MAXLEVEL and
	enlarged per Unicode 6.3.
	(enum bidi_bracket_type_t): New data type.
	(struct bidi_saved_info): Leave only 2 type members out of 4.
	Remove bytepos.
	(struct bidi_stack): Add members necessary to support isolating
	sequences.
	(struct bidi_it): Add new members necessary to support isolating
	sequences and bracket pair resolution.

	* xdisp.c (Fbidi_resolved_levels): New function.
	(syms_of_xdisp): Defsubr it.
	(append_glyph, append_composite_glyph, produce_image_glyph)
	(append_stretch_glyph, append_glyphless_glyph): Convert aborts to
	assertions.
	(syms_of_xdisp) <inhibit-bidi-mirroring>: New variable.

	* term.c (append_glyph, append_composite_glyph)
	(append_glyphless_glyph): Convert aborts to assertions.

	* .gdbinit (pgx): Display the character codepoint, resolved level,
	and bidi type also for glyphless glyphs.

957 958 959 960 961 962 963 964 965
2014-10-15  Dmitry Antipov  <dmantipov@yandex.ru>

	Avoid unwanted point motion in Fline_beginning_position.
	* lisp.h (scan_newline_from_point): Add prototype.
	* search.c (scan_newline_from_point): New function, refactored from...
	* cmds.c (Fforward_line): ...adjusted user.
	* editfns.c (Fline_beginning_position): Use scan_newline_from_point
	and simplify the former since the latter doesn't move point.

966 967 968 969 970 971 972 973 974 975 976 977 978 979 980 981 982 983 984 985 986 987 988 989 990 991 992 993 994 995 996 997 998
2014-10-14  Dmitry Antipov  <dmantipov@yandex.ru>

	Cleanup terminal handling code.
	* dispextern.h (get_named_tty): Remove prototype but...
	* termhooks.h (get_named_terminal): ...resurrect it under
	more meaningful name.
	(get_terminal): Likewise, but with...
	(decode_live_terminal): ...this name.
	(decode_tty_terminal): Add prototype.
	* term.c (get_tty_terminal): Remove.
	(get_named_tty): Remove.
	(Ftty_display_color_p, Ftty_display_color_cells, Ftty_type)
	(Fcontrolling_tty_p, Fsuspend_tty, Fresume_tty):
	Use decode_tty_terminal.
	(Ftty_no_underline, Ftty_top_frame): Use decode_live_terminal.
	* terminal.c (get_terminal): Refactor to...
	(decode_terminal, decode_live_terminal): ...new functions.
	(decode_tty_terminal): Replacement for get_tty_terminal.
	(get_named_terminal): Likewise for get_named_tty.
	* coding.c (Fset_terminal_coding_system_internal)
	(Fterminal_coding_system, Fset_keyboard_coding_system_internal):
	(Fkeyboard_coding_system):
	* composite.c (Fcomposition_get_gstring):
	* dispnew.c (Fsend_string_to_terminal):
	* frame.c (Fmake_terminal_frame):
	* nsfns.m (check_ns_display_info):
	* w32fns.c, xfns.c (check_x_display_info):
	* xselect.c (frame_for_x_selection): Use decode_live_terminal.
	* keyboard.c (handle_interrupt_signal, handle_interrupt)
	(Fset_quit_char): Use get_named_terminal.
	(Fset_output_flow_control, Fset_input_meta_mode):
	Use decode_tty_terminal.

999 1000 1001 1002 1003
2014-10-13  Eli Zaretskii  <eliz@gnu.org>

	* w32term.h (ALIGN_STACK): Use _WIN64, not _W64, to distinguish
	between 32-bit and 64-bit MinGW builds.  (Bug#18699)

1004
2014-10-12  Paul Eggert  <eggert@cs.ucla.edu>
1005 1006 1007 1008

	Fix port to Debian GNU/kFreeBSD 7 (wheezy) (Bug#18666).
	* process.c (accept4) [!HAVE_ACCEPT4]: New macro.

1009
2014-10-12  Stefan Monnier  <monnier@iro.umontreal.ca>
1010 1011 1012 1013

	* frame.c (Fmouse_pixel_position): Call Vmouse_position_function
	(bug#18638).

1014 1015
2014-10-12  Paul Eggert  <eggert@cs.ucla.edu>

1016 1017 1018
	* editfns.c (dump_tz_string): No longer const.
	It might be modified.

1019 1020
	* nsmenu.m (clear): Assume OS X 10.6 or later.

1021 1022 1023 1024 1025 1026 1027 1028 1029 1030 1031
2014-10-12  Jan Djärv  <jan.h.d@swipnet.se>

	* unexmacosx.c: Remove include ppc/reloc.h.
	(unrelocate, copy_dysymtab): Remove PPC code.
	(rebase_reloc_address): Remove, only used for PPC:

	* nsterm.m: Always include macfont.h on COCOA.
	(ns_update_auto_hide_menu_bar, ns_draw_fringe_bitmap)
	(ns_dumpglyphs_image, ns_check_menu_open)
	(applicationDidFinishLaunching)
	(antialiasThresholdDidChange:)
1032 1033
	(keyDown:, toggleFullScreen:, setPosition:portion:whole:):
	Remove checks for OSX <= 10.5/10.6.
1034 1035 1036 1037 1038 1039 1040 1041 1042 1043 1044 1045 1046 1047 1048 1049 1050 1051 1052 1053 1054 1055 1056 1057
	(changeFont:): Use macfont on COCOA, nsfont on GNUSTEP.
	(syms_of_nsterm): Call syms_of_macfont on COCOA, syms_of_nsfont on
	GNUSTEP.

	* nsterm.h (MAC_OS_X_VERSION_10_4, MAC_OS_X_VERSION_10_5): Remove.
	(NS_HAVE_NSINTEGER): Remove block.
	Remove >= OSX 10.6 tests.

	* nsmenu.m (NSMenuDidBeginTrackingNotification): Remove.
	(x_activate_menubar, trackingNotification:): Remove check for
	OSX >= 10.5.
	(menuNeedsUpdate:): Remove check for OSX < 10.5.

	* nsimage.m (allocInitFromFile:): Remove code for OSX < 10.6.

	* nsfns.m: Always include macfont.h on COCOA.
	(ns_filename_from_panel, ns_directory_from_panel)
	(Fx_create_frame, Fns_popup_font_panel, ns_run_file_dialog)
	(Fns_read_file_name, Fns_list_services): Remove code for OSX < 10.6

	* macfont.m: Remove >= 1050 check.
	(macfont_create_family_with_symbol)
	(macfont_get_glyph_for_character)
	(mac_font_get_glyphs_for_variants)
1058 1059
	(mac_ctfont_create_available_families, syms_of_macfont):
	Remove code for OSX < 10.6.
1060 1061 1062 1063 1064 1065 1066 1067 1068
	(mac_font_family_group, mac_font_family_compare): Remove, only used
	for OSX < 10.6.

	* macfont.h (MAC_FONT_FORMAT_ATTRIBUTE, MAC_FONT_FORMAT_BITMAP)
	(mac_font_copy_non_synthetic_table): Remove versions for OSX < 10.6

	* Makefile.in: Replace nsfont.o macfont.o with ns_fontfile in
	comment.

1069 1070 1071 1072 1073 1074 1075 1076 1077 1078 1079 1080 1081 1082 1083 1084 1085 1086 1087 1088 1089 1090 1091 1092 1093 1094 1095 1096 1097 1098 1099 1100 1101 1102 1103 1104 1105 1106
2014-10-12  Paul Eggert  <eggert@cs.ucla.edu>

	Fix putenv race conditions with undefined behavior (Bug#8705).
	Do all putenv calls before Emacs creates any threads.
	Use a safer way to modify the TZ environment variable in the
	presence of multiple threads.  For further thread-safety,
	prefer localtime_r and gmtime_r to localtime and gmtime,
	and prefer struct tm's tm_gmtoff (if available) to calling
	both localtime_r and gmtime_r.
	* dbusbind.c (Fdbus__init_bus): Move xputenv call from here ...
	(init_dbusbind): ... to this new function.
	* emacs.c (main) [HAVE_DBUS]: Call it before creating threads.
	* xterm.c (x_term_init): Move xputenv call from here ...
	(init_xterm): ... to this new function.
	* emacs.c (main) [USE_GTK]: Call it before creating threads.
	* editfns.c (HAVE_TM_GMTOFF): Default to false.
	(dump_tz_string): New constant.
	(init_editfns): Use it.  This centralizes the dump_tz stuff.
	Call set_time_zone_rule here, so that its xputenv is done
	before Emacs goes multithreaded.
	(mktime_z) [!HAVE_TZALLOC]: New function, which is typically
	thread-safe enough for Emacs.
	(format_time_string, Fdecode_time, Fcurrent_time_string)
	(Fcurrent_time_zone):
	Prefer localtime_r and gmtime_r, which are more thread-safe, to
	localtime and gmtime.  Remove now-unnecessary calls to block_input.
	(tm_gmtoff): New static function.
	(Fdecode_time, Fcurrent_time_zone): Use it.
	(Fencode_time): Use mktime_z, for better thread-safety.
	(set_time_zone_rule): Now static.  Rewrite to be mostly thread-safe,
	i.e., not quite thread-safe but good enough for Emacs typical usage.
	Do not reclaim storage that is in the environment; let it leak.
	Always call tzset, since localtime_r does not.
	* emacs.c (dump_tz, Fdump_emacs) [HAVE_TZSET]: Remove dump_tz stuff.
	This is now done in init_editfns.
	* systime.h (mktime_z, timezone_t, tzalloc, tzfree) [!HAVE_TZALLOC]:
	New macros and declarations, for platforms lacking tzalloc & friends.

1107 1108 1109 1110 1111 1112 1113 1114 1115
2014-10-09  Paul Eggert  <eggert@cs.ucla.edu>

	* lisp.h (USE_STACK_STRING): Now true only if USE_STACK CONS.
	On x86 platforms this works around GCC bug 63495
	<https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63495>,
	and more generally should fix a portability problem in Emacs.
	Problem reported by Stefan Monnier in:
	http://lists.gnu.org/archive/html/emacs-devel/2014-10/msg00261.html

1116 1117 1118 1119 1120 1121 1122 1123 1124 1125 1126
2014-10-08  Leo Liu  <sdl.web@gmail.com>

	Enhance terpri to allow conditionally output a newline.  (Bug#18652)
	* keymap.c (describe_vector_princ):
	* keyboard.c (Fcommand_error_default_function): Adapt to change to
	Fterpri.

	* print.c (printchar_stdout_last): Declare.
	(printchar): Record the last char written to stdout.
	(Fterpri): Add optional argument ENSURE.

1127 1128
2014-10-08  Eli Zaretskii  <eliz@gnu.org>

1129 1130 1131 1132 1133 1134 1135
	* w32inevt.c (maybe_generate_resize_event): Pass non-zero as the
	DELAY argument to change_frame_size, so that the frame size
	changes, if any are needed, are delayed until the next redisplay.
	This is to avoid a too early QUIT inside change_frame_size, when
	it calls Lisp in frame_windows_min_size, in case one of the events
	we've read sets the quit-flag.  (Bug#18649)

1136 1137 1138
	* w32fns.c (check_x_display_info): Accept terminal objects as
	argument, to follow what xfns.c does.

1139 1140 1141 1142 1143
2014-10-07  Glenn Morris  <rgm@gnu.org>

	* Makefile.in ($(srcdir)/macuvs.h)
	($(lispsource)/international/charprop.el): Add explicit FORCE.

1144 1145 1146 1147 1148 1149
2014-10-07  Eli Zaretskii  <eliz@gnu.org>

	* decompress.c (init_zlib_functions): Move the message about zlib
	being unavailable from here...
	(Fzlib_decompress_region): ...to here.  (Bug#18650)

1150 1151 1152 1153 1154
2014-10-07  Dmitry Antipov  <dmantipov@yandex.ru>

	* font.c (Ffont_get_glyphs): Use validate_subarray and fix
	the case where an optional string is used.  Adjust docstring.

1155 1156 1157 1158
2014-10-06  Stefan Monnier  <monnier@iro.umontreal.ca>

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

1159 1160 1161 1162 1163 1164 1165
2014-10-05  Paul Eggert  <eggert@cs.ucla.edu>

	* keyboard.c (Qleft, Qright): Remove duplicate definitions (Bug#9927).
	These were already defined in buffer.c, and the duplicate
	definitions cause problems on platforms like 'gcc -fno-common'.
	Reported by Peter Dyballa in: http://bugs.gnu.org/9927#137

Jan Djärv's avatar
Jan Djärv committed
1166 1167
2014-10-05  Jan Djärv  <jan.h.d@swipnet.se>

Jan Djärv's avatar
Jan Djärv committed
1168 1169 1170
	* nsterm.m (updateFrameSize:): Only call update_frame_tool_bar
	if toolbar is visible.

1171 1172 1173
	* nsfont.m (nsfont_draw): Use CGFloat for GNUstep newer than
	0.23 (Bug#18030).

1174 1175
	* nsterm.m (syms_of_nsterm): ns-use-fullscreen-animation is new.
	(toggleFullScreen:): Use ns-use-fullscreen-animation for animate.
1176 1177
	(ns_select, ns_read_socket): Use unwind_protect to decrease
	apploopnr (Bug#18345).
1178 1179
	(ns_draw_window_cursor): Adjust y for hbar cursor only if smaller than
	line height (Bug#17977).
1180

1181
	* macfont.m: Fix indentation and import changes from macport 24.3.94.
Jan Djärv's avatar
Jan Djärv committed
1182
	(macfont_closest_traits_index): New function.
1183
	(macfont_closest_traits_index_p): Rename from
Jan Djärv's avatar
Jan Djärv committed
1184 1185 1186
	macfont_closest_traits_index.
	(macfont_list): Use macfont_closest_traits_index_p.

1187 1188 1189 1190 1191
2014-10-05  K. Handa  <handa@gnu.org>

	* coding.c (detect_coding_iso_2022): Set coding->rejected
	correctly when an invalid escape sequence is found (Bug#18610).

1192 1193 1194 1195 1196 1197
2014-10-04  Jan Djärv  <jan.h.d@swipnet.se>

	* gtkutil.c (create_menus): Only add tearoffs to empty menus.
	(xg_update_submenu): Remove has_tearoff_p, pass 1 to create_menus
	for add_tearoff_p.

1198 1199
2014-10-04  Martin Rudalics  <rudalics@gmx.at>

1200 1201
	* buffer.c (scroll_bar_width, scroll_bar_height):
	Fix doc-strings.
1202 1203 1204
	* window.c (Fset_window_scroll_bars): Fix doc-string.
	(Fwindow_scroll_bars): Have it return what the doc-string says.

1205
2014-10-03  Eli Zaretskii  <eliz@gnu.org>
1206 1207 1208 1209

	* xdisp.c (move_it_by_lines): Call reseat_1 after moving the
	iterator backwards, to resync the bidi iterator.  (Bug#18584)

1210 1211 1212 1213 1214 1215 1216 1217 1218 1219 1220
2014-10-03  Dmitry Antipov  <dmantipov@yandex.ru>

	Consistently use min and max macros from lisp.h.
	* coding.c (min, max):
	* font.c (MAX):
	* unexhp9k800.c (min):
	* unexw32.c (min, max): Use definitions from lisp.h.
	* regex.c (MAX, MIN) [!emacs]: Define own max and min as such.
	Adjust users.
	* gmalloc.c (min): Tiny style change.

1221 1222 1223
	* fileio.c (emacs_readlinkat, Finsert_file_contents):
	* w32fns.c, xfns.c (x_create_tip_frame): Use AUTO_STRING.

1224 1225 1226 1227 1228 1229 1230 1231 1232 1233 1234
2014-10-03  Paul Eggert  <eggert@cs.ucla.edu>

	Fix x-focus-frame bug with "Not an in-range integer" (Bug#18586).
	* xselect.c (X_SHRT_MAX, X_SHRT_MIN, X_LONG_MAX, X_LONG_MIN)
	(X_ULONG_MAX): Move these macros to xterm.h.
	(x_fill_property_data): Be more generous about allowing either
	signed or unsigned data of the appropriate width.
	* xterm.h (x_display_set_last_user_time): New function.
	All setters of last_user_time changd to use this function.
	If ENABLE_CHECKING, check that the times are in range.

1235 1236 1237 1238 1239
2014-10-02  Eli Zaretskii  <eliz@gnu.org>

	* dispnew.c (adjust_decode_mode_spec_buffer): Use 'int' instead of
	'ssize_t'.  Suggested by Paul Eggert <eggert@cs.ucla.edu>.

Stefan Monnier's avatar
Stefan Monnier committed
1240
2014-10-02  Jan Djärv  <jan.h.d@swipnet.se>
1241 1242 1243

	* xfaces.c (Finternal_set_lisp_face_attribute): Don't try to
	make a font_object from a tty frame (Bug#18573).
Stefan Monnier's avatar
Stefan Monnier committed
1244 1245
	(Finternal_set_lisp_face_attribute): Add FIXME comment.

1246 1247 1248 1249 1250 1251 1252 1253
2014-10-02  Dmitry Antipov  <dmantipov@yandex.ru>

	* alloc.c (mark_overlay): Assume that overlay boundaries are
	always markers.  Add comment.
	* lread.c (read_internal_start): Use convenient validate_subarray.
	Adjust docstring.
	(Fread_from_string): Adjust docstring.

1254 1255
2014-10-02  Stefan Monnier  <monnier@iro.umontreal.ca>

1256 1257
	* lisp.h: Fix up compilation for USE_STACK_LISP_OBJECTS=false.

1258 1259 1260
	* nsselect.m (ns-own-selection-internal, ns-disown-selection-internal):
	Rename from the "x-" prefix.

1261 1262 1263 1264
2014-10-01  Stefan Monnier  <monnier@iro.umontreal.ca>

	* xselect.c (selection-converter-alist): Fix docstring.

1265 1266 1267 1268 1269 1270
2014-10-01  Eli Zaretskii  <eliz@gnu.org>

	* w32proc.c (sys_spawnve): Avoid modification of the CMDNAME
	argument passed by the caller, when we mirror all slashes into
	backslashes.

1271 1272 1273 1274 1275
2014-10-01  Dmitry Antipov  <dmantipov@yandex.ru>

	* gtkutil.c (xg_set_toolkit_horizontal_scroll_bar_thumb):
	Resurrect old code and fix compilation with GTK < 2.13.6.

1276 1277
2014-10-01  Paul Eggert  <eggert@cs.ucla.edu>

1278 1279 1280 1281 1282 1283 1284 1285 1286 1287 1288 1289 1290 1291 1292 1293
	Use AUTO_CONS instead of SCOPED_CONS, etc.
	* frame.h (AUTO_FRAME_ARG): Rename from FRAME_PARAMETER.
	* lisp.h (AUTO_CONS): Rename from scoped_cons.
	(AUTO_LIST1): Rename from scoped_list1.
	(AUTO_LIST2): Rename from scoped_list2.
	(AUTO_LIST3): Rename from scoped_list3.
	(AUTO_LIST4): Rename from scoped_list4.
	(AUTO_STRING): Rename from SCOPED_STRING.
	* frame.h (AUTO_FRAME_ARG):
	* lisp.h (AUTO_CONS, AUTO_LIST1, AUTO_LIST2, AUTO_LIST3)
	(AUTO_LIST4, AUTO_STRING):
	Prepend a new argument 'name'.
	Declare a variable instead of yielding a value.
	All uses changed.
	* lisp.h (STACK_CONS, AUTO_CONS_EXPR): New internal macros.

1294 1295 1296 1297 1298 1299 1300 1301 1302 1303
2014-09-30  Eli Zaretskii  <eliz@gnu.org>

	* w32fns.c (w32_createwindow): Accept an additional argument, an
	array of 2 values specifying the coordinates of the frame's
	top-left corner.  Use these values instead of calling x_get_arg,
	which can cons Lisp objects, and therefore cannot be called except
	from the main thread.  Remove redundant tests for the default
	values.
	(my_create_window): Move the calculation of the coordinates of the
	frame's top-left edge here.  Pass them to the input thread via the
1304 1305
	second parameter of the WM_EMACS_CREATEWINDOW message.
	See http://lists.gnu.org/archive/html/emacs-devel/2014-09/msg00892.html
1306 1307
	for the details.

Stefan Monnier's avatar
Stefan Monnier committed
1308
2014-09-30  Eli Zaretskii  <eliz@gnu.org>
1309 1310 1311 1312 1313 1314 1315

	* xdisp.c (cursor_row_fully_visible_p): Update commentary.
	(redisplay_window): Treat the frame's frozen_window_starts flag
	the same way as the optional_new_start flag for the window: only
	obey it if the glyph row showing point will be fully visible.
	Likewise when the window start is in a continuation line.  If,
	after trying everything under the 'force_start' label, point is
1316 1317
	still not fully visible, give up and scroll the window.
	Add debugging traces.  (Bug#18545)
1318 1319 1320

	* window.c (Frecenter): Set the window's redisplay flag.

Stefan Monnier's avatar
Stefan Monnier committed
1321
2014-09-30  Eli Zaretskii  <eliz@gnu.org>
1322 1323 1324 1325 1326 1327 1328 1329 1330 1331 1332 1333

	* w32term.c (w32_read_socket): Don't use frame dimensions for
	resizing if GetClientRect returned an empty (0, 0, 0, 0)
	rectangle.  Check the return value of GetClientRect, and don't use
	the results if it didn't succeed.

	* dispnew.c (change_frame_size_1): Recompute the frame dimensions
	in columns and lines after correcting the pixel dimensions in
	check_frame_size.
	(adjust_decode_mode_spec_buffer): Add assertion to avoid passing
	negative values to xrealloc.  (Bug#18528)

1334 1335
2014-09-30  Paul Eggert  <eggert@cs.ucla.edu>

1336 1337 1338 1339
	* alloc.c: Remove now-unnecessary check.
	Suggested by Dmitry Antipov in:
	http://lists.gnu.org/archive/html/emacs-devel/2014-09/msg00891.html

1340 1341
	* xterm.c (x_term_init): Allocate temps on stack, not on heap.

1342 1343 1344
	* frame.c (x_set_frame_parameters): Port --enable-gcc-warnings
	to Ubuntu 14.04.1 x86-64.

1345 1346 1347 1348 1349 1350 1351 1352 1353 1354 1355 1356 1357 1358 1359 1360 1361 1362 1363 1364 1365 1366 1367 1368 1369 1370 1371 1372 1373 1374 1375 1376 1377 1378 1379 1380 1381 1382 1383
	Simplify stack-allocated Lisp objects, and make them more portable.
	The build_local_string macro was used in two ways: (1) string
	literals for which scoped allocation suffices, and (2) file name
	components, where it's not safe in general to assume bounded-size
	ASCII data.  Simplify by defining a new macro SCOPED_STRING that
	allocates a block-scope string, and by using SCOPED_STRING for (1)
	and build_string for (2).  Furthermore, actually use stack
	allocation only for objects known to have sufficient alignment.
	This simpler implementation means Emacs can make
	USE_STACK_LISP_OBJECTS the default unless GC_MARK_STACK !=
	GC_MAKE_GCPROS_NOOPS.
	* lisp.h (GCALIGNED): Align even if !USE_STACK_LISP_OBJECTS,
	for fewer differences among implementations.
	(struct Lisp_String): Now GCALIGNED.
	(USE_STACK_LISP_OBJECTS): Default to true, since the
	implementation no longer insists on a nonempty GCALIGNED.
	But make it false if GC_MARK_STACK != GC_MAKE_GCPROS_NOOPS.
	(SCOPED_CONS_INITIALIZER): Remove, since it's no longer needed
	separately.  Move definiens to scoped_cons.  The old definition
	was incorrect when GCALIGNED was defined to be empty.
	(union Aligned_String): New type.
	(USE_STACK_CONS, USE_STACK_STRING): New constants, so that the
	implementation ports to compilers that don't align strictly enough.
	Don't worry about the union sizes; it's not worth bothering about.
	(scoped_cons, scoped_list1, scoped_list3, scoped_list4):
	Rewrite using USE_STACK_CONS.
	(scoped_cons): Assume the use of union Aligned_Cons.
	(lisp_string_size, make_local_string, build_local_string): Remove.
	Unless otherwise specified, all callers of build_local_string
	changed to use SCOPED_STRING.
	(SCOPED_STRING): New macro.
	* data.c (wrong_choice):
	* menu.c (single_menu_item):
	* process.c (Fformat_network_address):
	Hoist use of SCOPED_STRING out of a scope, so that its returned
	object lives long enough.
	* fileio.c (Fexpand_file_name): Use build_string, not SCOPED_STRING,
	as the string might be long or might not be ASCII.

1384 1385 1386 1387
2014-09-29  Eli Zaretskii  <eliz@gnu.org>

	* msdos.c (internal_terminal_init): Bump version to 25.

1388 1389 1390 1391 1392 1393 1394 1395 1396 1397 1398 1399 1400 1401 1402 1403 1404 1405 1406 1407 1408 1409 1410 1411 1412
2014-09-29  Dmitry Antipov  <dmantipov@yandex.ru>

	Keep stack-allocated Lisp objects fast rather than versatile.
	* lisp.h (union Aligned_Cons) [!GCALIGNED]: Define as such.
	(SCOPED_CONS_INITIALIZER): New macro.
	(scoped_cons) [USE_STACK_LISP_OBJECTS]: Use it.
	(USE_LOCAL_ALLOCA): Remove.
	(local_cons, local_list1, local_list2, local_list3, local_list4):
	Remove.  Stack overflow checking makes them too slow.
	(make_local_vector): Likewise.  Also we just don't have enough
	users for it.
	(enum LISP_STRING_OVERHEAD): Remove.
	(local_string_init, local_vector_init): Remove prototypes.
	(make_local_string, build_local_string): Redesign to target short
	compile-time string constants, fall back to regular string allocation
	where appropriate.
	(lisp_string_size): New function.
	(verify_ascii) [ENABLE_CHECKING]: Add prototype.
	* alloc.c (local_string_init, local_vector_init): Remove.
	(verify_ascii) [ENABLE_CHECKING]: New function.
	* buffer.c, charset.c, chartab.c, data.c, editfns.c, emacs.c, fileio.c:
	* fns.c, font.c, fontset.c, frame.c, keyboard.c, keymap.c, lread.c:
	* menu.c, minibuf.c, process.c, textprop.c, xdisp.c, xfns.c, xfont.c:
	* xselect.c, xterm.c: All related users changed.

1413 1414 1415 1416 1417
2014-09-28  Ken Brown  <kbrown@cornell.edu>

	* sheap.c (bss_sbrk_buffer_beg): Remove redundant variable.
	* gmalloc.c [CYGWIN]: Adapt to change in sheap.c.

1418 1419 1420 1421 1422 1423 1424 1425
2014-09-27  Ken Brown  <kbrown@cornell.edu>

	Fix implementation of HYBRID_MALLOC on Cygwin.
	* sheap.c (bss_sbrk_buffer_end): Cast to void *.
	(bss_sbrk_buffer_beg): New variable.  Use it...
	* gmalloc.c (ALLOCATED_BEFORE_DUMPING) [CYGWIN]: ...here, to fix
	incorrect definition.

1426 1427
2014-09-27  Stefan Monnier  <monnier@iro.umontreal.ca>

1428 1429 1430
	* keyboard.c (track-mouse): Rename to internal--track-mouse.
	Make it into a function and change arg to be a function.

1431 1432
	* lisp.mk (lisp): Add elisp-mode.elc.

1433 1434 1435 1436 1437 1438
2014-09-26  Paul Eggert  <eggert@cs.ucla.edu>

	* xfns.c (x_default_scroll_bar_color_parameter):
	Use USE_LOCAL_ALLOCA only if USE_TOOLKIT_SCROLL_BARS,
	to pacify --enable-gcc-warnings in non-scrollbar builds.

1439 1440 1441 1442
2014-09-26  Ken Brown  <kbrown@cornell.edu>

	* w32term.h (ALIGN_STACK): Fix the cpp condition.

1443 1444
2014-09-25  Eli Zaretskii  <eliz@gnu.org>

1445 1446 1447
	* lisp.h (USE_STACK_LISP_OBJECTS): Default to false for 32-bit
	MinGW builds that use GCC before 4.2.

1448 1449 1450 1451 1452 1453 1454 1455 1456 1457 1458 1459 1460 1461 1462 1463 1464
	Default to stack objects on DOS_NT platforms as well.
	* w32term.h (ALIGN_STACK) [__GNUC__]: Define to
	__attribute__((force_align_arg_pointer)) for GCC 4.2 and later.

	* lisp.h (USE_STACK_LISP_OBJECTS): Remove the !DOS_NT condition.

	* w32proc.c (enum_locale_fn, enum_codepage_fn): Add the
	ALIGN_STACK attribute.

	* w32fns.c (w32_monitor_enum): Add the ALIGN_STACK attribute.

	* w32uniscribe.c (add_opentype_font_name_to_list): Add the
	ALIGN_STACK attribute.

	* w32font.c (add_font_name_to_list, add_font_entity_to_list)
	(add_one_font_entity_to_list): Add the ALIGN_STACK attribute.

1465 1466 1467 1468 1469 1470 1471 1472 1473 1474 1475
2014-09-25  Martin Rudalics  <rudalics@gmx.at>

	* frame.c (frame_inhibit_resize):
	* widget.c (EmacsFrameResize):
	* window.c (resize_frame_windows, Fset_window_configuration):
	* xdisp.c (expose_frame):
	* xfns.c (x_change_tool_bar_height):
	* xmenu.c (update_frame_menubar):
	* xterm.c (handle_one_xevent, x_new_font, x_set_window_size_1):
	Remove code left dead after 2014-07-27 changes.

1476 1477 1478 1479 1480 1481 1482 1483 1484 1485 1486 1487 1488 1489 1490 1491 1492 1493 1494 1495 1496 1497 1498 1499 1500 1501 1502 1503 1504 1505 1506 1507 1508 1509 1510 1511 1512 1513 1514 1515 1516 1517 1518 1519 1520 1521 1522 1523 1524 1525
2014-09-25  Paul Eggert  <eggert@cs.ucla.edu>

	Fix local_cons etc. to not exhaust the stack when in a loop.
	Problem reported in:
	http://lists.gnu.org/archive/html/emacs-devel/2014-09/msg00696.html
	* buffer.c (Fother_buffer, other_buffer_safely, init_buffer):
	* charset.c (load_charset_map_from_file, Ffind_charset_region)
	(Ffind_charset_string):
	* chartab.c (uniprop_encode_value_numeric, uniprop_table):
	* data.c (wrong_range):
	* editfns.c (Fpropertize, format2):
	* emacs.c (init_cmdargs, decode_env_path):
	* fileio.c (auto_save_error):
	* fns.c (Fyes_or_no_p):
	* font.c (font_style_to_value, font_parse_xlfd)
	(font_parse_family_registry, font_delete_unmatched)
	(font_add_log):
	* fontset.c (Fset_fontset_font):
	* frame.c (x_get_arg):
	* keyboard.c (echo_dash, safe_run_hooks_error, parse_menu_item)
	(read_char_minibuf_menu_prompt):
	* keymap.c (silly_event_symbol_error, describe_vector):
	* lread.c (load_warn_old_style_backquotes):
	* menu.c (single_menu_item):
	* minibuf.c (Fread_buffer):
	* process.c (status_message, Fformat_network_address)
	(server_accept_connection):
	* textprop.c (copy_text_properties):
	* xdisp.c (Fcurrent_bidi_paragraph_direction):
	* xfns.c (x_default_scroll_bar_color_parameter):
	* xfont.c (xfont_open):
	* xselect.c (x_clipboard_manager_error_1):
	* xterm.c (x_term_init):
	Put USE_LOCAL_ALLOCA at the start of the function.
	* fns.c (maybe_resize_hash_table): Use build_string instead of
	build_local_string, since we'd otherwise need a conditional
	USE_LOCAL_ALLOCA here, but this is just debugging output and is
	not worth the bother of optimization.
	* font.c (font_delete_unmatched): Remove by-hand code that
	observed MAX_ALLOCA limit, since it's now done automatically.
	* keymap.c (Fsingle_key_description): Put USE_SAFE_ALLOCA at top,
	since build_local_string needs its sa_alloc.
	* lisp.h (lisp_word_count): New function.
	(SAFE_ALLOCA_LISP): Use it.
	(USE_LOCAL_ALLOCA): New macro.
	(local_cons, make_local_vector, make_local_string):
	Observe the MAX_ALLOCA limit.
	(LISP_STRING_OVERHEAD): New constant.
	(make_local_string): Use it.

1526 1527
2014-09-24  Paul Eggert  <eggert@cs.ucla.edu>

1528 1529 1530 1531 1532
	Default to stack objects on non-GNU/Linux, non-DOS_NT platforms.
	* lisp.h (USE_STACK_LISP_OBJECTS): Also default to true
	if !defined DOS_NT && !defined GNU_LINUX.  I've tested this on AIX
	and Solaris and it's likely to work on similar platforms.

1533 1534 1535 1536 1537 1538 1539 1540 1541 1542 1543
	Avoid signed integer overflow when converting Time to ptrdiff_t.
	* keyboard.c (INPUT_EVENT_POS_MAX, INPUT_EVENT_POS_MIN):
	New macros.
	(position_to_Time, Time_to_position): New functions.
	(gen_help_event, kbd_buffer_get_event): Use them.
	* systime.h (Time) [emacs && !HAVE_X_WINDOWS]:
	Go back to plain 'unsigned long', so that 'Time' is the same
	for both X and non-X builds; this is less likely to cause surprise.
	* termhooks.h: Remove compile-time check that Time and ptrdiff_t
	are the same size; this is no longer required.

1544 1545 1546 1547
	* keyboard.c (make_lispy_event): Avoid unnecessary tests
	of bit 28 and of whether an unsigned value is negative.
	This simplifies the code a bit, and pacifies clang 3.4.

1548 1549 1550 1551 1552 1553 1554 1555 1556 1557 1558 1559 1560
2014-09-24  Eli Zaretskii  <eliz@gnu.org>

	* systime.h (Time): Define as size_t, to be consistent with 64-bit
	Windows builds, where 'long' is a 32-bit type.

	* w32inevt.h (w32_console_mouse_position): Update the argument
	types to use 'Time'.

	* w32term.c (w32_mouse_position)
	(x_horizontal_scroll_bar_report_motion)
	(x_scroll_bar_report_motion): Update the argument types to use
	'Time'.

1561 1562 1563 1564 1565 1566 1567 1568 1569 1570 1571 1572 1573 1574 1575 1576 1577 1578 1579 1580
2014-09-24  Dmitry Antipov  <dmantipov@yandex.ru>

	* termhooks.h (enum scroll_bar_part): Begin from 0 to allow...
	(struct input_event): ...unsigned bitfields.  Likewise for
	`event_kind' member.  Prefer unsigned for `code' and 'modifiers'.
	Use `timestamp' for HELP_EVENT position.  Add compile-time assert.
	* keyboard.c (gen_help_event, kbd_buffer_store_help_event)
	(kbd_buffer_get_event): Adjust users.
	(scroll_bar_parts): Add Qnil to match scroll_bar_nowhere.
	(make_scroll_bar_position): New function, refactored out of...
	(make_lispy_event): ...adjusted user.
	* nsterm.h (EmacsScroller): Use enum for `last_hit_part' member.
	* nsterm.m (ns_mouse_position, mouseUp):
	* term.c (term_mouse_position):
	* w32inevt.c (w32_console_mouse_position):
	* w32term.c (w32_mouse_position):
	* xterm.c (XTmouse_position): Use scroll_bar_above_handle.
	(x_send_scroll_bar_event, xm_scroll_callback, xg_scroll_callback):
	Prefer enum and explicit enum members to integers and numeric values.

1581 1582 1583 1584 1585
	* chartab.c (uniprop_encode_value_numeric):
	* font.c (font_style_to_value): Use make_local_vector.
	(font_delete_unmatched): Use local_cons but respect MAX_ALLOCA.
	* keymap.c (append_key): Use scoped_list1.

1586 1587 1588
	* lisp.h (USE_STACK_LISP_OBJECTS): Enable by default if GNU_LINUX
	&& __GNUC__ && !__clang__.  Mention known problems.  Adjust comment.

1589 1590 1591 1592 1593 1594 1595 1596 1597 1598 1599
2014-09-24  Paul Eggert  <eggert@cs.ucla.edu>

	Fix some slow uses and misuses of strcat.
	* doc.c (get_doc_string):
	* gtkutil.c (get_utf8_string):
	* xsmfns.c (x_session_initialize):
	Avoid recomputation of string length.
	* ftfont.c (ftfont_spec_pattern):
	* xfns.c (xic_create_fontsetname):
	Don't assume output buffer is initially zero.

1600 1601
2014-09-23  Paul Eggert  <eggert@cs.ucla.edu>

1602 1603 1604 1605 1606
	* lisp.h (lispstpcpy): Rename from lispstrcpy, and act like stpcpy.
	All callers changed.
	* xterm.c (x_term_init): Use new functionality to avoid two needs
	to compute a string length.

1607 1608
	* dispextern.h, xdisp.c (window_box_right_offset): Now static.

1609 1610 1611 1612 1613 1614 1615 1616 1617 1618 1619 1620 1621 1622 1623 1624 1625
2014-09-23  Dmitry Antipov  <dmantipov@yandex.ru>

	Use known length of a Lisp string to copy it faster.
	* lisp.h (lispstrcpy): New function.  Add comment.
	* callproc.c (child_setup):
	* dbusbind.c (xd_append_arg):
	* doc.c (get_doc_string):
	* font.c (Ffont_xlfd_name):
	* frame.c (xrdb_get_resource):
	* process.c (Fmake_network_process, network_interface_info):
	* w32fns.c (Fx_open_connection):
	* w32proc.c (sys_spawnve):
	* xfns.c (select_visual):
	* xfont.c (xfont_list):
	* xsmfns.c (x_session_initialize):
	* xterm.c (x_term_init): Use it.

1626 1627 1628
2014-09-23  Paul Eggert  <eggert@cs.ucla.edu>

	Fix SAFE_ALLOCA to not exhaust the stack when in a loop.
1629
	Problem reported by Dmitry Antipov in thread leading to:
1630 1631 1632 1633 1634 1635 1636 1637 1638 1639
	http://lists.gnu.org/archive/html/emacs-devel/2014-09/msg00713.html
	This patch fixes only SAFE_ALLOCA, SAFE_NALLOCA, and SAFE_ALLOCA_LISP;
	the experimental local_* macros enabled by USE_LOCAL_ALLOCATORS
	remain unfixed.
	* callproc.c (call_process): Save and restore sa_avail.
	* lisp.h (USE_SAFE_ALLOCA): Define sa_avail.
	(AVAIL_ALLOCA): New macro.
	(SAFE_ALLOCA, SAFE_NALLOCA, SAFE_ALLOCA_LISP):
	Use it, and check against sa_avail rather than MAX_ALLOCA.