Reviewed-by: Alejandro Piñeiro <apinhe...@igalia.com>

On 15/02/17 19:35, Eduardo Lima Mitev wrote:
> glGetTextureSubImage() and glGetCompressedTextureSubImage() return
> GL_INVALID_VALUE instead of GL_INVALID_OPERATION when the 'texture'
> argument does not correspond to an existing texture object.
>
> From OpenGL 4.5 spec PDF, section '8.11. Texture Queries', page 236:
>
>     "An INVALID_VALUE error is generated if texture is not the name of
>      an existing texture object."
> ---
>  tests/spec/arb_get_texture_sub_image/errors.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/tests/spec/arb_get_texture_sub_image/errors.c 
> b/tests/spec/arb_get_texture_sub_image/errors.c
> index 8eb005e61..0a662ee23 100644
> --- a/tests/spec/arb_get_texture_sub_image/errors.c
> +++ b/tests/spec/arb_get_texture_sub_image/errors.c
> @@ -46,7 +46,7 @@ test_texture_id(void)
>                            8, 8, 1, /* size */
>                            GL_RGBA, GL_UNSIGNED_BYTE,
>                            sizeof(buffer), buffer);
> -     if (!piglit_check_gl_error(GL_INVALID_OPERATION))
> +     if (!piglit_check_gl_error(GL_INVALID_VALUE))
>               pass = false;
>  
>       /* Test compressed get with bad texture ID */
> @@ -54,7 +54,7 @@ test_texture_id(void)
>                                      0, 0, 0, /* offset */
>                                      8, 8, 1, /* size */
>                                      sizeof(buffer), buffer);
> -     if (!piglit_check_gl_error(GL_INVALID_OPERATION))
> +     if (!piglit_check_gl_error(GL_INVALID_VALUE))
>               pass = false;
>  
>       /* Test get with undefined texture */

_______________________________________________
Piglit mailing list
Piglit@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/piglit

Reply via email to