D9904: Wayland: don't share the same Config between backend and clients

2018-01-16 Thread Daniel Vrátil
This revision was automatically updated to reflect the committed changes. Closed by commit R110:4abdc5b134a3: Wayland: dont share the same Config between backend and clients (authored by dvratil). REPOSITORY R110 KScreen Library CHANGES SINCE LAST UPDATE

D9904: Wayland: don't share the same Config between backend and clients

2018-01-16 Thread Sebastian Kügler
sebas accepted this revision. sebas added a comment. This revision is now accepted and ready to land. LGTM as well. REPOSITORY R110 KScreen Library BRANCH master REVISION DETAIL https://phabricator.kde.org/D9904 To: dvratil, sebas, davidedmundson Cc: plasma-devel, ZrenBot, progwolff,

D9904: Wayland: don't share the same Config between backend and clients

2018-01-15 Thread Daniel Vrátil
dvratil added a comment. Indeed for out-of-process backends the config arrives serialized via DBus and a new copy which is then passed to a client is created during serialization. The change you mention is likely unrelated - it only mentions X11 which is always out-of-process, so not

D9904: Wayland: don't share the same Config between backend and clients

2018-01-15 Thread David Edmundson
davidedmundson added a comment. Looks sensible to me. To make sure I'm on the same page, out of process configs implicitly are copies of the one in the backend. So for consistent behaviour, in-process ones need to be explicitly cloned. Can you look if

D9904: Wayland: don't share the same Config between backend and clients

2018-01-15 Thread Daniel Vrátil
dvratil created this revision. dvratil added reviewers: sebas, davidedmundson. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. dvratil requested review of this revision. REVISION SUMMARY Sharing a single config leads to undesired