romangg accepted this revision. romangg added a comment. This revision is now accepted and ready to land.
Looks fine to me. Tested runtime together with your KWayland patch. Probably you know this, but please push as one commit only to master. INLINE COMMENTS > egl_gbm_backend.cpp:160 > +{ > + if (qEnvironmentVariableIsSet("KWIN_NO_REMOTE")) > + return; Use braces: https://techbase.kde.org/Policies/Frameworks_Coding_Style#Braces REPOSITORY R108 KWin BRANCH fix-clang REVISION DETAIL https://phabricator.kde.org/D1230 To: Kanedias, graesslin, davidedmundson, romangg, #kwin Cc: kossebau, jgrulich, romangg, ngraham, alexeymin, aacid, kwin, #kwin, davidedmundson, plasma-devel, ragreen, schernikov, iodelay, zzag, bwowk, ZrenBot, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, eliasp, sebas, apol, mart, hein