<table><tr><td style="">ekurzinger updated this revision to Diff 56075.<br />ekurzinger added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D18824">View Revision</a></tr></table><br /><div><div><p>Ok, now we will bail out early during create()  if we fail to dynamically load the interface (before calling wl_global_create). The compositor can then use isValid() to check if whether creating the interface succeeded or not, it will also print a warning if it fails. Unfortunately, as Roman has pointed out, since Global::create is not virtual the compositor technically *could* still break things by calling the parent class method, but I'm not sure if it's worth changing that for just this one somewhat special case, especially considering that this will probably only ever actually be used by KWin's EGLStream backend anyway.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R127 KWayland</div></div></div><br /><div><strong>CHANGES SINCE LAST UPDATE</strong><div><a href="https://phabricator.kde.org/D18824?vs=56067&id=56075">https://phabricator.kde.org/D18824?vs=56067&id=56075</a></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D18824">https://phabricator.kde.org/D18824</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>src/client/protocols/wayland-eglstream-controller.xml<br />
src/server/CMakeLists.txt<br />
src/server/display.cpp<br />
src/server/display.h<br />
src/server/eglstream_controller_interface.cpp<br />
src/server/eglstream_controller_interface.h<br />
src/server/eglstream_controller_interface_p.h</div></div></div><br /><div><strong>To: </strong>ekurzinger, romangg, davidedmundson, KWin<br /><strong>Cc: </strong>ngraham, mati865, schernikov, guilhermesi, kde-frameworks-devel, michaelh, bruns<br /></div>