View Revision
davidedmundson added inline comments.

INLINE COMMENTS
View Inlineapol wrote in aurorae.cpp:344

#if QT_VERSION < QT_VERSION_CHECK(5,10,0)?

Yeah, makes sense, it got +2'd quicker than I expected.


View Inlinegraesslin wrote in aurorae.cpp:354

stupid question: but why is this code working? I would assume that making the context current will fail here and then we return? And even if the creation of the FBO should fail and then also return?

Changing the rendering doesn't block making calls to any of the openGL calls in QtBase.
The only changes are purely inside QtQuick

So:
creating a context still works fine
creating an FBO still works fine

why m_view->setRenderTarget and m_view->resetOpenGLState still work are the more interesting questions
but that's because they're virtual methods that no-op.

With this code:
we create an OpenGL context
set it to be active
then ignore it and do something completely different with QPainter instead


REPOSITORY
R108 KWin

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

To: davidedmundson, Plasma
Cc: graesslin, apol, plasma-devel, kwin, KWin, bwowk, ZrenBot, progwolff, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, sebas, mart