<div>sebas accepted this revision.<br />
sebas added a reviewer: sebas.<br />
sebas added a comment.<br />
This revision is now accepted and ready to land.</div><br /><div><div><p>I read over it, and as far as I understand what's the problem, your patches look like they make sense.</p></div></div><br /><div><strong>INLINE COMMENTS</strong><div><div>autotests/client/test_wayland_subsurface.cpp:220 use the function pointer version of QSignalSpy?</div><div>autotests/client/test_wayland_subsurface.cpp:589 its pending state</div><div>src/server/surface_interface.cpp:243 that's not a "factor", perhaps just "transformChanged"?</div><div>src/server/surface_interface.cpp:243 whitespace after &&<br />
<br />
I generally prefer to put parentheses around these right hand sides of the bool assignment, btw. More like a personal preference, which makes it a bit easier to read.</div></div></div><br /><div><strong>REPOSITORY</strong><div><div>rKWAYLAND KWayland</div></div></div><br /><div><strong>BRANCH</strong><div><div>subsurface-fixes</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D1191" rel="noreferrer">https://phabricator.kde.org/D1191</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>graesslin, Plasma, sebas<br /><strong>Cc: </strong>sebas, plasma-devel, Plasma<br /></div>