[Mesa-dev] [PATCH 1/7] main/extensions: Don't advertise unknown extensions overrides with (-)

[Anuj Phogat]

On Sun, Jun 8, 2014 at 3:17 PM, Jordan Justen <jordan.l.justen at intel.com> wrote:
> Previously setting:
> MESA_EXTENSION_OVERRIDE=-GL_MESA_ham_sandwich
>
> Would cause Mesa to advertise support for the GL_MESA_ham_sandwich
> extension, even though the override specifically asked for it to be
> disabled.
>
> Signed-off-by: Jordan Justen <jordan.l.justen at intel.com>
> ---
>  Series is available in the early-extension-override branch of
>  git://people.freedesktop.org/~jljusten/mesa
>
>  src/mesa/main/extensions.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/mesa/main/extensions.c b/src/mesa/main/extensions.c
> index c2ff7e3..c141576 100644
> --- a/src/mesa/main/extensions.c
> +++ b/src/mesa/main/extensions.c
> @@ -553,7 +553,7 @@ get_extension_override( struct gl_context *ctx )
>           break;
>        }
>        recognized = set_extension(ctx, ext, enable);
> -      if (!recognized) {
> +      if (!recognized && enable) {
>           strcat(extra_exts, ext);
>           strcat(extra_exts, " ");
>        }
> --
> 2.0.0
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Did a basic testing with your branch.  Things seem to work fine.
This series is: Reviewed-by: Anuj Phogat <anuj.phogat at gmail.com>