ChangeLog 279 KB
Newer Older
Paul Eggert's avatar
Paul Eggert committed
1 2
2013-11-05  Paul Eggert  <eggert@cs.ucla.edu>

3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46 47 48
	Simplify and port recent bool vector changes.
	* alloc.c (ROUNDUP): Move here from lisp.h, since it's now used
	only in this file.  Use a more-efficient implementation if the
	second argument is a power of 2.
	(ALIGN): Rewrite in terms of ROUNDUP.  Make it a function.
	Remove no-longer-necessary compile-time checks.
	(bool_vector_exact_payload_bytes): New function.
	(bool_vector_payload_bytes): Remove 2nd arg; callers that need
	exact payload changed to call the new function.  Do not assume
	that the arg or result fits in ptrdiff_t.
	(bool_vector_fill): New function.
	(Fmake_bool_vector): Use it.  Don't assume bit counts fit
	in ptrdiff_t.
	(vroundup_ct): Don't assume arg fits in size_t.
	* category.c (SET_CATEGORY_SET): Remove.  All callers now just
	invoke set_category_set.
	(set_category_set): 2nd arg is now EMACS_INT and 3rd is now bool.
	All callers changed.  Use bool_vector_set.
	* category.h (XCATEGORY_SET): Remove; no longer needed.
	(CATEGORY_MEMBER): Now a function.  Rewrite in terms of
	bool_vector_bitref.
	* data.c (Faref): Use bool_vector_ref.
	(Faset): Use bool_vector_set.
	(bits_word_to_host_endian): Don't assume you can shift by CHAR_BIT.
	(Fbool_vector_not, Fbool_vector_count_matches)
	(Fbool_vector_count_matches_at): Don't assume CHAR_BIT == 8.
	* fns.c (concat): Use bool_vector_ref.
	(Ffillarray): Use bool_vector_fill.
	(mapcar1): Use bool_vector_ref.
	(sxhash_bool_vector): Hash words, not bytes.
	* lisp.h (BOOL_VECTOR_BITS_PER_CHAR): Now a macro as well as
	a constant, since it's now used in #if.
	(bits_word, BITS_WORD_MAX, BITS_PER_BITS_WORD): Fall back on
	unsigned char on unusual architectures, so that we no longer
	assume that the number of bits per bits_word is a power of two or
	is a multiple of 8 or of CHAR_BIT.
	(Qt): Add forward decl.
	(struct Lisp_Bool_Vector): Don't assume EMACS_INT is aligned
	at least as strictly as bits_word.
	(bool_vector_data, bool_vector_uchar_data): New accessors.
	All data structure accesses changed to use them.
	(bool_vector_words, bool_vector_bitref, bool_vector_ref)
	(bool_vector_set): New functions.
	(bool_vector_fill): New decl.
	(ROUNDUP): Move to alloc.c as described above.

Paul Eggert's avatar
Paul Eggert committed
49 50 51 52
	Fix recent gnutls changes.
	* gnutls.c (Fgnutls_boot): Don't assume C99.
	* process.c (wait_reading_process_output): Fix typo in recent change.

53 54 55 56 57 58 59 60
2013-11-05  Teodor Zlatanov  <tzz@lifelogs.com>

	* process.c (wait_reading_process_output, read_process_output)
	(send_process): Check gnutls_state is not NULL.

	* gnutls.c (emacs_gnutls_handle_error): Adjust log level for EAGAIN.
	(Fgnutls_boot): Set process gnutls_p later, after initialization.

Jan Djärv's avatar
Jan Djärv committed
61 62 63 64 65 66 67 68 69 70 71 72 73 74
2013-11-04  Jan Djärv  <jan.h.d@swipnet.se>

	* nsterm.m (init, run, stop:): New methods in EmacsApp for
	OSX >= 10.9 to prevent memory leak of GCD dispatch source.

	* nsterm.h (EmacsApp): Add shouldKeepRunning and isFirst for
	OSX >= 10.9.

	* nsfns.m (Fx_create_frame): Fix memory leak.

	* macfont.m (CG_SET_FILL_COLOR_WITH_GC_FOREGROUND)
	(CG_SET_FILL_COLOR_WITH_GC_BACKGROUND)
	(CG_SET_STROKE_COLOR_WITH_GC_FOREGROUND): Fix memory leak.

75 76 77 78 79 80 81 82 83 84 85 86 87 88 89 90 91 92 93 94 95 96 97 98 99 100
2013-11-04  Eli Zaretskii  <eliz@gnu.org>

	* xdisp.c (message3_nolog, message_with_string): Encode the string
	before writing it to the terminal in a non-interactive session.

	* lread.c (openp): If both FILENAME and SUFFIX are unibyte, make
	sure we concatenate them into a unibyte string.

	* fileio.c (make_temp_name): Encode PREFIX, and decode the
	resulting temporary name before returning it to the caller.
	(Fexpand_file_name): If NAME is pure-ASCII and DEFAULT_DIRECTORY
	is a unibyte string, convert NAME to a unibyte string to ensure
	that the result is also a unibyte string.

	* emacs.c (init_cmdargs): Use build_unibyte_string to make sure we
	create unibyte strings from default paths and directory/file
	names.

	* coding.h (ENCODE_FILE): Do not attempt to encode a unibyte
	string.

	* callproc.c (init_callproc): Use build_unibyte_string to make
	sure we create unibyte strings from default paths and
	directory/file names.

	* buffer.c (init_buffer): Don't store default-directory of
101
	*scratch* in multibyte form.  The original problem which led to
102 103 104 105
	that is described in
	http://lists.gnu.org/archive/html/emacs-pretest-bug/2004-11/msg00532.html,
	but it was solved long ago.  (Bug#15260)

Paul Eggert's avatar
Paul Eggert committed
106 107 108 109 110 111 112 113 114 115 116 117 118 119 120 121 122 123 124 125 126 127 128 129 130 131 132 133 134 135 136 137 138 139
2013-11-04  Paul Eggert  <eggert@cs.ucla.edu>

	Port to stricter C99 platforms.
	Especially, C99 prohibits nesting a struct X inside struct Y if
	struct X has a flexible array member.
	* alloc.c (struct sdata): New type.
	(sdata): Implement in terms of struct sdata.
	Remove u member; all uses replaced by next_vector, set_next_vector.
	(SDATA_SELECTOR, SDATA_DATA, SDATA_DATA_OFFSET): Adjust to sdata change.
	(SDATA_DATA_OFFSET): Now a constant, not a macro.
	(struct sblock): Rename first_data member to data, which is now
	a flexible array member.  All uses changed.
	(next_vector, set_next_vector, large_vector_vec): New functions.
	(vector_alignment): New constant.
	(roundup_size): Make it a multiple of ALIGNOF_STRUCT_LISP_VECTOR, too.
	(struct large-vector): Now merely a NEXT member, since the old approach
	ran afoul of stricter C99.  All uses changed to use
	large_vector_vec or large_vector_offset.
	(large_vector_offset): New constant.
	* dispnew.c: Include tparam.h, for tgetent.
	Do not include term.h; no longer needed.
	* gnutls.c (Fgnutls_boot): Don't continue after calling a _Noreturn.
	* lisp.h (ENUM_BF) [__SUNPRO_C && __STDC__]: Use unsigned int.
	(struct Lisp_Vector): Use a flexible array member for contents,
	instead of a union with a member that is an array of size 1.
	All uses changed.
	(ALIGNOF_STRUCT_LISP_VECTOR): New constant, to make up for the
	fact that the struct no longer contains a union.
	(struct Lisp_Misc_Any, struct Lisp_Marker, struct Lisp_Overlay)
	(struct Lisp_Save_Value, struct Lisp_Free):
	Use unsigned, not int, for spacers, to avoid c99 warning.
	(union specbinding): Use unsigned, not bool, for bitfield, as
	bool is not portable to pre-C99 hosts.

140 141 142 143
2013-11-04  Glenn Morris  <rgm@gnu.org>

	* emacs.c (usage_message): Mention that `-L :...' appends.

144 145
2013-11-02  Glenn Morris  <rgm@gnu.org>

146 147 148
	* Makefile.in (abs_builddir): Remove.
	(bootstrap_exe): Use relative filename.

149 150 151 152 153 154 155
	Use relative filenames in TAGS files.
	* Makefile.in (abs_srcdir): Remove it again.
	(.PHONY): Remove frc.
	(maintainer-clean): No more TAGS-LISP file.
	(TAGS): Pass relative file names to etags.
	(../lisp/TAGS): Rename from TAGS-LISP.  Work in ../lisp.

156
	* Makefile.in (abs_srcdir): New, set by configure.
157
	(lispdir): Remove.
158 159 160
	(maintainer-clean): Remove pointless echo.  That should be in the
	top-level Makefile, if anywhere.  Delete TAGS-LISP.
	(extraclean): No s/ and m/ directories for some time.
161 162
	(TAGS): Also depend on ctagsfiles3.
	Remove no-longer-defined S_FILE.
163
	Pass absolute filenames to etags once more.
164
	(TAGS-LISP): Replace lispdir with its expansion.
165 166 167
	(TAGS-LISP, $(lwlibdir)/TAGS): Correctly pass ETAGS to sub-makes.
	($(lwlibdir)/TAGS): Remove useless subshell, check cd return value.

168 169 170 171
2013-11-02  Jan Djärv  <jan.h.d@swipnet.se>

	* xfaces.c (check_lface_attrs, realize_default_face): Add
	LFACE_DISTANT_FOREGROUND_INDEX (Bug#15788).
172
	(realize_default_face): Set DISTANT_FOREGROUND to unspecified_fg.
173

174 175 176 177 178
2013-11-02  Paul Eggert  <eggert@cs.ucla.edu>

	* emacs.c (original_pwd): Remove global var by making it local.
	(init_cmdargs): New arg ORIGINAL_PWD; caller changed.

Jan Djärv's avatar
Jan Djärv committed
179 180 181 182 183 184 185 186 187 188 189 190 191 192 193 194 195 196 197
2013-11-01  Jan Djärv  <jan.h.d@swipnet.se>

	* xfaces.c: Declare color_distance.
	(QCdistant_foreground): New variable.
	(NEAR_SAME_COLOR_THRESHOLD): New define.
	(load_color2): New function.
	(load_color): Call load_color2.
	(load_face_colors): Call load_color2 and if distant-color is specified
	calculate distant and use distant-color if colors are near.
	(LFACE_DISTANT_FOREGROUND): New define.
	(merge_face_ref, Finternal_set_lisp_face_attribute)
	(Finternal_get_lisp_face_attribute)
	(x_supports_face_attributes_p): Handle distant-foreground similar to
	foreground.
	(syms_of_xfaces): DEFSYM QCdistant_foreground.

	* dispextern.h (lface_attribute_index): Add
	LFACE_DISTANT_FOREGROUND_INDEX.

198 199
2013-11-01  Claudio Bley  <claudio.bley@googlemail.com>

200 201
	* image.c (pbm_next_char): New function.  See
	http://netpbm.sourceforge.net/doc/pbm.html for the details.
202 203
	(pbm_scan_number): Use it.
	(Qlibjpeg_version): New variable.
204 205
	(syms_of_image): DEFSYM and initialize it.

206 207 208 209
2013-10-31  Jan Djärv  <jan.h.d@swipnet.se>

	* emacs.c (main): Skip -psn args on OSX even if ! isatty (0).

210 211 212 213 214 215 216
2013-10-31  Glenn Morris  <rgm@gnu.org>

	* emacs.c (original_pwd): New char.
	(main): If using --chdir, store original_pwd.
	(init_cmdargs): When setting Vinvocation_directory based on a
	relative argv[0], use original_pwd if set.  (Bug#15768)

217 218 219 220
2013-10-29  Stefan Monnier  <monnier@iro.umontreal.ca>

	* keyboard.c (command_loop_1): If command is nil, call `undefined'.

221 222 223 224 225 226
2013-10-29  Paul Eggert  <eggert@cs.ucla.edu>

	* insdel.c: Fix minor problems found by static checking.
	(Qregion_extract_function): Now static.
	(prepare_to_modify_buffer_1): Remove unused locals.

227 228 229 230 231 232 233 234 235 236 237 238 239 240 241 242 243 244 245 246 247 248 249 250
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.

251 252 253 254 255 256 257 258 259 260 261 262 263 264 265
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.

266 267 268 269
2013-10-29  Stefan Monnier  <monnier@iro.umontreal.ca>

	* eval.c (run_hook_with_args): Use FUNCTIONP.

270 271 272 273 274 275 276 277 278 279 280 281 282 283 284
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.

285 286 287 288 289 290
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.

291 292 293 294 295 296 297 298 299 300 301
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.

302 303 304 305 306 307 308
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.

309 310 311 312 313
2013-10-26  Eli Zaretskii  <eliz@gnu.org>

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

314 315 316 317
2013-10-26  Xue Fuqiao  <xfq.free@gmail.com>

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

318 319
2013-10-25  Eli Zaretskii  <eliz@gnu.org>

320 321 322
	* w32uniscribe.c (uniscribe_close): Adjust the argument list to
	the changed signature of the font driver's 'close' method.

323 324 325 326 327 328 329 330
	* 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.

331 332 333 334 335 336 337 338 339 340 341
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):
342 343 344 345 346 347 348 349 350
	* 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.
351

352 353 354 355
2013-10-24  Glenn Morris  <rgm@gnu.org>

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

356 357
2013-10-23  Dmitry Antipov  <dmantipov@yandex.ru>

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

362 363 364 365 366 367 368
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.

369 370 371 372 373 374
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.

375 376 377 378 379 380 381 382 383 384 385 386 387
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
388 389
2013-10-20  Jan Djärv  <jan.h.d@swipnet.se>

390 391 392
	* 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
393 394
	* nsterm.m (Qcocoa, Qgnustep): New variables.
	(syms_of_nsterm): Defsym Qcocoa, Qgnustep.  Fprovide appropriate one.
395
	(ns_get_color): Make selection color work for GNUStep also.
Jan Djärv's avatar
Jan Djärv committed
396

397 398 399 400 401 402 403 404
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.

405 406 407 408 409 410 411 412 413 414 415 416 417 418 419 420 421 422 423 424 425 426 427 428
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.

429 430 431 432 433 434 435
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
436 437
2013-10-17  Paul Eggert  <eggert@cs.ucla.edu>

438 439 440 441 442 443 444 445 446 447 448 449 450 451 452 453 454 455 456 457 458 459 460 461 462 463 464 465 466 467
	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.

468 469 470 471 472
	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
473 474 475 476 477 478 479 480
	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.

481 482 483 484 485
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.

486 487 488 489 490
2013-10-16  Jan Djärv  <jan.h.d@swipnet.se>

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

491 492
2013-10-16  Eli Zaretskii  <eliz@gnu.org>

493 494 495 496 497 498
	* 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.

499 500
	* process.h (network_interface_list, network_interface_info):
	New prototypes.
501 502 503 504 505 506 507 508

	* 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.

509 510 511
	* menu.c (have_boxes): Fix redundant simulation of radio buttons
	in NS GUI sessions.  (Bug#15629)

512 513 514 515
2013-10-16  Dmitry Antipov  <dmantipov@yandex.ru>

	* fns.c (Fstring_as_unibyte): Use xlispstrdup.

516 517 518 519
2013-10-15  Paul Eggert  <eggert@cs.ucla.edu>

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

520 521 522 523 524 525 526
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.

527 528
2013-10-15  Paul Eggert  <eggert@cs.ucla.edu>

529 530 531 532 533 534 535
	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.

536 537
	* keyboard.c, keyboard.h (all_kboards): Now static.

538 539 540 541 542
2013-10-15  Xue Fuqiao  <xfq.free@gmail.com>

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

543 544 545 546 547 548 549
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.

550 551
2013-10-14  Eli Zaretskii  <eliz@gnu.org>

552 553 554 555
	* w32proc.c: Include mingw_time.h.

	* w32.c: Include mingw_time.h.

556 557 558 559 560 561 562 563 564 565 566 567 568 569
	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.

570 571 572 573 574 575 576 577 578 579 580 581 582 583 584 585 586 587 588
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.

589 590 591 592 593 594 595 596 597 598 599
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.

600 601 602 603 604 605 606 607 608
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.

609 610 611 612 613 614
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.

615 616 617 618 619 620 621 622 623 624
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.

625 626 627 628 629
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.

630 631 632 633 634 635
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.

636 637 638 639 640
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.

641 642 643 644 645 646 647
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.

648 649 650 651 652 653 654 655 656 657 658 659
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
660 661 662 663 664 665 666 667 668 669 670 671 672 673 674 675 676 677 678 679 680
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.

681 682 683 684
2013-10-11  Dmitry Antipov  <dmantipov@yandex.ru>

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

685 686 687 688 689 690
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.

691 692 693 694 695 696 697 698 699 700 701
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.

702 703 704 705 706 707
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.

708 709 710 711 712 713 714 715 716 717
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)

718 719 720 721 722 723 724 725 726 727 728 729 730 731
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.

732 733 734 735 736 737 738 739 740 741 742 743 744 745
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.

746 747 748 749 750
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)

751 752 753 754
2013-10-09  Paul Eggert  <eggert@cs.ucla.edu>

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

755 756 757 758 759 760 761
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.

762 763 764 765 766
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).

767 768
2013-10-09  Paul Eggert  <eggert@cs.ucla.edu>

769 770 771
	* intervals.c (temp_set_point_both): Move test into 'eassert',
	for speed.

772 773 774 775 776
	* 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.
777
	Problem reported by Dmitry Antipov in
778 779 780
	<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
781 782 783 784
2013-10-09  Glenn Morris  <rgm@gnu.org>

	* eval.c (Fcond): Doc tweak.

785 786 787 788 789
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)

790 791 792 793 794
2013-10-09  Stefan Monnier  <monnier@iro.umontreal.ca>

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

795 796 797 798 799 800 801 802 803 804 805 806 807 808 809 810 811 812 813 814 815 816 817 818 819 820
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.

821 822 823 824 825 826 827 828 829 830
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.

831 832
	* xdisp.c (deep_copy_glyph_row, display_tty_menu_item):
	New functions.
833 834 835 836 837 838 839 840 841 842 843 844 845 846 847 848 849 850 851 852 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 883 884 885 886 887 888 889 890 891 892 893 894 895 896

	* 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
897 898 899 900 901
	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
902 903 904 905 906 907 908 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
	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.

934 935 936 937 938 939 940 941 942
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.

943 944 945 946 947 948 949 950 951 952 953 954 955
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.

956 957 958 959 960
2013-10-08  Jan Djärv  <jan.h.d@swipnet.se>

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

961 962
2013-10-08  Paul Eggert  <eggert@cs.ucla.edu>

963 964 965 966
	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.

967 968 969 970
	* 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.
971 972 973
	* 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.
974

975 976 977 978 979 980 981 982 983 984 985 986 987 988 989 990 991 992 993
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.

994 995 996 997 998
2013-10-07  Jan Djärv  <jan.h.d@swipnet.se>

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

999 1000 1001
2013-10-07  Dmitry Antipov  <dmantipov@yandex.ru>

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

1004 1005 1006 1007 1008 1009 1010 1011 1012
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.

1013 1014 1015 1016 1017 1018 1019
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.

1020 1021 1022 1023 1024 1025
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).
1026 1027
	(ns_update_auto_hide_menu_bar): Remove runtime check.
	Don't auto hide dock unless menubar is also auto hidden.
1028

Xue Fuqiao's avatar
Xue Fuqiao committed
1029 1030 1031 1032
2013-10-05  Xue Fuqiao  <xfq.free@gmail.com>

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

1033 1034 1035 1036
2013-10-05  Jan Djärv  <jan.h.d@swipnet.se>

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

1037 1038 1039 1040 1041
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.

1042 1043
2013-10-04  Paul Eggert  <eggert@cs.ucla.edu>

1044 1045 1046 1047 1048 1049 1050 1051 1052
	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.

1053 1054 1055 1056
	* 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.

1057 1058
2013-10-03  Paul Eggert  <eggert@cs.ucla.edu>

1059 1060 1061 1062 1063
	* 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.

1064 1065
	* xdisp.c (Qglyphless_char): Now static.

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

1070 1071 1072 1073 1074 1075 1076 1077 1078
	* 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.

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 1107 1108
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.

1109 1110 1111 1112 1113
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.

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

1116 1117
	* macfont.m (mac_ctfont_create_preferred_family_for_attributes):
	Remove unused variable (from mac-port).
1118 1119
	(macfont_draw): Use s->ybase for correct y position.

1120 1121 1122 1123 1124 1125
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.

1126 1127 1128 1129 1130 1131 1132
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.

1133 1134 1135 1136 1137
2013-10-02  Dmitry Antipov  <dmantipov@yandex.ru>

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

1138 1139 1140 1141 1142
2013-10-01  Alp Aker  <alp.tekin.aker@gmail.com>

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

1143 1144 1145 1146 1147
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)

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

1150 1151
	* window.c (calc_absolute_offset): #elif should be #elif defined.

1152 1153 1154
	* frame.c (delete_frame): Block/unblock input to overcome race
	condition (Bug#15475).

1155 1156 1157 1158 1159
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).

1160 1161 1162 1163 1164 1165 1166 1167
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.

1168
	* nsterm.h (NS_SELECTION_BG_COLOR_DEFAULT): Rename from
1169 1170 1171
	NS_SELECTION_COLOR_DEFAULT.
	(NS_SELECTION_FG_COLOR_DEFAULT): New.

1172 1173 1174 1175 1176
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.

1177 1178 1179 1180 1181 1182 1183
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

1184 1185
2013-09-24  Paul Eggert  <eggert@cs.ucla.edu>

1186 1187 1188 1189
	* 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

1190 1191 1192 1193 1194 1195 1196 1197 1198 1199 1200
	* 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.

1201 1202 1203 1204 1205
2013-09-24  Daniel Colascione  <dancol@dancol.org>

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

1206 1207 1208 1209 1210 1211 1212 1213
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 *.

1214 1215 1216 1217 1218 1219 1220 1221 1222 1223 1224 1225 1226
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.

1227 1228 1229 1230 1231 1232 1233 1234 1235 1236
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.

1237 1238
2013-09-24  Paul Eggert  <eggert@cs.ucla.edu>

1239 1240 1241 1242 1243 1244 1245 1246 1247 1248 1249 1250 1251 1252
	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.

1253 1254
	* alloc.c (valgrind_p): Use bool for boolean.

1255 1256 1257 1258 1259 1260 1261
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.
1262 1263 1264
	* dispextern.h (cancel_line, init_desired_glyphs):
	Remove ancient leftover.
	(help_echo_showing_p, hourglass_shown_p):
1265 1266
	* lisp.h (noninteractive_need_newline): Adjust declaration.

1267 1268 1269 1270 1271 1272 1273 1274 1275 1276 1277 1278
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.

1279 1280
2013-09-23  Eli Zaretskii  <eliz@gnu.org>

1281 1282 1283
	* 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
1284 1285
	(note_mouse_highlight): Adapt calculation of last argument to
	mouse_face_from_string_pos to the above change.
1286

1287 1288 1289
	* conf_post.h (__has_builtin): Define to zero, if undefined, on
	all platforms, not just for clang.

1290 1291 1292 1293 1294 1295 1296
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.

1297 1298 1299 1300 1301
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.

1302 1303 1304 1305
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
1306
	(mark_maybe_object, mark_maybe_pointer)
1307 1308 1309 1310 1311
	[USE_VALGRIND]: Mark conservatively-scanned regions valid for
	valgrind purposes.
	(valgrind_p) [USE_VALGRIND]: New variable.
	(init_alloc) [USE_VALGRIND]: Initialize valgrind_p.

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

1314 1315 1316 1317 1318 1319 1320 1321 1322 1323
	* 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).

1324 1325
	* conf_post.h (assume): Fix compiler error: x shall be cond.

1326 1327 1328 1329 1330
2013-09-22  Daniel Colascione  <dancol@dancol.org>

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

1331 1332 1333
2013-09-22  Daniel Colascione  <dancol@dancol.org>

	* data.c (Qbool_vector_p): New symbol.
Paul Eggert's avatar
Paul Eggert committed
1334 1335
	(bool_vector_spare_mask, popcount_size_t_generic)
	(popcount_size_t_msc, popcount_size_t_gcc)
1336 1337
	(popcount_size_t)
	(bool_vector_binop_driver)
Paul Eggert's avatar
Paul Eggert committed
1338
	(count_trailing_zero_bits, size_t_to_host_endian)
1339 1340
	(Fbool_vector_exclusive_or)
	(Fbool_vector_union)
Paul Eggert's avatar
Paul Eggert committed
1341 1342
	(Fbool_vector_intersection, Fbool_vector_set_difference)
	(Fbool_vector_subsetp, Fbool_vector_not)
1343 1344 1345 1346 1347 1348 1349 1350
	(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
1351
	(vector_nbytes, sweep_vectors): Use bool_vector_payload_bytes
1352 1353 1354
	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
1355
	* casetab.c (shuffle, set_identity): Change lint_assume to assume.
1356 1357
	* composite.c (composition_gstring_put_cache):
	Change lint_assume to assume.
1358 1359 1360 1361 1362 1363 1364 1365
	* 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
1366 1367
	(CHECK_BOOL_VECTOR, ROUNDUP, BITS_PER_SIZE_T): New macros.
	(swap16, swap32, swap64): New inline functions.
1368 1369 1370 1371 1372
	* 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.

1373 1374 1375 1376 1377 1378
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)

1379 1380 1381 1382 1383 1384 1385 1386 1387 1388 1389 1390 1391 1392 1393 1394 1395 1396
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.

1397 1398 1399 1400 1401 1402
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
1403 1404
2013-09-20  Paul Eggert  <eggert@cs.ucla.edu>

1405 1406 1407 1408 1409
	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
1410 1411 1412 1413 1414 1415 1416 1417 1418 1419 1420 1421 1422 1423 1424 1425 1426 1427 1428 1429 1430 1431 1432 1433 1434 1435
	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.

1436 1437 1438 1439 1440
2013-09-20  Dmitry Antipov  <dmantipov@yandex.ru>

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

1441 1442 1443 1444 1445 1446
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.

1447 1448 1449 1450 1451 1452 1453 1454 1455 1456 1457 1458 1459 1460 1461 1462 1463 1464 1465 1466 1467 1468 1469 1470 1471 1472 1473
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.

1474 1475 1476 1477 1478 1479 1480 1481
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.

1482 1483 1484 1485 1486 1487 1488 1489
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.

1490 1491 1492 1493 1494 1495 1496 1497 1498 1499 1500 1501 1502 1503 1504 1505 1506 1507 1508 1509 1510 1511 1512
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.

1513 1514 1515 1516 1517 1518 1519 1520 1521 1522 1523 1524 1525 1526 1527 1528
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.

1529 1530 1531 1532 1533 1534
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.

1535 1536 1537 1538 1539 1540 1541 1542 1543 1544
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.

1545 1546 1547 1548 1549 1550 1551 1552 1553 1554 1555 1556
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.

1557 1558 1559 1560 1561 1562 1563 1564 1565 1566 1567
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.

1568 1569 1570 1571 1572
2013-09-17  Glenn Morris  <rgm@gnu.org>

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

1573 1574 1575 1576 1577 1578 1579 1580 1581 1582 1583 1584 1585 1586 1587 1588 1589 1590 1591 1592 1593
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.

1594 1595 1596 1597 1598 1599 1600
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.
1601 1602
	* xterm.c (handle_one_xevent): Do not call to x_clear_area
	if GTK >= 2.7.0.
1603
	(toplevel) [USE_MOTIF]: Include xlwmenu.h to pacify GCC.
1604

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

1607 1608 1609
	* 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
1610 1611 1612 1613
	* font.c (syms_of_font): Move call to syms_of_(ns|mac)font ...

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

1614 1615 1616 1617 1618 1619
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.

1620 1621 1622 1623 1624 1625 1626 1627 1628 1629 1630 1631 1632 1633 1634 1635 1636 1637 1638 1639 1640
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.