[Piglit] [PATCH 1/5] fbo-drawbuffers-none: Add the instructions for running subtests

Marek Olšák maraeo at gmail.com
Tue May 20 09:03:46 PDT 2014


If you fix the typo that Ilia pointed out, the series is:

Reviewed-by: Marek Olšák <marek.olsak at amd.com>

Marek

On Mon, May 19, 2014 at 10:51 PM, Anuj Phogat <anuj.phogat at gmail.com> wrote:
> Signed-off-by: Anuj Phogat <anuj.phogat at gmail.com>
> ---
>  .../arb_framebuffer_object/fbo-drawbuffers-none.c  | 22 ++++++++++++++++++----
>  1 file changed, 18 insertions(+), 4 deletions(-)
>
> diff --git a/tests/spec/arb_framebuffer_object/fbo-drawbuffers-none.c b/tests/spec/arb_framebuffer_object/fbo-drawbuffers-none.c
> index b4b476d..58e3f32 100644
> --- a/tests/spec/arb_framebuffer_object/fbo-drawbuffers-none.c
> +++ b/tests/spec/arb_framebuffer_object/fbo-drawbuffers-none.c
> @@ -381,16 +381,30 @@ test_glBlitFramebuffer(const GLenum drawbufs[4])
>         return probe_buffers(drawbufs, colors_all_red);
>  }
>
> +static void
> +print_usage_and_exit(const char *prog_name)
> +{
> +       printf("Usage: %s <test_name>\n"
> +              "  where <test_name> is one of:\n"
> +              "    glClear\n"
> +              "    glClearBuffer\n"
> +              "    gl_FragColor\n"
> +              "    gl_FragData\n"
> +              "    glColorMaskIndexed\n"
> +              "    glBlendFunci\n"
> +              "    glDrawPixels\n"
> +              "    glBlitFramebuffer\n", prog_name);
> +       piglit_report_result(PIGLIT_FAIL);
> +}
> +
>  void
>  piglit_init(int argc, char **argv)
>  {
>         bool pass = true;
>         int i, max_draw_bufs;
>
> -       if (argc != 2) {
> -               printf("Expected one parameter, got %i.\n", argc-1);
> -               piglit_report_result(PIGLIT_FAIL);
> -       }
> +       if (argc != 2)
> +               print_usage_and_exit(argv[0]);
>         test_name = argv[1];
>
>         piglit_require_gl_version(21);
> --
> 1.8.3.1
>
> _______________________________________________
> Piglit mailing list
> Piglit at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/piglit


More information about the Piglit mailing list