Re: [Piglit] [PATCH] ARB_texture_view: Add test suite

2013-10-16 Thread Jon Ashburn
I am not sure why the texture-immutable-levels.c file is in the arb_texture_view directory since it doesn't actually call glTextureView(). Should it be in the arb_texture_storage directory instead? On 10/15/2013 05:34 PM, Matt Turner wrote: On Mon, Oct 14, 2013 at 2:31 PM, Jon Ashburn

Re: [Piglit] [PATCH] ARB_texture_view: Add test suite

2013-10-16 Thread Matt Turner
On Wed, Oct 16, 2013 at 12:30 PM, Jon Ashburn j...@lunarg.com wrote: I am not sure why the texture-immutable-levels.c file is in the arb_texture_view directory since it doesn't actually call glTextureView(). Should it be in the arb_texture_storage directory instead? The background is that ES

Re: [Piglit] [PATCH] ARB_texture_view: Add test suite

2013-10-16 Thread Matt Turner
On Wed, Oct 16, 2013 at 12:58 PM, Jon Ashburn j...@lunarg.com wrote: Matt: Thanks for clarification, I understand now why the test is in arb_texture_view directory and the purpose it serves. I still think testing TEXTURE_IMMUTABLE_LEVELS and TEXTURE_VIEW_NUM_LEVELS after a glTextureView()

Re: [Piglit] [PATCH] ARB_texture_view: Add test suite

2013-10-15 Thread Matt Turner
On Mon, Oct 14, 2013 at 2:31 PM, Jon Ashburn j...@lunarg.com wrote: diff --git a/tests/spec/arb_texture_view/CMakeLists.gl.txt b/tests/spec/arb_texture_view/CMakeLists.gl.txt index c400759..293ef72 100644 --- a/tests/spec/arb_texture_view/CMakeLists.gl.txt +++