Re: [Mesa-dev] [PATCH] glcpp: Don't define macros for extensions that aren't in ES

2012-11-14 Thread Matt Turner
On Wed, Nov 14, 2012 at 4:45 PM, Kenneth Graunke wrote: > On 11/14/2012 10:34 AM, Matt Turner wrote: >> >> Fixes 54 gles3conform tests. >> --- >> src/glsl/glcpp/glcpp-parse.y | 59 >> + >> 1 files changed, 30 insertions(+), 29 deletions(-) >> >> The cha

Re: [Mesa-dev] [PATCH] glcpp: Don't define macros for extensions that aren't in ES

2012-11-14 Thread Kenneth Graunke
On 11/14/2012 10:34 AM, Matt Turner wrote: Fixes 54 gles3conform tests. --- src/glsl/glcpp/glcpp-parse.y | 59 + 1 files changed, 30 insertions(+), 29 deletions(-) The changes in whitespace make this patch hard to read. Here it is without whitespace c

Re: [Mesa-dev] [PATCH] glcpp: Don't define macros for extensions that aren't in ES

2012-11-14 Thread Ian Romanick
On 11/14/2012 10:34 AM, Matt Turner wrote: Fixes 54 gles3conform tests. --- src/glsl/glcpp/glcpp-parse.y | 59 + 1 files changed, 30 insertions(+), 29 deletions(-) The changes in whitespace make this patch hard to read. Here it is without whitespace c

Re: [Mesa-dev] [PATCH] glcpp: Don't define macros for extensions that aren't in ES

2012-11-14 Thread Matt Turner
On Wed, Nov 14, 2012 at 10:34 AM, Matt Turner wrote: > - if (extensions->OES_EGL_image_external) > - add_builtin_define(parser, "GL_OES_EGL_image_external", 1); > + if (extensions->OES_EGL_image_external) > +add_builtin_define(parser, "GL_OES_EGL_im

[Mesa-dev] [PATCH] glcpp: Don't define macros for extensions that aren't in ES

2012-11-14 Thread Matt Turner
Fixes 54 gles3conform tests. --- src/glsl/glcpp/glcpp-parse.y | 59 + 1 files changed, 30 insertions(+), 29 deletions(-) The changes in whitespace make this patch hard to read. Here it is without whitespace changes: --- a/src/glsl/glcpp/glcpp-parse.y +++