Implement GUI display of R2L lines, fix TTY display of R2L lines.
[bpt/emacs.git] / src / ChangeLog
index 8859a10..481588e 100644 (file)
@@ -1,3 +1,458 @@
+2010-04-20  Eli Zaretskii  <eliz@gnu.org>
+
+       Fix R2L paragraph display on TTY.
+
+       * xdisp.c (unproduce_glyphs): New function.
+       (display_line): Use it when produced glyphs are discarded from R2L
+       glyph rows.
+       (append_composite_glyph): In R2L rows, prepend the glyph rather
+       than appending it.
+
+       * term.c (append_composite_glyph): In R2L rows, prepend the glyph
+       rather than append it.  Set up the resolved_level and bidi_type
+       attributes of the appended glyph.
+       (produce_special_glyphs): Mirror the backslash continuation
+       character in R2L lines.
+
+       Implement display of R2L paragraphs in GUI sessions.
+
+       * xdisp.c [HAVE_WINDOW_SYSTEM]: Add prototype for
+       append_stretch_glyph.
+       (set_cursor_from_row) <cursor_x>: Remove unused variable.  Fix
+       off-by-one error in computing x at end of text in the row.
+       (append_stretch_glyph): In reversed row, prepend the glyph rather
+       than append it.  Set resolved_level and bidi_type of the glyph.
+       (extend_face_to_end_of_line): If the row is reversed, prepend a
+       stretch glyph whose width is such that the rightmost glyph will be
+       drawn at the right margin of the window.  Fix off-by-one error on
+       TTY frames in testing whether a line needs face extension.  Fix
+       face extension at ZV.  If this is the last glyph row, use
+       DEFAULT_FACE_ID, to avoid painting the rest of the window with the
+       region face.
+       (set_cursor_from_row, display_line): Use
+       MATRIX_ROW_CONTINUATION_LINE_P instead of testing value of
+       row->continuation_lines_width.
+       (next_element_from_buffer): Don't call bidi_paragraph_init if we
+       are at ZV.  Fixes a crash when reseated to ZV by
+       try_window_reusing_current_matrix.
+       (display_and_set_cursor, erase_phys_cursor): Handle negative HPOS,
+       which happens with R2L glyph rows.  Fixes a crash when inserting a
+       character at end of an R2L line.
+       (set_cursor_from_row): Don't be fooled by truncated rows: don't
+       treat them as having zero-width characters.  Improve comments.
+       Don't reverse pos_before and pos_after for reversed glyph rows.
+       Set cursor.x to negative value when the cursor might be on the
+       left fringe.
+       (IT_OVERFLOW_NEWLINE_INTO_FRINGE): For R2L lines, consider the
+       left fringe, not the right one.
+       (notice_overwritten_cursor, draw_phys_cursor_glyph)
+       (erase_phys_cursor): For reversed cursor_row, support cursor on
+       the left fringe.
+
+       * fringe.c (update_window_fringes): For R2L rows, swap the bitmaps
+       of continuation indicators on the fringes.
+       (draw_fringe_bitmap): For reversed glyph rows, allow cursor on the
+       left fringe.
+
+       * w32term.c (w32_draw_window_cursor): For reversed glyph rows,
+       draw cursor on the left fringe.
+
+       * xterm.c (x_draw_window_cursor): For reversed glyph rows, draw
+       cursor on the left fringe.
+
+       * dispnew.c (update_text_area): Handle reversed desired rows when
+       the cursor is on the left fringe.
+       (set_window_cursor_after_update): Limit cursor's hpos by -1 from
+       below, not by 0, for when the cursor is on the left fringe.
+
+2010-04-20  Jan Djärv  <jan.h.d@swipnet.se>
+
+       * gtkutil.c (xg_event_is_for_scrollbar): Check if grabbed
+       widget is a scrollbar.
+
+2010-04-20  Kenichi Handa  <handa@m17n.org>
+
+       * charset.c (char_charset): Consider Vcharset_non_preferred_head
+       only when the arg CHARSET_LIST is nil.
+
+2010-04-20  Stefan Monnier  <monnier@iro.umontreal.ca>
+
+       Make variable forwarding explicit rather the using special values.
+       Basically, this makes the structure of buffer-local values and object
+       forwarding explicit in the type of Lisp_Symbols rather than use
+       special Lisp_Objects for that.  This tends to lead to slightly more
+       verbose code, but is more C-like, simpler, and makes it easier to make
+       sure we handled all cases, among other things by letting the compiler
+       help us check it.
+       * lisp.h (enum Lisp_Misc_Type, union Lisp_Misc):
+       Removing forwarding objects.
+       (enum Lisp_Fwd_Type, enum symbol_redirect, union Lisp_Fwd): New types.
+       (struct Lisp_Symbol): Make the various forms of variable-forwarding
+       explicit rather than hiding them inside Lisp_Object "values".
+       (XFWDTYPE): New macro.
+       (XINTFWD, XBOOLFWD, XOBJFWD, XKBOARD_OBJFWD): Redefine.
+       (XBUFFER_LOCAL_VALUE): Remove.
+       (SYMBOL_VAL, SYMBOL_ALIAS, SYMBOL_BLV, SYMBOL_FWD, SET_SYMBOL_VAL)
+       (SET_SYMBOL_ALIAS, SET_SYMBOL_BLV, SET_SYMBOL_FWD): New macros.
+       (SYMBOL_VALUE, SET_SYMBOL_VALUE): Remove.
+       (struct Lisp_Intfwd, struct Lisp_Boolfwd, struct Lisp_Objfwd)
+       (struct Lisp_Buffer_Objfwd, struct Lisp_Kboard_Objfwd):
+       Remove the Lisp_Misc_* header.
+       (struct Lisp_Buffer_Local_Value): Redefine.
+       (BLV_FOUND, SET_BLV_FOUND, BLV_VALUE, SET_BLV_VALUE): New macros.
+       (struct Lisp_Misc_Any): Add filler to get the right size.
+       (struct Lisp_Free): Use struct Lisp_Misc_Any rather than struct
+       Lisp_Intfwd.
+       (DEFVAR_LISP, DEFVAR_LISP_NOPRO, DEFVAR_BOOL, DEFVAR_INT)
+       (DEFVAR_KBOARD): Allocate a forwarding object.
+       * data.c (do_blv_forwarding, store_blv_forwarding): New macros.
+       (let_shadows_global_binding_p): New function.
+       (union Lisp_Val_Fwd): New type.
+       (make_blv): New function.
+       (swap_in_symval_forwarding, indirect_variable, do_symval_forwarding)
+       (store_symval_forwarding, swap_in_global_binding, Fboundp)
+       (swap_in_symval_forwarding, find_symbol_value, Fset)
+       (let_shadows_buffer_binding_p, set_internal, default_value)
+       (Fset_default, Fmake_variable_buffer_local, Fmake_local_variable)
+       (Fkill_local_variable, Fmake_variable_frame_local)
+       (Flocal_variable_p, Flocal_variable_if_set_p)
+       (Fvariable_binding_locus):
+       * xdisp.c (select_frame_for_redisplay):
+       * lread.c (Fintern, Funintern, init_obarray, defvar_int)
+       (defvar_bool, defvar_lisp_nopro, defvar_lisp, defvar_kboard):
+       * frame.c (store_frame_param):
+       * eval.c (Fdefvaralias, Fuser_variable_p, specbind, unbind_to):
+       * bytecode.c (Fbyte_code) <varref, varset>: Adapt to the new symbol
+       value structure.
+       * buffer.c (PER_BUFFER_SYMBOL): Move from buffer.h.
+       (clone_per_buffer_values): Only adjust markers into the current buffer.
+       (reset_buffer_local_variables): PER_BUFFER_IDX is never -2.
+       (Fbuffer_local_value, set_buffer_internal_1)
+       (swap_out_buffer_local_variables):
+       Adapt to the new symbol value structure.
+       (DEFVAR_PER_BUFFER): Allocate a Lisp_Buffer_Objfwd object.
+       (defvar_per_buffer): Take a new arg for the fwd object.
+       (buffer_lisp_local_variables): Return a proper alist (different fix
+       for bug#4138).
+       * alloc.c (Fmake_symbol): Use SET_SYMBOL_VAL.
+       (Fgarbage_collect): Don't handle buffer_defaults specially.
+       (mark_object): Handle new symbol value structure rather than the old
+       special Lisp_Misc_* objects.
+       (gc_sweep) <symbols>: Free also the buffer-local-value objects.
+       * term.c (set_tty_color_mode):
+       * bidi.c (bidi_initialize): Don't access the ->value field directly.
+       * buffer.h (PER_BUFFER_VAR_OFFSET): Don't bother with
+       a buffer_local_flags.
+       * print.c (print_object): Get rid of impossible forwarding objects.
+
+2010-04-19  Eli Zaretskii  <eliz@gnu.org>
+
+       * bidi.c (bidi_get_type, bidi_get_category)
+       (bidi_at_paragraph_end, bidi_resolve_weak, bidi_resolve_neutral)
+       (bidi_type_of_next_char, bidi_level_of_next_char):
+       Declare static.  Use `INLINE' rather than `inline'.
+
+2010-04-19  Juanma Barranquero  <lekktu@gmail.com>
+
+       * dired.c (Ffile_attributes): Fix typo in docstring.
+
+2010-04-19  Adrian Robert  <Adrian.B.Robert@gmail.com>
+
+       * nsmenu.m (EmacsDialog-runDialogAt:): Declare ret as
+       NSInteger (Bug#5811).
+
+2010-04-19  YAMAMOTO Mitsuharu  <mituharu@math.s.chiba-u.ac.jp>
+
+       * s/darwin.h (PTY_ITERATION, PTY_NAME_SPRINTF, PTY_TTY_NAME_SPRINTF)
+       (PTY_OPEN): New defines.  Use openpty (Bug#726, Bug#5819).
+
+2010-04-19  YAMAMOTO Mitsuharu  <mituharu@math.s.chiba-u.ac.jp>
+
+       * frame.h (FRAME_LINE_TO_PIXEL_Y): Add missing parenthesis.
+
+2010-04-19  Jan Djärv  <jan.h.d@swipnet.se>
+
+       * frame.h (FRAME_TEXT_LINES_TO_PIXEL_HEIGHT): Don't use
+       FRAME_LINE_TO_PIXEL_Y.
+
+       * xterm.c (x_set_window_size_1): Don't add border_width/height to
+       pixelwidth/height.
+
+2010-04-19  Chong Yidong  <cyd@stupidchicken.com>
+
+       * xdisp.c (prepare_menu_bars): Don't call ns_set_doc_edited for
+       terminal frames (Bug#5837).
+
+2010-04-19  Eli Zaretskii  <eliz@gnu.org>
+
+       * .gdbinit (xsubchartable): New command.
+
+2010-04-19  Eli Zaretskii  <eliz@gnu.org>
+
+       * xdisp.c (display_line): Don't write beyond the last glyph row in
+       the desired matrix.  Fixes a crash in "emacs -nw" (bug#5972), see
+       http://lists.gnu.org/archive/html/emacs-devel/2010-04/msg00075.html
+       and
+       http://lists.gnu.org/archive/html/emacs-devel/2010-04/msg00213.html
+
+2010-04-18  Stefan Monnier  <monnier@iro.umontreal.ca>
+
+       * alloc.c (Fpurecopy): Hash-cons if requested.
+       (syms_of_alloc): Update purify-flag docstring.
+
+2010-04-18  Jan Djärv  <jan.h.d@swipnet.se>
+
+       * gtkutil.c (xg_set_geometry): Set size in geometry string also.
+       (x_wm_set_size_hint): Set USER_POS in hint_flags (Bug#5968).
+
+2010-04-17  Eli Zaretskii  <eliz@gnu.org>
+
+       Fix a crash when an NSM character is inserted at BEGV.
+
+       * bidi.c (bidi_init_it): Fix initialization of bidi_it->prev.
+       (bidi_resolve_weak): Don't use prev.type_after_w1 if it is
+       NEUTRAL_B or UNKNOWN_BT.
+
+2010-04-16  Eli Zaretskii  <eliz@gnu.org>
+
+       * xdisp.c (set_cursor_from_row): Don't consider possibility of
+       other rows with cursor unless they are different from this row and
+       this row is part of a continued line.  (Bug#5943)
+
+2010-04-16  Dan Nicolaescu  <dann@ics.uci.edu>
+
+       * s/freebsd.h: Restore osreldate.h include.
+       Suggested by Naohiro Aota.
+
+2010-04-16  Jan Djärv  <jan.h.d@swipnet.se>
+
+       * xmenu.c (apply_systemfont_to_menu): *childs was incorrectly used.
+
+2010-04-16  Ken Brown  <kbrown@cornell.edu>  (tiny change)
+
+       * s/cygwin.h: Avoid linking against static libgcc.
+
+2010-04-15  Juri Linkov  <juri@jurta.org>
+
+       * window.c: Add Qscroll_command.
+       Remove Vscroll_preserve_screen_position_commands.
+       (window_scroll_pixel_based, window_scroll_line_based): Check the
+       `scroll-command' property on the last command instead of searching
+       the last command in Vscroll_preserve_screen_position_commands.
+       (syms_of_window): Initialize and staticpro `Qscroll_command'.
+       Put Qscroll_command property on Qscroll_up and Qscroll_down.
+       (scroll-preserve-screen-position): Doc fix.
+       (Vscroll_preserve_screen_position_commands): Remove variable.
+
+2010-04-15  Dan Nicolaescu  <dann@ics.uci.edu>
+
+       * xdisp.c (message): Do not use NO_ARG_ARRAY.
+
+2010-04-14  Dan Nicolaescu  <dann@ics.uci.edu>
+
+       Reduce cpp use in Makefile.in.
+       * Makefile.in (DBUS_CFLAGS, DBUS_LIBS, GCONF_CFLAGS, GCONF_LIBS)
+       (LIBSOUND, CFLAGS_SOUND, RSVG_LIBS, RSVG_CFLAGS, INTERVALS_H)
+       (GETLOADAVG_LIBS, RUN_TEMACS): Move to the autoconf section.
+       (ORDINARY_LINK): Remove, defined in src/s/gnu.h.
+       (CRT0_COMPILE): Remove, inline it in the only user.
+
+2010-04-14  Juri Linkov  <juri@jurta.org>
+
+       * window.c (keys_of_window): Rebind `C-v' from `scroll-up' to
+       `scroll-up-command' and `M-v' from `scroll-down' to
+       `scroll-down-command'.
+
+2010-04-14  Juri Linkov  <juri@jurta.org>
+
+       * window.c (Vscroll_preserve_screen_position_commands): New variable
+       with the default value as the list of Qscroll_down and Qscroll_up.
+       (window_scroll_pixel_based, window_scroll_line_based): Search the
+       last command in the list Vscroll_preserve_screen_position_commands
+       instead of comparing with Qscroll_up and Qscroll_down.
+
+2010-04-13  Jan Djärv  <jan.h.d@swipnet.se>
+
+       * gtkutil.c (xg_set_geometry): Set geometry for PPosition also.
+       (x_wm_set_size_hint): Dont set position flags, gtk_window_parse_geometry
+       does that.
+
+       * xfns.c (Fx_create_frame, x_create_tip_frame): Set default border width
+       to zero.
+
+2010-04-13  Stefan Monnier  <monnier@iro.umontreal.ca>
+
+       * term.c (init_tty): Move common text outside of #ifdef TERMINFO.
+
+       Try to solve the problem of spurious EOF chars in long lines of text
+       sent to interactive subprocesses.
+       * sysdep.c (child_setup_tty): Do not enable ICANON any more.
+       (system_process_attributes): Remove unused var `ttotal'.
+       * process.c (send_process): Don't bother breaking long line with EOF
+       chars when talking to ttys any more.
+       (wait_reading_process_output): Output a warning when called in such
+       a way that it could block without being interruptible.
+
+       Try to detect file modification within the same second.
+       * buffer.h (struct buffer): New field modtime_size.
+       * buffer.c (reset_buffer): Initialize it.
+       * fileio.c (Finsert_file_contents, Fwrite_region): Set it.
+       (Fverify_visited_file_modtime): Check it.
+       (Fclear_visited_file_modtime, Fset_visited_file_modtime): Clear it.
+       (Fset_visited_file_modtime): Set (or clear) it.
+
+2010-04-12  Stefan Monnier  <monnier@iro.umontreal.ca>
+
+       * process.c (status_notify): Remove unused var `ro'.
+
+2010-04-12  Jan Djärv  <jan.h.d@swipnet.se>
+
+       * xfns.c (select_visual): Don't call error if XGetVisualInfo returns
+       more than one visual (Bug#5938).
+
+2010-04-12  Dan Nicolaescu  <dann@ics.uci.edu>
+
+       * Makefile.in (C_SWITCH_SYSTEM,C_SWITCH_MACHINE,C_SWITCH_X_SITE): Undefine.
+
+2010-04-11  Dan Nicolaescu  <dann@ics.uci.edu>
+
+       Remove C_SWITCH_SYSTEM_TEMACS.
+       * s/darwin.h (C_SWITCH_SYSTEM_TEMACS): Remove.
+       (malloc, realloc, free): Use emacs, not temacs for conditional
+       definition.
+
+       * Makefile.in (C_SWITCH_SYSTEM_TEMACS): Remove.
+       (ALL_CFLAGS): Do not use C_SWITCH_SYSTEM_TEMACS.
+
+       Use autoconf, not cpp for some variables.
+       * Makefile.in (C_SWITCH_SYSTEM, C_SWITCH_MACHINE)
+       (C_SWITCH_X_SITE): Define using autoconf, not cpp.
+       (ALL_CFLAGS): Use them as make variables.
+       (really-lwlib, really-oldXMenu): Do not pass them.
+
+2010-04-11  Jan Djärv  <jan.h.d@swipnet.se>
+
+       * xmenu.c (apply_systemfont_to_dialog): New.
+       (create_and_show_dialog): Call apply_systemfont_to_dialog if HAVE_XFT.
+
+2010-04-11  Stefan Monnier  <monnier@iro.umontreal.ca>
+
+       * process.c (exec_sentinel): Preserve current-buffer.
+
+       * process.c (read_process_output): Move the save-current-buffer to
+       apply to both the filter and the non-filter branches.
+
+2010-04-10  Dan Nicolaescu  <dann@ics.uci.edu>
+
+       * s/msdos.h (UNEXEC): New definition.
+
+2010-04-10  YAMAMOTO Mitsuharu  <mituharu@math.s.chiba-u.ac.jp>
+
+       * dispextern.h (TRY_WINDOW_CHECK_MARGINS)
+       (TRY_WINDOW_IGNORE_FONTS_CHANGE): New defines.
+
+       * xdisp.c (try_window): Change arg from CHECK_MARGINS to FLAGS.
+       Don't abort with fonts change if TRY_WINDOW_IGNORE_FONTS_CHANGE is
+       set in FLAGS.  Callers with non-zero CHECK_MARGINS changed to use
+       TRY_WINDOW_CHECK_MARGINS.
+
+       * xfns.c (Fx_show_tip): Undo last change.  Call try_window with
+       TRY_WINDOW_IGNORE_FONTS_CHANGE (Bug#2423).  Subtract last glyph's
+       width only when it is for padding.
+
+2010-04-09  Jan Djärv  <jan.h.d@swipnet.se>
+
+       * xfns.c (Fx_show_tip): Call try_window in a loop until
+       fonts_changed_p is zero (Bug#2423).
+
+2010-04-08  Eli Zaretskii  <eliz@gnu.org>
+
+       * xdisp.c (set_cursor_from_row): Don't dereference glyphs beyond
+       the end of TEXT_AREA.  (Bug#5856)
+
+2010-04-08  Jan Djärv  <jan.h.d@swipnet.se>
+
+       * xsettings.c (XSETTINGS_FONT_NAME): Move XSETTINGS_FONT_NAME out of
+       HAVE_GCONF.
+
+2010-04-08  Eli Zaretskii  <eliz@gnu.org>
+
+       * bidi.c (bidi_resolve_weak): Use prev.type_after_w1, instead of
+       prev.orig_type, for resolving type of NSM.  (Bug#5858)
+
+2010-04-08  Jan Djärv  <jan.h.d@swipnet.se>
+
+       * xsettings.c (current_font, SYSTEM_FONT, XSETTINGS_FONT_NAME): New.
+       (parse_xft_settings): Also check for XSETTINGS_FONT_NAME and save that
+       in current_font.
+       (init_gconf): Read value of SYSTEM_FONT and save it in current_font.
+       (Ffont_get_system_normal_font, xsettings_get_system_normal_font): New
+       functions.
+       (syms_of_xsettings): Initialize current_font. defsubr
+       Sfont_get_system_normal_font.
+
+       * xsettings.h (Ffont_get_system_normal_font,
+       xsettings_get_system_normal_font): Declare.
+
+       * xfns.c (extern xlwmenu_default_font): Remove.
+       (Fx_create_frame): Remove setting of xlwmenu_default_font, moved
+       to xlwmenu.c.
+
+       * menu.c (digest_single_submenu): If USE_LUCID and HAVE_XFT, encode
+       menu items in UTF-8.
+
+       * xmenu.c: include xsettings.h and xlwmenu.h if USE_LUCID.
+       (apply_systemfont_to_menu): New function.
+       (set_frame_menubar, create_and_show_popup_menu): Call
+       apply_systemfont_to_menu.
+
+2010-04-07  Jan Djärv  <jan.h.d@swipnet.se>
+
+       * frame.h (FRAME_TEXT_LINES_TO_PIXEL_HEIGHT): Don't use
+       FRAME_LINE_TO_PIXEL_Y.
+
+       * xterm.c (x_set_window_size_1): Don't add border_width/height to
+       pixelwidth/height.
+
+2010-04-07  Dan Nicolaescu  <dann@ics.uci.edu>
+
+       Simplify code for HP machines.
+       * m/hp800.h (LOAD_AVE_TYPE, LOAD_AVE_CVT, NO_REMAP): Do not define
+       for GNU_LINUX, not needed.
+       (UNEXEC, NEED_BSDTTY): Move definitions...
+       * s/hpux10-20.h (UNEXEC, NEED_BSDTTY): ... here.
+
+       * m/iris4d.h (UNEXEC): Move definition ...
+       * s/irix6-5.h (UNEXEC): ... here.
+
+2010-04-04  Jan Djärv  <jan.h.d@swipnet.se>
+
+       * xfns.c (set_machine_and_pid_properties): New function.
+       (Fx_create_frame): Call set_machine_and_pid_properties.
+
+2010-04-03  Eli Zaretskii  <eliz@gnu.org>
+
+       * bidi.c (bidi_resolve_explicit, bidi_level_of_next_char): Check
+       bidi_it->bytepos against ZV_BYTE instead of bidi_it->ch against
+       BIDI_EOB.  Fixes infloop with vertical cursor motion at ZV.
+
+       * w32fns.c (x_create_tip_frame): Copy `parms' before we modify it
+       in this function.  (Bug#5703)
+
+2010-04-03  Chong Yidong  <cyd@stupidchicken.com>
+
+       * nsterm.h: Fix last change.
+
+2010-04-03  Dan Nicolaescu  <dann@ics.uci.edu>
+
+       * m/intel386.h (NO_REMAP): Move definition ...
+       * s/msdos.h (NO_REMAP): ... here.
+
+       * m/vax.h (CRT0_DUMMIES): Remove, unused.
+
+       * ecrt0.c: Remove MSDOS, m68k and __sparc__ conditionals, file not
+       used on those platforms.
+
 2010-04-02  Dan Nicolaescu  <dann@ics.uci.edu>
 
        Remove extern errno declarations.
        (gif_load): Put GIF extension data to the property
        `Qextension_data'.
 
+2010-04-05  Chong Yidong  <cyd@stupidchicken.com>
+
+       * xdisp.c (prepare_menu_bars): Don't call ns_set_doc_edited for
+       terminal frames (Bug#5837).
+
 2010-03-31  Chong Yidong  <cyd@stupidchicken.com>
 
        * nsfns.m (ns_set_doc_edited): Remove unused arg OLDVAL.
 
        * xdisp.c (x_consider_frame_title, update_window_cursor):
        Remove HAVE_NS conditionals.
-       (prepare_menu_bars)[HAVE_NS]: Call ns_set_doc_edited.
+       (prepare_menu_bars) [HAVE_NS]: Call ns_set_doc_edited.
 
        * nsfns.m (x_implicitly_set_name): If frame-title-format is t, use
        filename for the title.
 2010-03-12  Eli Zaretskii  <eliz@gnu.org>
 
        These changes remove termcap.c from the build on Posix platforms.
-       * Makefilie.in (termcapobj): Move termcap.o from here...
+       * Makefile.in (termcapobj): Move termcap.o from here...
        (MSDOS_OBJ): ...to here.
        (termcapobj) [!LIBS_TERMCAP]: Remove specialized value, as it is
        now identical to when LIBS_TERMCAP is defined.
 2010-01-19  Alan Mackenzie  <acm@muc.de>
 
        Fix spurious before-change-functions invocation from (insert ?\n).
-       * textprop.c (set_text_properties): rename parameter
+       * textprop.c (set_text_properties): Rename parameter
        `signal_after_change_p' to `coherent_change_p', and make the
        invocation of `modify_region' conditional on it.
 
 
        * font.c (Vfont_encoding_alist, find_font_encoding): Move from
        fontset.c.
-       (font_pixel_size)[! HAVE_WINDOW_SYSTEM]: Return 1.
+       (font_pixel_size) [! HAVE_WINDOW_SYSTEM]: Return 1.
        (font_open_entity): Update FRAME_X_DISPLAY_INFO (f)->n_fonts,
        FRAME_SMALLEST_CHAR_WIDTH (f), and FRAME_SMALLEST_FONT_HEIGHT (f)
        only when HAVE_WINDOW_SYSTEM is defined.
 
 2007-08-29  ARISAWA Akihiro  <ari@mbf.ocn.ne.jp>  (tiny change)
 
-       * term.c (dissociate_if_controlling_tty)[USG]: Fix parse error.
+       * term.c (dissociate_if_controlling_tty) [USG]: Fix parse error.
 
 2007-08-29  Yoshiaki Kasahara  <kasahara@nc.kyushu-u.ac.jp>  (tiny change)