On 10/03/2011 04:51 PM, Eric Anholt wrote:
> ---
>  src/glsl/ast_to_hir.cpp |    6 ++++++
>  1 files changed, 6 insertions(+), 0 deletions(-)
> 
> diff --git a/src/glsl/ast_to_hir.cpp b/src/glsl/ast_to_hir.cpp
> index 91a2231..99ca321 100644
> --- a/src/glsl/ast_to_hir.cpp
> +++ b/src/glsl/ast_to_hir.cpp
> @@ -2803,6 +2803,12 @@ ast_declarator_list::hir(exec_list *instructions,
>           _mesa_glsl_error(& loc, state,
>                            "identifier `%s' uses reserved `gl_' prefix",
>                            decl->identifier);
> +      else if (state->language_version >= 130 &&
> +               strstr(decl->identifier, "__")) {
> +         _mesa_glsl_error(& loc, state,
> +                          "identifier `%s' uses reserved `__' string",
> +                          decl->identifier);
> +      }
>  
>        /* Add the variable to the symbol table.  Note that the initializer's
>         * IR was already processed earlier (though it hasn't been emitted

That text is present in 1.10 and ES as well, so you should probably just
drop the version check.

With that change,
Reviewed-by: Kenneth Graunke <kenn...@whitecape.org>
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to