Lars Gullik Bjønnes wrote:
> Peter Kümmel <[EMAIL PROTECTED]> writes:
> 
> | Peter Kümmel wrote:
> | > 
> | > And no keys will be dropped?
> | > 
> | > We could also put in your patch, as it is only a problem on Linux.
> | > Do you think we could use QApplication::synxX() instead of the X11
> | > call?
> | 
> | The Qt code:
> | 
> http://websvn.kde.org/*checkout*/trunk/qt-copy/src/gui/kernel/qapplication_x11.cpp?content-type=text%2Fplain
> | 
> | void QApplication::syncX()
> | {
> |     if (X11->display)
> |         XSync(X11->display, False);                        // don't discard 
> events
> | }
> | 
> | 
> | Seems we could use this function instead.
> 
> Did you test? It is not equal to the  XSync I use.

No, I didn't.

> False <-> True

OK, I thought it was also False in your patch.

> I am quite certain the True is required.

http://tronche.com/gui/x/xlib/event-handling/XSync.html

yes, true enables discarding.

> 
> And we still need the XEventsQueued.
> 


-- 
Peter Kümmel

Reply via email to