Re: Contiguous redisplay of the menu and beeps

[martin rudalics]

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

It's too minimal.  The following

>     SET_PT_BOTH (opoint, opoint_byte);

still spoils everything on Windows.  Commenting out that line works,
though.  I continue to have no idea how current_column gets called with
Qinhibit_point_motion_hooks not bound to Qt here ...

Anyway, could people on Windows affected by this please try whether it
works with Yamamoto's change or the attached one?

*** indent.c    Mon Apr  9 09:24:48 2007
--- indent.c    Sun May 27 11:02:30 2007
***************
*** 522,528 ****
    scan_newline (PT, PT_BYTE, BEGV, BEGV_BYTE, -1, 1);
    current_column_bol_cache = PT;
    scan = PT, scan_byte = PT_BYTE;
!   SET_PT_BOTH (opoint, opoint_byte);
    next_boundary = scan;

    if (tab_width <= 0 || tab_width > 1000) tab_width = 8;
--- 522,533 ----
    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);
!   /* The following commented out to avoid that SET_PT_BOTH runs point-entered
!      hooks when displaying the mode-line.  If this forms a regression, restore
!      it and find out why these hooks are run during redisplay at all. */
!   /***   SET_PT_BOTH (opoint, opoint_byte); ***/
    next_boundary = scan;

    if (tab_width <= 0 || tab_width > 1000) tab_width = 8;