This revision was automatically updated to reflect the committed changes.
Closed by commit R127:2b3f9509ac12: Only send OutputConfig sendApplied /
sendFailed to the right resource (authored by davidedmundson).
REPOSITORY
R127 KWayland
CHANGES SINCE LAST UPDATE
sebas added a comment.
FWIW, I tested it, it fixes kwin crashing when adding a setApplied() call on
the OutputConfiguration, and it also means the applied signal arrives.
REPOSITORY
R127 KWayland
BRANCH
master
REVISION DETAIL
https://phabricator.kde.org/D7898
To: davidedmundson,
sebas accepted this revision.
sebas added a comment.
This revision is now accepted and ready to land.
Yes, the initial idea was to send applied() to all connected resources. That
is not s_allResources, so your patch is correct. The resources that are
actually bound are not currently tracked.
davidedmundson created this revision.
davidedmundson added a reviewer: Plasma.
Restricted Application added projects: Plasma on Wayland, Frameworks.
Restricted Application added subscribers: Frameworks, plasma-devel.
REVISION SUMMARY
I don't understand the original logic.
s_allResources