[Piglit] [PATCH] texture-integer-glsl130: fix expected values
Brian Paul
brianp at vmware.com
Fri Aug 31 15:02:00 PDT 2012
On 08/31/2012 11:43 AM, Vadim Girlin wrote:
> ---
> tests/spec/ext_texture_integer/texture-integer-glsl130.c | 13 ++++++++++---
> 1 file changed, 10 insertions(+), 3 deletions(-)
>
> diff --git a/tests/spec/ext_texture_integer/texture-integer-glsl130.c b/tests/spec/ext_texture_integer/texture-integer-glsl130.c
> index 49b2f0e..3200528 100644
> --- a/tests/spec/ext_texture_integer/texture-integer-glsl130.c
> +++ b/tests/spec/ext_texture_integer/texture-integer-glsl130.c
> @@ -368,10 +368,14 @@ test_format(const struct format_info *info)
> value[0] = temp;
> break;
> case GL_RGB_INTEGER_EXT:
> - expected[3] = 0.0;
> + expected[3] = 1.0;
> + value[3] = 1.0;
> break;
> case GL_RG_INTEGER:
> - expected[2] = expected[3] = 0.0;
> + expected[2] = 0.0;
> + value[2] = 0.0;
> + expected[3] = 1.0;
> + value[3] = 1.0;
> break;
> case GL_ALPHA_INTEGER_EXT:
> expected[0] = expected[1] = expected[2] = 0.0;
> @@ -399,7 +403,10 @@ test_format(const struct format_info *info)
> expected[0] = expected[1] = expected[2] = expected[3] = 0.25;
> value[1] = value[2] = value[3] = value[0];
> } else {
> - expected[1] = expected[2] = expected[3] = 0.0;
> + expected[1] = expected[2] = 0.0;
> + value[1] = value[2] = 0.0;
> + expected[3] = 1.0;
> + value[3] = 1.0;
> }
> break;
> default:
Looks good. I tested the patch w/ NVIDIA's driver and it passes there.
Reviewed-by: Brian Paul <brianp at vmware.com>
More information about the Piglit
mailing list