[Piglit] [PATCH 1/3] depthstencil-render-miplevels: Move the rendering to display() time.

[Paul Berry]

On 26 February 2013 12:39, Eric Anholt <eric at anholt.net> wrote:

> I want to actually present results on the screen so that debugging
> this test involves less meditation before achieving enlightenment.
> ---
>  tests/texturing/depthstencil-render-miplevels.cpp |   13 +++++--------
>  1 file changed, 5 insertions(+), 8 deletions(-)
>

Reviewed-by: Paul Berry <stereotype441 at gmail.com>


>
> diff --git a/tests/texturing/depthstencil-render-miplevels.cpp
> b/tests/texturing/depthstencil-render-miplevels.cpp
> index ac37b38..4fcc95a 100644
> --- a/tests/texturing/depthstencil-render-miplevels.cpp
> +++ b/tests/texturing/depthstencil-render-miplevels.cpp
> @@ -415,7 +415,11 @@ piglit_init(int argc, char **argv)
>         } else {
>                 print_usage_and_exit(argv[0]);
>         }
> +}
>
> +extern "C" enum piglit_result
> +piglit_display()
> +{
>         bool pass = true;
>
>         color_tex = create_mipmapped_tex(GL_RGBA);
> @@ -454,14 +458,7 @@ piglit_init(int argc, char **argv)
>                 pass = test_miplevel(level) && pass;
>         }
>
> -       piglit_report_result(pass ? PIGLIT_PASS : PIGLIT_FAIL);
> -}
> -
> -extern "C" enum piglit_result
> -piglit_display()
> -{
> -       /* Should never be reached */
> -       return PIGLIT_FAIL;
> +       return pass ? PIGLIT_PASS : PIGLIT_FAIL;
>  }
>
>  }; /* Anonymous namespace */
> --
> 1.7.10.4
>
> _______________________________________________
> Piglit mailing list
> Piglit at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/piglit
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/piglit/attachments/20130304/976d9fc8/attachment.html>