Re: address@hidden: Re: redisplay]

[Jan Djärv]

Jan Djärv skrev:
> Stefan Monnier skrev:
> > > 1.  SYNC_INPUT.  That way we won't get expose events in the middle of an
> > > ongoing redisplay.
> >
> > Hmm... I've been running with SYNC_INPUT for a long time now, and I'm 
> > pretty
> > sure that I still see those problems.  So maybe SYNC_INPUT would allow to
> > write a solution, but it's not a solution in itself.
>
> Yes, you are right.  It would also require to run redisplay directly on 
> the Expose event.

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?

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.