On 01/23/2014 11:03 AM, Emil Velikov wrote:
> The former symbol is never defined within mesa. Based
> it seems that the original intent was to use NDEBUG.

That seems right to me.

Reviewed-by: Ian Romanick <ian.d.roman...@intel.com>

It looks like this code was originally written by Paul, so he may have
some comment.

> Signed-off-by: Emil Velikov <emil.l.veli...@gmail.com>
> ---
>  src/glsl/ir.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/src/glsl/ir.h b/src/glsl/ir.h
> index 19e8383..e266328 100644
> --- a/src/glsl/ir.h
> +++ b/src/glsl/ir.h
> @@ -471,7 +471,7 @@ public:
>     void reinit_interface_type(const struct glsl_type *type)
>     {
>        if (this->max_ifc_array_access != NULL) {
> -#ifndef _NDEBUG
> +#ifndef NDEBUG
>           /* Redeclaring gl_PerVertex is only allowed if none of the built-ins
>            * it defines have been accessed yet; so it's safe to throw away the
>            * old max_ifc_array_access pointer, since all of its values are
> 

_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to