Re: address@hidden: Re: redisplay]

[Stefan Monnier]

> After some experimenting it seems that this patch fixes it.  Note that
> a call to redisplay() does not fix the bug.  I don't know why.  But a full
> redisplay before every dialog maybe is too much?

Am I wrong in thinking that Fredisplay(Qt) does not redraw every single
frame, but just does a normal redisplay (i.e. update the out-of-date
elements of the display), with the exception that it will complete the
redisplay even if there's input pending?
I.e. if the display is already up-to-date, Fredisplay(Qt) is a no-op?
If so, and it works, then your patch looks fine to me.


        Stefan


> diff -w -c src/xmenu.c.~1.317.~ src/xmenu.c
> Index: src/xmenu.c
> *** src/xmenu.c.~1.317.~        2007-01-21 11:33:57.000000000 +0100
> --- src/xmenu.c 2007-04-25 20:27:02.000000000 +0200
> ***************
> *** 3333,3338 ****
> --- 3333,3340 ----
>     /* No selection has been chosen yet.  */
>     menu_item_selection = 0;

> +   Fredisplay (Qt);
> +
>     /* Actually create and show the dialog.  */
>     create_and_show_dialog (f, first_wv);


>       Jan D.



> _______________________________________________
> Emacs-devel mailing list
> address@hidden
> http://lists.gnu.org/mailman/listinfo/emacs-devel