On 01/16/2013 11:25 AM, Tom Gall wrote: > Thanks for the discussion on this one. Let's move forward on it. > > On Wed, Jan 16, 2013 at 12:47 PM, Chad Versace > <[email protected]> wrote: >> On 01/15/2013 07:25 PM, Stuart Abercrombie wrote: >>> The current syntax isn't compatible with the same shader_test >>> supporting GL and GLES in the future. >>> >>> Modify existing GL ES tests to use the new syntax, and remove >>> explicit #version directives, which will instead be inserted based on >>> GLSL >= requirements. >>> >>> v2 Added check for trailing chars. Formatting. >>> --- >>> tests/shaders/shader_runner.c | 24 >>> +++++++++---------- >>> .../spec/glsl-es-1.00/execution/sanity.shader_test | 7 +---- >>> .../spec/glsl-es-3.00/execution/sanity.shader_test | 8 +----- >>> 3 files changed, 15 insertions(+), 24 deletions(-) >> >> Reviewed-by: Chad Versace <[email protected]> >> > > Reviewed-by: Tom Gall <[email protected]>
Patch is committed. _______________________________________________ Piglit mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/piglit
