<div dir="ltr">On 27 February 2013 10:46, Tom Gall <span dir="ltr"><<a href="mailto:tom.gall@linaro.org" target="_blank">tom.gall@linaro.org</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">


Hmm is there some reason this has to be a compile time decision as<br>
compared to use of the piglit_is_gles() function?<br></blockquote><div><br></div><div>No, I don't really see a reason to prefer one way over the other.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">



<br>
Here's a link to the patch that I had posted on the 18th.<br>
<br>
<a href="http://lists.freedesktop.org/archives/piglit/2013-February/004952.html" target="_blank">http://lists.freedesktop.org/archives/piglit/2013-February/004952.html</a><br></blockquote><div><br></div><div>Wow, looks like we both failed to notice each other's patch :)<br>
<br></div><div>It looks like your version only addresses GLES2, not GLES3.  Any objection to going forward with my version?  I have the series ready to push, and it's already received Eric's review.<br></div><div>
 </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div><div><br>
<br>
<br>
On Wed, Feb 27, 2013 at 12:32 PM, Paul Berry <<a href="mailto:stereotype441@gmail.com" target="_blank">stereotype441@gmail.com</a>> wrote:<br>
> glslparsertest should only check for the presence of<br>
> GL_ARB_ES{2,3}_compatibility when the test is running under desktop<br>
> GL.  GLES never exposes these extensions.<br>
> ---<br>
>  tests/glslparsertest/glslparsertest.c | 36 ++++++++++++++++++++++++-----------<br>
>  1 file changed, 25 insertions(+), 11 deletions(-)<br>
><br>
> diff --git a/tests/glslparsertest/glslparsertest.c b/tests/glslparsertest/glslparsertest.c<br>
> index 26a558f..43bef03 100644<br>
> --- a/tests/glslparsertest/glslparsertest.c<br>
> +++ b/tests/glslparsertest/glslparsertest.c<br>
> @@ -366,6 +366,30 @@ parse_glsl_version_string(const char *str)<br>
>         return parse_glsl_version_number(str);<br>
>  }<br>
><br>
> +<br>
> +static void<br>
> +check_version(unsigned glsl_version)<br>
> +{<br>
> +#ifdef PIGLIT_USE_OPENGL<br>
> +       if (requested_version == 100) {<br>
> +               piglit_require_extension("GL_ARB_ES2_compatibility");<br>
> +               return;<br>
> +       } else if (requested_version == 300) {<br>
> +               piglit_require_extension("GL_ARB_ES3_compatibility");<br>
> +               return;<br>
> +       }<br>
> +#endif<br>
> +<br>
> +       if (glsl_version < requested_version) {<br>
> +               fprintf(stderr,<br>
> +                       "GLSL version is %u.%u, but requested version %u.%u is required\n",<br>
> +                       glsl_version / 100, glsl_version % 100,<br>
> +                       requested_version / 100, requested_version % 100);<br>
> +               piglit_report_result(PIGLIT_SKIP);<br>
> +       }<br>
> +}<br>
> +<br>
> +<br>
>  void<br>
>  piglit_init(int argc, char**argv)<br>
>  {<br>
> @@ -406,17 +430,7 @@ piglit_init(int argc, char**argv)<br>
>         if (glsl_version_string != NULL)<br>
>                 glsl_version = parse_glsl_version_string(glsl_version_string);<br>
><br>
> -       if (requested_version == 100) {<br>
> -               piglit_require_extension("GL_ARB_ES2_compatibility");<br>
> -       } else if (requested_version == 300) {<br>
> -               piglit_require_extension("GL_ARB_ES3_compatibility");<br>
> -       } else if (glsl_version < requested_version) {<br>
> -               fprintf(stderr,<br>
> -                       "GLSL version is %u.%u, but requested version %u.%u is required\n",<br>
> -                       glsl_version / 100, glsl_version % 100,<br>
> -                       requested_version / 100, requested_version % 100);<br>
> -               piglit_report_result(PIGLIT_SKIP);<br>
> -       }<br>
> +       check_version(glsl_version);<br>
><br>
>         for (i = 4; i < argc; i++) {<br>
>                 if (argv[i][0] == '!') {<br>
> --<br>
> 1.8.1.4<br>
><br>
</div></div>> _______________________________________________<br>
> Piglit mailing list<br>
> <a href="mailto:Piglit@lists.freedesktop.org" target="_blank">Piglit@lists.freedesktop.org</a><br>
> <a href="http://lists.freedesktop.org/mailman/listinfo/piglit" target="_blank">http://lists.freedesktop.org/mailman/listinfo/piglit</a><br>
<br>
<br>
<br>
--<br>
Regards,<br>
Tom<br>
<br>
"Where's the kaboom!? There was supposed to be an earth-shattering<br>
kaboom!" Marvin Martian<br>
Tech Lead, Graphics Working Group | Linaro.org │ Open source software<br>
for ARM SoCs<br>
w) tom.gall att <a href="http://linaro.org" target="_blank">linaro.org</a><br>
h) tom_gall att <a href="http://mac.com" target="_blank">mac.com</a><br>
</blockquote></div><br></div></div>