<table><tr><td style="">madcatx updated this revision to Diff 16377.<br />madcatx 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/D6565" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>v2. The code now tries to leverage GLX_MESA_query_renderer when it is available and falls back to the old-fashioned probing otherwise. As for some of the suggestions by Martin:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">The new code now uses nullptrs instead of NULLs</li>
<li class="remarkup-list-item">qCDebug() does not seem to be used anywhere in the whole kinfocenter so I cannot simply replace qDebug()s with their qCDebug() counterparts without declaring the logging category first.</li>
<li class="remarkup-list-item">As for the GlxContext builders, is it worth it to pull in some KWin dependencies just to query the OpenGL core profile? Currently the kinfocenter does not seem to depend of KWin in any way.</li>
</ul></div></div><br /><div><strong>REPOSITORY</strong><div><div>R102 KInfoCenter</div></div></div><br /><div><strong>CHANGES SINCE LAST UPDATE</strong><div><a href="https://phabricator.kde.org/D6565?vs=16331&id=16377" rel="noreferrer">https://phabricator.kde.org/D6565?vs=16331&id=16377</a></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D6565" rel="noreferrer">https://phabricator.kde.org/D6565</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>Modules/opengl/opengl.cpp</div></div></div><br /><div><strong>To: </strong>madcatx, graesslin<br /><strong>Cc: </strong>fredrik, graesslin, cfeck, plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart, lukas<br /></div>