Review Request 112684: Don't inspect XEvent queue in KUserTimestamp::updateUserTimestamp
Aurélien Gâteau
agateau at kde.org
Thu Sep 12 14:05:17 UTC 2013
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/112684/#review39882
-----------------------------------------------------------
Looks good to me, but I don't feel qualified to press the "Ship It" button.
- Aurélien Gâteau
On Sept. 12, 2013, 3:30 p.m., Martin Gräßlin wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/112684/
> -----------------------------------------------------------
>
> (Updated Sept. 12, 2013, 3:30 p.m.)
>
>
> Review request for KDE Frameworks, David Faure and Aurélien Gâteau.
>
>
> Description
> -------
>
> updateUserTimestamp() used to fetch the current time by creating a property notify event and wait till it is received by inspecting the XEvent queue. That doesn't work with XCB anymore, but Qt provides this functionality for us, so just use that.
>
> At the same time the argument is changed from int to unsigned long as that's what the method in QX11Info returns.
>
>
> Diffs
> -----
>
> staging/kinterprocesswindowing/src/kusertimestamp.h 004def5
> staging/kinterprocesswindowing/src/kusertimestamp.cpp 950579f
>
> Diff: http://git.reviewboard.kde.org/r/112684/diff/
>
>
> Testing
> -------
>
> compiles.
>
>
> Thanks,
>
> Martin Gräßlin
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20130912/d537bb8f/attachment-0001.html>
More information about the Kde-frameworks-devel
mailing list