Re: [Stephen.Berman@gmx.net: Re: redisplay]

["Jan Djärv" <jan.h.d@swipnet.se>]

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.