D23493: [server] Cache current mode

2019-08-28 Thread Roman Gilg
This revision was automatically updated to reflect the committed changes. romangg marked an inline comment as done. Closed by commit R127:ce5e25a4b63c: [server] Cache current mode (authored by romangg). REPOSITORY R127 KWayland CHANGES SINCE LAST UPDATE

D23493: [server] Cache current mode

2019-08-28 Thread Roman Gilg
romangg marked 2 inline comments as done. romangg added inline comments. INLINE COMMENTS > romangg wrote in outputdevice_interface.cpp:143 > Thanks. Will put it into a second diff. See D23535 REPOSITORY R127 KWayland BRANCH cacheMode REVISION DETAIL

D23493: [server] Cache current mode

2019-08-28 Thread Roman Gilg
romangg added inline comments. INLINE COMMENTS > davidedmundson wrote in outputdevice_interface.cpp:143 > we can also just use currentMode directly here and remove the loop above Thanks. Will put it into a second diff. REPOSITORY R127 KWayland BRANCH cacheMode REVISION DETAIL

D23493: [server] Cache current mode

2019-08-28 Thread David Edmundson
davidedmundson accepted this revision. davidedmundson added inline comments. This revision is now accepted and ready to land. INLINE COMMENTS > outputdevice_interface.cpp:143 > for (auto it = d->resources.constBegin(); it != > d->resources.constEnd(); ++it) { >

D23493: [server] Cache current mode

2019-08-27 Thread Roman Gilg
romangg added a dependent revision: D23495: Get output pixel size from output device. REPOSITORY R127 KWayland REVISION DETAIL https://phabricator.kde.org/D23493 To: romangg, #kwin Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns

D23493: [server] Cache current mode

2019-08-27 Thread Roman Gilg
romangg added a task: T11459: AbstractWaylandOutput and output device consolidation. REPOSITORY R127 KWayland REVISION DETAIL https://phabricator.kde.org/D23493 To: romangg, #kwin Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns

D23493: [server] Cache current mode

2019-08-27 Thread Roman Gilg
romangg created this revision. romangg added a reviewer: KWin. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. romangg requested review of this revision. REVISION SUMMARY Instead of looping over all modes for retrieving current mode data copy it to a