Re: Review Request 121197: KStartupInfo: use QX11Info::getTimestamp if appUserTime is 0

2014-11-24 Thread Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/121197/ --- (Updated Nov. 24, 2014, 9:09 a.m.) Review request for KDE Frameworks, kwi

Re: Review Request 121197: KStartupInfo: use QX11Info::getTimestamp if appUserTime is 0

2014-11-21 Thread Martin Gräßlin
> On Nov. 21, 2014, 4:21 nachm., Thomas Lübking wrote: > > src/kstartupinfo.cpp, line 1010 > > > > > > Should this not rather be ::getTimestamp() unconditionally? > > > > This is supposed to be a hint w

Re: Review Request 121197: KStartupInfo: use QX11Info::getTimestamp if appUserTime is 0

2014-11-21 Thread Martin Gräßlin
> On Nov. 21, 2014, 4:21 nachm., Thomas Lübking wrote: > > src/kstartupinfo.cpp, line 1010 > > > > > > Should this not rather be ::getTimestamp() unconditionally? > > > > This is supposed to be a hint w

Re: Review Request 121197: KStartupInfo: use QX11Info::getTimestamp if appUserTime is 0

2014-11-21 Thread Thomas Lübking
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/121197/#review70739 --- src/kstartupinfo.cpp

Review Request 121197: KStartupInfo: use QX11Info::getTimestamp if appUserTime is 0

2014-11-21 Thread Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/121197/ --- Review request for KDE Frameworks, kwin and Plasma. Repository: kwindowsy