davidedmundson added a comment.
I don't think we did go in there because we have a selection, otherwise other->selection() wouldn't be null. If we want DDI::sendSelection(DDI *other) to always keep the client in sync we should do: auto selection = other->selection(); if (!selection) { sendClearSelection(); return; } the send_selection spec says we should be passing a null resource if we have no selection, whereas this version creates a data offer with nothing in it. REPOSITORY R127 KWayland REVISION DETAIL https://phabricator.kde.org/D7078 To: graesslin, #kwin, #plasma, #frameworks Cc: davidedmundson, plasma-devel, leezu, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, eliasp, sebas, apol, mart, hein, lukas