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]> -- Regards, Tom "Where's the kaboom!? There was supposed to be an earth-shattering kaboom!" Marvin Martian Graphics Working Group | Linaro.org │ Open source software for ARM SoCs w) tom.gall att linaro.org h) tom_gall att mac.com _______________________________________________ Piglit mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/piglit
