Wayland: don't update output configuration if there are no changes
ClosedPublic

Authored by dvratil on Jan 15 2018, 9:23 PM.

Details

Summary

This prevents the code from unnecesarilly emitting configChanged() if there
was in fact no change. This could lead to an endless loop when a client applies
current config and despite there not being no changes the backend emits
configChanged() to which the client reacts by applying the current config again
etc.

Diff Detail

Repository
R110 KScreen Library
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
dvratil created this revision.Jan 15 2018, 9:23 PM
Restricted Application added a project: Plasma. · View Herald TranscriptJan 15 2018, 9:23 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
dvratil requested review of this revision.Jan 15 2018, 9:23 PM
dvratil updated this revision to Diff 25428.Jan 15 2018, 9:30 PM

Fix commit message

sebas accepted this revision.Jan 16 2018, 4:08 PM

LGTM.

This revision is now accepted and ready to land.Jan 16 2018, 4:08 PM
This revision was automatically updated to reflect the committed changes.