Re: [Mesa-dev] [PATCH] glsl: Mark tessellation qualifier maps static const.

2016-08-23 Thread Ilia Mirkin
Reviewed-by: Ilia Mirkin 

Ah so that's why they weren't static const in my patch... coz I copied
it :) Nice sleuth work.

On Tue, Aug 23, 2016 at 4:39 PM, Kenneth Graunke  wrote:
> Signed-off-by: Kenneth Graunke 
> ---
>  src/compiler/glsl/glsl_parser.yy | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/src/compiler/glsl/glsl_parser.yy 
> b/src/compiler/glsl/glsl_parser.yy
> index 4f4a83c..5b65861 100644
> --- a/src/compiler/glsl/glsl_parser.yy
> +++ b/src/compiler/glsl/glsl_parser.yy
> @@ -1373,7 +1373,7 @@ layout_qualifier_id:
>
>/* Layout qualifiers for tessellation evaluation shaders. */
>if (!$$.flags.i) {
> - struct {
> + static const struct {
>  const char *s;
>  GLenum e;
>   } map[] = {
> @@ -1396,7 +1396,7 @@ layout_qualifier_id:
>   }
>}
>if (!$$.flags.i) {
> - struct {
> + static const struct {
>  const char *s;
>  GLenum e;
>   } map[] = {
> --
> 2.9.3
>
> ___
> mesa-dev mailing list
> mesa-dev@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev


[Mesa-dev] [PATCH] glsl: Mark tessellation qualifier maps static const.

2016-08-23 Thread Kenneth Graunke
Signed-off-by: Kenneth Graunke 
---
 src/compiler/glsl/glsl_parser.yy | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/compiler/glsl/glsl_parser.yy b/src/compiler/glsl/glsl_parser.yy
index 4f4a83c..5b65861 100644
--- a/src/compiler/glsl/glsl_parser.yy
+++ b/src/compiler/glsl/glsl_parser.yy
@@ -1373,7 +1373,7 @@ layout_qualifier_id:
 
   /* Layout qualifiers for tessellation evaluation shaders. */
   if (!$$.flags.i) {
- struct {
+ static const struct {
 const char *s;
 GLenum e;
  } map[] = {
@@ -1396,7 +1396,7 @@ layout_qualifier_id:
  }
   }
   if (!$$.flags.i) {
- struct {
+ static const struct {
 const char *s;
 GLenum e;
  } map[] = {
-- 
2.9.3

___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev