Rémi Bernon : opengl32: Introduce new disabled_extensions_index helper.

[Alexandre Julliard]

Module: wine
Branch: master
Commit: 1001745dde5e78a5f832362d0154348efc7e3b3b
URL:    https://gitlab.winehq.org/wine/wine/-/commit/1001745dde5e78a5f832362d0154348...

Author: Rémi Bernon 
Date:   Mon Nov  7 11:46:56 2022 +0100

opengl32: Introduce new disabled_extensions_index helper.

---

dlls/opengl32/wgl.c | 49 +++++++++++++++----------------------------------
 1 file changed, 15 insertions(+), 34 deletions(-)

diff --git a/dlls/opengl32/wgl.c b/dlls/opengl32/wgl.c
index d5b0a78b916..a9039d606c1 100644
--- a/dlls/opengl32/wgl.c
+++ b/dlls/opengl32/wgl.c
@@ -455,39 +455,32 @@ static BOOL filter_extensions( const char *extensions, GLubyte **exts_list, GLui
     return (exts_list && *exts_list) || *disabled_exts;
 }

+static const GLuint *disabled_extensions_index(void)
+{
+    struct wgl_handle *ptr = get_current_context_ptr();
+    GLuint **disabled = &ptr->u.context->disabled_exts;
+    if (*disabled || filter_extensions( NULL, NULL, disabled )) return *disabled;
+    return NULL;
+}
+
 void WINAPI glGetIntegerv(GLenum pname, GLint *data)
 {
     struct glGetIntegerv_params args = { .pname = pname, .data = data, };
+    const GLuint *disabled;
     NTSTATUS status;

TRACE( "pname %d, data %p\n", pname, data );

-    if (pname == GL_NUM_EXTENSIONS)
-    {
-        struct wgl_handle *ptr = get_current_context_ptr();
-        GLuint **disabled = &ptr->u.context->disabled_exts;
-
-        if (*disabled || filter_extensions( NULL, NULL, disabled ))
-        {
-            const GLuint *disabled_exts = *disabled;
-            GLint count, disabled_count = 0;
-
-            args.data = &count;
-            if ((status = UNIX_CALL( glGetIntegerv, &args ))) WARN( "glGetIntegerv returned %#x\n", status );
-
-            while (*disabled_exts++ != ~0u)
-                disabled_count++;
-            *data = count - disabled_count;
-            return;
-        }
-    }
-
     if ((status = UNIX_CALL( glGetIntegerv, &args ))) WARN( "glGetIntegerv returned %#x\n", status );
+
+    if (pname == GL_NUM_EXTENSIONS && (disabled = disabled_extensions_index()))
+        while (*disabled++ != ~0u) (*data)--;
 }

const GLubyte * WINAPI glGetStringi(GLenum name, GLuint index)
 {
     const struct opengl_funcs *funcs = NtCurrentTeb()->glTable;
+    const GLuint *disabled;

TRACE("(%d, %d)\n", name, index);
     if (!funcs->ext.p_glGetStringi)
@@ -497,21 +490,9 @@ const GLubyte * WINAPI glGetStringi(GLenum name, GLuint index)
         *func_ptr = funcs->wgl.p_wglGetProcAddress("glGetStringi");
     }

-    if (name == GL_EXTENSIONS)
-    {
-        struct wgl_handle *ptr = get_current_context_ptr();
-
-        if (ptr->u.context->disabled_exts ||
-            filter_extensions(NULL, NULL, &ptr->u.context->disabled_exts))
-        {
-            const GLuint *disabled_exts = ptr->u.context->disabled_exts;
-            unsigned int disabled_count = 0;
+    if (name == GL_EXTENSIONS && (disabled = disabled_extensions_index()))
+        while (index >= *disabled++) index++;

-            while (index + disabled_count >= *disabled_exts++)
-                disabled_count++;
-            return funcs->ext.p_glGetStringi(name, index + disabled_count);
-        }
-    }
     return funcs->ext.p_glGetStringi(name, index);
 }