Review Request 112684: Don't inspect XEvent queue in KUserTimestamp::updateUserTimestamp

[Commit Hook]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/112684/#review39967
-----------------------------------------------------------


This review has been submitted with commit 0be299aa017f717926fc0f342fc13d217c2a5353 by Martin Gräßlin to branch frameworks.

- Commit Hook


On Sept. 12, 2013, 1: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, 1: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/20130913/16502748/attachment.html>