[Piglit] [PATCH 2/4] copyteximage: add ARB_depth_texture dependencies

[Ilia Mirkin]

It does not appear to be tested. If you look in the test_vectors array:

{ GL_DEPTH_COMPONENT,   fcolor[7] },
{ GL_DEPTH_COMPONENT16, fcolor[7] },
{ GL_DEPTH_COMPONENT24, fcolor[7] },
{ GL_DEPTH_COMPONENT32F, fcolor[7] },

{ GL_DEPTH24_STENCIL8, fcolor[7] },
{ GL_DEPTH32F_STENCIL8, fcolor[7] },


On Sun, Mar 16, 2014 at 10:03 AM, Marek Olšák <maraeo at gmail.com> wrote:
> Did you forget GL_DEPTH_COMPONENT32 or is it not tested by this test?
>
> Marek
>
> On Thu, Mar 13, 2014 at 5:07 PM, Ilia Mirkin <imirkin at alum.mit.edu> wrote:
>> Signed-off-by: Ilia Mirkin <imirkin at alum.mit.edu>
>> ---
>>  tests/texturing/copyteximage.c | 7 ++++++-
>>  1 file changed, 6 insertions(+), 1 deletion(-)
>>
>> diff --git a/tests/texturing/copyteximage.c b/tests/texturing/copyteximage.c
>> index 0afcba5..85b2215 100644
>> --- a/tests/texturing/copyteximage.c
>> +++ b/tests/texturing/copyteximage.c
>> @@ -184,8 +184,13 @@ supported_format(GLenum format)
>>         case GL_COMPRESSED_RED:
>>         case GL_COMPRESSED_RG:
>>                 return piglit_is_extension_supported("GL_ARB_texture_rg");
>> +       case GL_DEPTH_COMPONENT:
>> +       case GL_DEPTH_COMPONENT16:
>> +       case GL_DEPTH_COMPONENT24:
>> +               return piglit_is_extension_supported("GL_ARB_depth_texture");
>>         case GL_DEPTH24_STENCIL8:
>> -               return piglit_is_extension_supported("GL_EXT_packed_depth_stencil");
>> +               return piglit_is_extension_supported("GL_EXT_packed_depth_stencil") &&
>> +                       piglit_is_extension_supported("GL_ARB_depth_texture");
>>         case GL_DEPTH_COMPONENT32F:
>>         case GL_DEPTH32F_STENCIL8:
>>                 return piglit_is_extension_supported("GL_ARB_depth_buffer_float");
>> --
>> 1.8.3.2
>>
>> _______________________________________________
>> Piglit mailing list
>> Piglit at lists.freedesktop.org
>> http://lists.freedesktop.org/mailman/listinfo/piglit