<div>graesslin added a comment.</div><br /><div><div><p>Implementation of the protocol looks good, but please see my comment on whether the interface has a correct semantic. I fear it cannot properly handle the case that the compositor produces several buffers before the client consumed them.</p>

<p>Please also add an auto test for the code, we try to have a complete test coverage for KWayland.</p></div></div><br /><div><strong>INLINE COMMENTS</strong><div><div>src/client/protocols/remoteaccess.xml:21 In Wayland it's common to use underscore names instead of camel case. Thus it would be buffer_no_longer_needed.<br />
<br />
Do we need the request at all or can we just use a deconstructor?<br />
<br />
Btw. I'm wondering how would the mapping between a buffer ready and a buffer no longer needed be done. It's possible that the producer provides more buffers than the consumer can consume.<br />
<br />
Maybe we need a dedicated remote-buffer interface for each one?</div></div></div><br /><div><strong>REPOSITORY</strong><div><div>rKWAYLAND KWayland</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D1231" rel="noreferrer">https://phabricator.kde.org/D1231</a></div></div><br /><div><strong>EMAIL PREFERENCES</strong><div><a href="https://phabricator.kde.org/settings/panel/emailpreferences/" rel="noreferrer">https://phabricator.kde.org/settings/panel/emailpreferences/</a></div></div><br /><div><strong>To: </strong>Kanedias, graesslin<br /><strong>Cc: </strong>plasma-devel, sebas<br /></div>