<div>graesslin requested changes to this revision.<br />
graesslin added a comment.<br />
This revision now requires changes to proceed.</div><br /><div><div><p>There are still a few unrelated changes included and some not-needed includes in header files</p></div></div><br /><div><strong>INLINE COMMENTS</strong><div><div>backends/drm/drm_buffer.h:24 I don't see QPointer used</div><div>backends/drm/drm_output.h:23 either forward declare (see below the forward declaration for DrmBackend) or include, not both :-) I assume that forward declaration should be sufficient here.</div><div>backends/drm/drm_pointer.h:23 QPointer not used in this file</div><div>backends/drm/scene_qpainter_drm_backend.cpp:54-58 still unrelated</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>