D8012: Only assert on broken GL when we're rendering using GL

2017-09-27 Thread David Edmundson
davidedmundson added inline comments. INLINE COMMENTS > graesslin wrote in desktopview.cpp:60 > Is it correct to compare to software? What if vulkan get's introduced? Then we change it.. The reason it's done this way round is currently is that setSceneGraph("aaerkwlakrjaldkfj") will result in

D8012: Only assert on broken GL when we're rendering using GL

2017-09-27 Thread Martin Flöser
graesslin added inline comments. INLINE COMMENTS > desktopview.cpp:60 > > -connect(this, &DesktopView::sceneGraphInitialized, this, > -[this, corona]() { > -// check whether the GL Context supports OpenGL > -// Note: hasOpenGLShaderPrograms is broken, see QTB

D8012: Only assert on broken GL when we're rendering using GL

2017-09-27 Thread David Edmundson
This revision was automatically updated to reflect the committed changes. Closed by commit R120:3eb76ed6c126: Only assert on broken GL when we're rendering using GL (authored by davidedmundson). REPOSITORY R120 Plasma Workspace CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D8012?vs=1

D8012: Only assert on broken GL when we're rendering using GL

2017-09-27 Thread Kai Uwe Broulik
broulik accepted this revision. This revision is now accepted and ready to land. REPOSITORY R120 Plasma Workspace BRANCH master REVISION DETAIL https://phabricator.kde.org/D8012 To: davidedmundson, #plasma, broulik Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterb

D8012: Only assert on broken GL when we're rendering using GL

2017-09-27 Thread David Edmundson
davidedmundson created this revision. davidedmundson added a reviewer: Plasma. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. TEST PLAN set backend. Didn't get an error message REPOSITORY R120 Plasma Workspace BRANCH master REVISIO