ekurzinger updated this revision to Diff 51709. ekurzinger added a comment.
Thanks for taking a look, Roman. I've addressed your style suggestions and added support for passing stream attributes to the compositor. I had ignored these initially since the driver doesn't actually use this (fairly new) feature of the extension right now, but it probably is best to support it for future-proofness. REPOSITORY R127 KWayland CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D18824?vs=51123&id=51709 REVISION DETAIL https://phabricator.kde.org/D18824 AFFECTED FILES src/client/protocols/wayland-eglstream-controller.xml src/server/CMakeLists.txt src/server/display.cpp src/server/display.h src/server/eglstream_controller_interface.cpp src/server/eglstream_controller_interface.h src/server/eglstream_controller_interface_p.h To: ekurzinger, romangg, davidedmundson, zzag, #kwin Cc: schernikov, guilhermesi, kde-frameworks-devel, michaelh, ngraham, bruns
