[Piglit] [PATCH 3/3] arb_get_texture_sub_image: update cube map tests to complete the cube map

2018-03-28 Thread Anthony Pesch
From: Anthony Pesch Update cube map tests to complete the cube map before performing the final query. This final query is expected to succeed, however, querying a cube map which is not cube complete should set INVALID_OPERATION as per the OpenGL 4.6 Core spec: "An INVALID_OPERATION err

[Piglit] [PATCH 2/3] arb_get_texture_sub_image: fix depth parameter when performing zero-sized texture queries

2018-03-28 Thread Anthony Pesch
From: Anthony Pesch Change zero-sized texture queries to pass a valid depth parameter, otherwise they should error as per the OpenGL 4.6 Core spec: "An INVALID_VALUE error is generated if the effective target is TEXTURE_1D, TEXTURE_1D_ARRAY, TEXTURE_2D or TEXTURE_RECTANGLE, and either zo

[Piglit] [PATCH 1/3] arb_get_texture_sub_image: fix expected error when querying a level which hasn't been explicitly defined

2018-03-28 Thread Anthony Pesch
From: Anthony Pesch Change expected error from INVALID_OPERATION to INVALID_VALUE when querying a level which hasn't been explicitly defined. This is a valid operation, the error set should be due to the requested width and height being greater than the default width and height of

Re: [Piglit] [PATCH 1/3] arb_get_texture_sub_image: fix expected error when querying a level which hasn't been explicitly defined

2018-03-28 Thread Anthony Pesch
viewed-by: Arthur Huillet > > > On 28.03.2018 17:15, Anthony Pesch wrote: > >> From: Anthony Pesch >> >> Change expected error from INVALID_OPERATION to INVALID_VALUE when >> querying >> a level which hasn't been explicitly defined. This is a valid operat

Re: [Piglit] [PATCH 1/3] arb_get_texture_sub_image: fix expected error when querying a level which hasn't been explicitly defined

2018-04-04 Thread Anthony Pesch
ure’s depth." - Anthony From: Piglit on behalf of Juan A. Suarez Romero Sent: Wednesday, April 4, 2018 7:12 AM To: Anthony Pesch; piglit@lists.freedesktop.org Subject: Re: [Piglit] [PATCH 1/3] arb_get_texture_sub_image: fix expected error when querying a leve

Re: [Piglit] [PATCH 1/3] arb_get_texture_sub_image: fix expected error when querying a level which hasn't been explicitly defined

2018-04-08 Thread Anthony Pesch
Hey Juan, I've updated the patch to include the OpenGL spec references: Author: Anthony Pesch Date: Thu Mar 22 11:06:13 2018 -0400 arb_get_texture_sub_image: fix expected error when querying a level which hasn't been explicitly defined Change expected

Re: [Piglit] [PATCH 3/3] arb_get_texture_sub_image: update cube map tests to complete the cube map

2018-04-08 Thread Anthony Pesch
Hey Juan, Good catch - I'd only been focusing on the errors test. I've updated the patch to also fix the errors in arb_get_texture_sub_image-get: Author: Anthony Pesch Date: Thu Mar 22 13:11:22 2018 -0400 arb_get_texture_sub_image: update cube map tests to make textures cube

Re: [Piglit] [PATCH 3/3] arb_get_texture_sub_image: update cube map tests to complete the cube map

2018-04-10 Thread Anthony Pesch
ril 9, 2018 3:41 AM To: Anthony Pesch; Anthony Pesch; piglit@lists.freedesktop.org Subject: Re: [Piglit] [PATCH 3/3] arb_get_texture_sub_image: update cube map tests to complete the cube map For the series, Reviewed-by: Juan A. Suarez On Sun, 2018-04-08 at 18:35 +0000, Anthony Pesch wrote: >

[Piglit] [PATCH 1/3] arb_get_texture_sub_image: fix expected error when querying a level which hasn't been explicitly defined

2018-04-11 Thread Anthony Pesch
From: Anthony Pesch Change expected error from INVALID_OPERATION to INVALID_VALUE when querying a level which hasn't been explicitly defined. The level is valid, however, the level hasn't been explicitly defined so it should have a default width and height of 0, making the 8x8 query

[Piglit] [PATCH 3/3] arb_get_texture_sub_image: update cube map tests to make textures cube complete

2018-04-11 Thread Anthony Pesch
From: Anthony Pesch Update cube map tests to ensure cube map textures are cube complete before querying them. Querying a cube map which is not cube complete should set INVALID_OPERATION as per the OpenGL 4.6 Core spec: "An INVALID_OPERATION error is generated if the effective targ

[Piglit] [PATCH 2/3] arb_get_texture_sub_image: fix depth parameter when performing zero-sized texture queries

2018-04-11 Thread Anthony Pesch
From: Anthony Pesch Change zero-sized texture queries to pass a valid depth parameter, otherwise they should error as per the OpenGL 4.6 Core spec: "An INVALID_VALUE error is generated if the effective target is TEXTURE_1D, TEXTURE_1D_ARRAY, TEXTURE_2D or TEXTURE_RECTANGLE, and either zo

[Piglit] [PATCH] arb_texture_buffer_range: Fix buffer alignment in ranges-2 test

2019-04-15 Thread Anthony Pesch
From: Anthony Pesch The ranges-2 test was failing due to glTexBufferRange returning GL_INVALID_VALUE when the offset parameter wasn't GL_TEXTURE_BUFFER_OFFSET_ALIGNMENT byte aligned. From the OpenGL 4.6 Core spec: An INVALID_VALUE error is generated if offset is not an integer multiple o

Re: [Piglit] [PATCH] arb_texture_buffer_range: Fix buffer alignment in ranges-2 test

2019-05-09 Thread Anthony Pesch
Pinging to see if anyone has the time to review. - Anthony On 4/15/19 4:24 PM, Anthony Pesch wrote: From: Anthony Pesch The ranges-2 test was failing due to glTexBufferRange returning GL_INVALID_VALUE when the offset parameter wasn't GL_TEXTURE_BUFFER_OFFSET_ALIGNMENT byte aligned.

Re: [Piglit] [PATCH] arb_texture_buffer_range: Fix buffer alignment in ranges-2 test

2019-05-24 Thread Anthony Pesch
nthony commit 770effba655f97be229e13f894884f0e4c3e604a Author: Anthony Pesch Date: Thu Apr 11 12:37:07 2019 -0400 arb_texture_buffer_range: Fix buffer alignment in ranges-2 test The ranges-2 test was failing due to glTexBufferRange returning GL_INVALID_VALUE when the offset p

Re: [Piglit] [PATCH] arb_texture_buffer_range: Fix buffer alignment in ranges-2 test

2019-05-24 Thread Anthony Pesch
iglit on behalf of Anthony Pesch Sent: Friday, May 24, 2019 8:21 PM To: Pierre-Eric Pelloux-Prayer; Anthony Pesch; piglit@lists.freedesktop.org Subject: Re: [Piglit] [PATCH] arb_texture_buffer_range: Fix buffer alignment in ranges-2 test [CAUTION: External Email] Hi Pierre, Thanks! I'

Re: [Piglit] [PATCH] arb_texture_buffer_range: Fix buffer alignment in ranges-2 test

2019-06-03 Thread Anthony Pesch
merge request on gitlab if you prefer) Thanks! Pierre-Eric From: Anthony Pesch Sent: Friday, May 24, 2019 8:51 PM To: Pelloux-prayer, Pierre-eric; Pierre-Eric Pelloux-Prayer; Anthony Pesch; piglit@lists.freedesktop.org Subject: Re: [Piglit] [