D19208: [platforms/x11/standalone] Port to AbstractOutput

zzag accepted this revision.
zzag added a comment.
This revision is now accepted and ready to land.


  Some style nitpicks. Given how intrusive this change is please wait for another "Ship it" from #kwin <https://phabricator.kde.org/tag/kwin/>.

INLINE COMMENTS

> x11_platform.cpp:38
>  #include "logging.h"
> +#include "x11_output.h"
>  #include "screens_xrandr.h"

Coding style nitpick: this include directive should be below line 45, i.e. below

  #include "x11_decoration_renderer.h"

> x11_platform.h:108
>  
> +    QVector<X11Output*> m_outputs;
>  };

Coding style nitpick: missing whitespace between the asterisk and X11Output

  QVector<X11Output *> m_outputs;

REPOSITORY
  R108 KWin

BRANCH
  outputX11

REVISION DETAIL
  https://phabricator.kde.org/D19208

To: romangg, #kwin, zzag
Cc: ngraham, zzag, kwin, LeGast00n, ericadams, jraleigh, GB_2, mkulinski, ragreen, jackyalcine, Pitel, iodelay, bwowk, ZrenBot, alexeymin, himcesjf, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, sebas, apol, mart