D15422: [server] Fix remote access buffer handling when output not bound

2018-09-11 Thread Roman Gilg
This revision was automatically updated to reflect the committed changes. Closed by commit R127:1faa629d2791: [server] Fix remote access buffer handling when output not bound (authored by romangg). REPOSITORY R127 KWayland CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D15422?vs=41423

D15422: [server] Fix remote access buffer handling when output not bound

2018-09-11 Thread Roman Gilg
romangg added a comment. At several places only one of two signal spies were waiting, what can lead to flickering tests. Fixed this with an if (spy.size() == 0) { spy.wait(); } on the second spy. Is there a nicer solution? REPOSITORY R127 KWayland BRANCH remo

D15422: [server] Fix remote access buffer handling when output not bound

2018-09-11 Thread Roman Gilg
romangg updated this revision to Diff 41423. romangg added a comment. - Some more cleanup of test. REPOSITORY R127 KWayland CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D15422?vs=41422&id=41423 BRANCH remoteBufferClientsUnbindOutput REVISION DETAIL https://phabricator.kde.o

D15422: [server] Fix remote access buffer handling when output not bound

2018-09-11 Thread Roman Gilg
romangg updated this revision to Diff 41422. romangg added a comment. Add autotest. Needed larger refactoring to allow multiple clients with different outputs being bound. REPOSITORY R127 KWayland CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D15422?vs=41400&id=41422 BRANCH re

D15422: [server] Fix remote access buffer handling when output not bound

2018-09-11 Thread Jan Grulich
jgrulich added a comment. Both versions solve issue I have and KWin no longer opens dozens of fds and abort when reaches the limit. REPOSITORY R127 KWayland REVISION DETAIL https://phabricator.kde.org/D15422 To: romangg, #kwin, #frameworks Cc: jgrulich, davidedmundson, kde-frameworks-de

D15422: [server] Fix remote access buffer handling when output not bound

2018-09-11 Thread David Edmundson
davidedmundson accepted this revision. This revision is now accepted and ready to land. REPOSITORY R127 KWayland BRANCH remoteBufferClientsUnbindOutput REVISION DETAIL https://phabricator.kde.org/D15422 To: romangg, #kwin, #frameworks, davidedmundson Cc: jgrulich, davidedmundson, kde-fram

D15422: [server] Fix remote access buffer handling when output not bound

2018-09-11 Thread Roman Gilg
romangg marked an inline comment as done. REPOSITORY R127 KWayland REVISION DETAIL https://phabricator.kde.org/D15422 To: romangg, #kwin, #frameworks Cc: davidedmundson, kde-frameworks-devel, michaelh, ngraham, bruns

D15422: [server] Fix remote access buffer handling when output not bound

2018-09-11 Thread Roman Gilg
romangg edited the summary of this revision. REPOSITORY R127 KWayland REVISION DETAIL https://phabricator.kde.org/D15422 To: romangg, #kwin, #frameworks Cc: davidedmundson, kde-frameworks-devel, michaelh, ngraham, bruns

D15422: [server] Fix remote access buffer handling when output not bound

2018-09-11 Thread Roman Gilg
romangg retitled this revision from "[server] Fix remote access buffer handlig when output not bound" to "[server] Fix remote access buffer handling when output not bound". REPOSITORY R127 KWayland REVISION DETAIL https://phabricator.kde.org/D15422 To: romangg, #kwin, #frameworks Cc: david