[Piglit] [PATCH] copy-pixels: disable texture state before copying

Anuj Phogat anuj.phogat at gmail.com
Fri Jul 15 19:33:39 UTC 2016


On Fri, Jul 15, 2016 at 11:29 AM, Ilia Mirkin <imirkin at alum.mit.edu> wrote:
> The intent was to copy the texture around, not to test the texture
> combining logic. (A separate test should probably be done with that
> enabled.)
>
> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=96943
> Signed-off-by: Ilia Mirkin <imirkin at alum.mit.edu>
> ---
>  tests/general/copy-pixels.c | 2 ++
>  1 file changed, 2 insertions(+)
>
> diff --git a/tests/general/copy-pixels.c b/tests/general/copy-pixels.c
> index cff1cb6..a45d821 100644
> --- a/tests/general/copy-pixels.c
> +++ b/tests/general/copy-pixels.c
> @@ -61,6 +61,8 @@ test_color_copypix(int x, int y)
>
>         piglit_draw_rect_tex(0, 0, IMAGE_WIDTH, IMAGE_HEIGHT, 0, 0, 1, 1);
>
> +       glDisable(GL_TEXTURE_2D);
> +
>         glRasterPos2i(x, y);
>         glCopyPixels(0, 0, IMAGE_WIDTH, IMAGE_HEIGHT, GL_COLOR);
>         pass = piglit_probe_image_color(x, y, IMAGE_WIDTH, IMAGE_HEIGHT,
> --
> 2.7.3
>
Thanks for fixing the test.

Reviewed-by: Anuj Phogat <anuj.phogat at gmail.com>


More information about the Piglit mailing list