On Fri, Jul 15, 2016 at 11:29 AM, Ilia Mirkin <[email protected]> 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 <[email protected]>
> ---
>  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 <[email protected]>
_______________________________________________
Piglit mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/piglit

Reply via email to