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
