D10346: check for EGL_KHR_platform_gbm extension as well

David Edmundson noreply at phabricator.kde.org
Tue Feb 6 17:42:53 UTC 2018


davidedmundson requested changes to this revision.
davidedmundson added inline comments.
This revision now requires changes to proceed.
Restricted Application edited projects, added KWin; removed Plasma.

INLINE COMMENTS

> egl_gbm_backend.cpp:107
>  
>          display = eglGetPlatformDisplayEXT(EGL_PLATFORM_GBM_MESA, device, nullptr);
>      }

this is going to crash and burn when you have KHR but not Mesa.

You'll need to keep track of which extension you have, and alter any extension specific code accordingly.

REPOSITORY
  R108 KWin

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

To: mart, #kwin, #plasma, davidedmundson
Cc: davidedmundson, plasma-devel, kwin, iodelay, bwowk, ZrenBot, progwolff, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20180206/6e94be85/attachment-0001.html>


More information about the Plasma-devel mailing list