Skip to content

Commit

Permalink
Correct EGL_EXT_config_select_group extension query
Browse files Browse the repository at this point in the history
EGL_EXT_config_select_group is a display extension,
not a client extension.

Affects:
dEQP-EGL.functional.choose_config.simple.selection_and_sort.*

Ref: KhronosGroup/EGL-Registry#199

Fixes: 88ba9ac ("Implement support for the EGL_EXT_config_select_group extension")

Change-Id: I38956511bdcb8e99d585ea9b99aeab53da0457e2
Signed-off-by: David Heidelberg <[email protected]>
  • Loading branch information
okias authored and lordalcol committed May 24, 2024
1 parent 8ca5f34 commit ad5c854
Showing 1 changed file with 2 additions and 3 deletions.
5 changes: 2 additions & 3 deletions framework/egl/egluConfigInfo.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -169,8 +169,7 @@ void queryCoreConfigInfo(const Library &egl, EGLDisplay display, EGLConfig confi

void queryExtConfigInfo(const eglw::Library &egl, eglw::EGLDisplay display, eglw::EGLConfig config, ConfigInfo *dst)
{
const std::vector<std::string> extensions = getDisplayExtensions(egl, display);
const std::vector<std::string> clientExtensions = getClientExtensions(egl);
const std::vector<std::string> extensions = getDisplayExtensions(egl, display);

if (de::contains(extensions.begin(), extensions.end(), "EGL_EXT_yuv_surface"))
{
Expand Down Expand Up @@ -200,7 +199,7 @@ void queryExtConfigInfo(const eglw::Library &egl, eglw::EGLDisplay display, eglw
else
dst->colorComponentType = EGL_COLOR_COMPONENT_TYPE_FIXED_EXT;

if (de::contains(clientExtensions.begin(), clientExtensions.end(), "EGL_EXT_config_select_group"))
if (hasExtension(egl, display, "EGL_EXT_config_select_group"))
{
egl.getConfigAttrib(display, config, EGL_CONFIG_SELECT_GROUP_EXT, (EGLint *)&dst->groupId);

Expand Down

0 comments on commit ad5c854

Please sign in to comment.