<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Apr 10, 2015 at 1:02 PM, Emil Velikov <span dir="ltr"><<a href="mailto:emil.l.velikov@gmail.com" target="_blank">emil.l.velikov@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Anuj,<br>
<div><div class="h5"><br>
On 10 April 2015 at 18:41, Anuj Phogat <<a href="mailto:anuj.phogat@gmail.com">anuj.phogat@gmail.com</a>> wrote:<br>
> Signed-off-by: Anuj Phogat <<a href="mailto:anuj.phogat@gmail.com">anuj.phogat@gmail.com</a>><br>
> ---<br>
>  tests/texturing/getteximage-targets.c | 43 ++++++++++++++++++++++++++++-------<br>
>  1 file changed, 35 insertions(+), 8 deletions(-)<br>
><br>
> diff --git a/tests/texturing/getteximage-targets.c b/tests/texturing/getteximage-targets.c<br>
> index 03bb4f8..0e1c3f1 100644<br>
> --- a/tests/texturing/getteximage-targets.c<br>
> +++ b/tests/texturing/getteximage-targets.c<br>
> @@ -221,45 +221,69 @@ getTexImage(bool doPBO, GLenum target, GLubyte data[][IMAGE_SIZE],<br>
>         return pass;<br>
>  }<br>
><br>
> +NORETURN void<br>
> +print_usage_and_exit(char *prog_name)<br>
> +{<br>
> +       printf("Usage: %s <target>\n"<br>
> +              "  where <target> is one of:\n"<br>
> +              "    1D\n"<br>
> +              "    2D\n"<br>
> +              "    3D\n"<br>
> +              "    RECT\n"<br>
> +              "    CUBE\n"<br>
> +              "    1D_ARRAY\n"<br>
> +              "    2D_ARRAY\n"<br>
> +              "    CUBE_ARRAY\n"<br>
> +              "    S3TC\n",<br>
> +              prog_name);<br>
> +       piglit_report_result(PIGLIT_FAIL);<br>
> +}<br>
> +<br>
>  void<br>
>  piglit_init(int argc, char **argv)<br>
>  {<br>
>         int i;<br>
> -       GLenum target = GL_TEXTURE_2D;<br>
> +       GLenum target;<br>
>         bool pass = true;<br>
>         GLenum internalformat = GL_RGBA8;<br>
>         GLubyte data[18][IMAGE_SIZE];<br>
>         int tolerance = 0;<br>
><br>
> +       if (argc != 2)<br>
> +               print_usage_and_exit(argv[0]);<br>
> +<br>
</div></div>Not sure if we want this. One can choose TEXTURE_2D with or without<br>
S3TC. Our current all.py has a few such cases.<br>
As a follow up we (most likely I'll go for it) can error out if one<br>
opts for strange combinations such as $ gettexture-target 2D 3D<br></blockquote><div> Sorry. I didn't test the patch properly.  Will send out a V2.</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Cheers<br>
<span class="HOEnZb"><font color="#888888">Emil<br>
</font></span></blockquote></div><br></div></div>