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

Reply via email to