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

Ship it!


Much simpler indeed

- David Faure


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
> 
>

_______________________________________________
Kde-frameworks-devel mailing list
Kde-frameworks-devel@kde.org
https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Reply via email to