From: Emil Velikov Date: Mon, 6 Nov 2017 15:33:52 +0000 (+0000) Subject: mesa: call _mesa_make_extension_string only as needed X-Git-Tag: upstream/18.1.0~4012 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=167e958a87d53614e6bbb78b09bafcea17335b4b;p=platform%2Fupstream%2Fmesa.git mesa: call _mesa_make_extension_string only as needed As of previous commit we removed the extension overrides from this function. Thus we no longer need to call it during MakeCurrent, so we can construct the extensions string when needed - _mesa_GetString. This commit effectively reverts a879d14ecf8 ("mesa: initialize extension string when context is first bound") Signed-off-by: Emil Velikov Reviewed-by: Brian Paul --- diff --git a/src/mesa/main/context.c b/src/mesa/main/context.c index 61099eb..762765c 100644 --- a/src/mesa/main/context.c +++ b/src/mesa/main/context.c @@ -1576,8 +1576,6 @@ handle_first_current(struct gl_context *ctx) return; } - ctx->Extensions.String = _mesa_make_extension_string(ctx); - check_context_limits(ctx); /* According to GL_MESA_configless_context the default value of diff --git a/src/mesa/main/getstring.c b/src/mesa/main/getstring.c index 4c805ca..931f6a4 100644 --- a/src/mesa/main/getstring.c +++ b/src/mesa/main/getstring.c @@ -143,6 +143,8 @@ _mesa_GetString( GLenum name ) _mesa_error(ctx, GL_INVALID_ENUM, "glGetString(GL_EXTENSIONS)"); return (const GLubyte *) 0; } + if (!ctx->Extensions.String) + ctx->Extensions.String = _mesa_make_extension_string(ctx); return (const GLubyte *) ctx->Extensions.String; case GL_SHADING_LANGUAGE_VERSION: if (ctx->API == API_OPENGLES)