emacs-pretest-bug
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: Contiguous redisplay of the menu and beeps


From: YAMAMOTO Mitsuharu
Subject: Re: Contiguous redisplay of the menu and beeps
Date: Sun, 27 May 2007 15:46:08 +0900
User-agent: Wanderlust/2.14.0 (Africa) SEMI/1.14.6 (Maruoka) FLIM/1.14.8 (Shijō) APEL/10.6 Emacs/22.0.990 (sparc-sun-solaris2.8) MULE/5.0 (SAKAKI)

>>>>> On Sun, 27 May 2007 00:45:55 +0200, martin rudalics <address@hidden> said:

> (2) A bug in redisplay.  Somewhere Qinhibit_point_motion_hooks
> doesn't get bound to Qt but so far I was not able to find out where.
> There's evidence that the bug shows up when `column-number-mode' is
> enabled, `current-column' is called and triggers sgml's
> point-entered hook.  Maybe it's got something to do with sgml's
> invisibility properties.  Kim would find this in five minutes, but
> where are thou ...

Unless someone who is familiar with this matter gives a better one,
I'd propose the following minimal and conservative change to avoid
regression at this stage.

                                     YAMAMOTO Mitsuharu
                                address@hidden

Index: src/indent.c
===================================================================
RCS file: /cvsroot/emacs/emacs/src/indent.c,v
retrieving revision 1.192
diff -c -p -r1.192 indent.c
*** src/indent.c        8 Apr 2007 23:59:19 -0000       1.192
--- src/indent.c        27 May 2007 06:37:11 -0000
*************** current_column_1 ()
*** 522,527 ****
--- 522,531 ----
    scan_newline (PT, PT_BYTE, BEGV, BEGV_BYTE, -1, 1);
    current_column_bol_cache = PT;
    scan = PT, scan_byte = PT_BYTE;
+   /* Restore point to the original value.  */
+   TEMP_SET_PT_BOTH (opoint, opoint_byte);
+   /* This might be unnecessary, but we leave it in order to avoid
+      regression.  */
    SET_PT_BOTH (opoint, opoint_byte);
    next_boundary = scan;




reply via email to

[Prev in Thread] Current Thread [Next in Thread]