<div>graesslin added a comment.</div><br /><div><div><p>Thanks for doing the split!</p></div></div><br /><div><strong>INLINE COMMENTS</strong><div><div>backends/drm/drm_backend.cpp:50-52 please remove if no longer needed</div><div>backends/drm/drm_backend.h:55-57 why are the forward declarations removed? If that compiled before the split with forward declarations instead of includes that should still be possible</div><div>backends/drm/drm_buffer.cpp:20-21 Krazy would tell you to include own header first</div><div>backends/drm/drm_buffer.cpp:24 I don't see this one used</div><div>backends/drm/drm_buffer.cpp:25-27 I don't see this one used</div><div>backends/drm/drm_inputeventfilter.h:25 I don't see QPointer used her</div><div>backends/drm/drm_inputeventfilter.h:26 I don't see any QSize in this file</div><div>backends/drm/drm_output.cpp:274-275 Please don't push your local TODO commits</div><div>backends/drm/drm_output.cpp:597-599 similar</div><div>backends/drm/drm_pointer.h:23 no QPointer in this file, though I see a QScopedPointer</div><div>backends/drm/scene_qpainter_drm_backend.cpp:53-58 unrelated to the split</div></div></div><br /><div><strong>REPOSITORY</strong><div><div>rKWIN KWin</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D1168" rel="noreferrer">https://phabricator.kde.org/D1168</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>sebas, graesslin<br /><strong>Cc: </strong>plasma-devel<br /></div>