[Piglit] [PATCH 6/9] arb_program_interface_query/resource-query: tess tests need subroutines

[Marek Olšák]

I don't think so. This patch fixes the test when tessellation is
available but subroutines aren't.

Marek

On Wed, Jun 24, 2015 at 9:24 AM, Tapani Pälli <tapani.palli at intel.com> wrote:
>
>
> On 06/16/2015 11:57 PM, Marek Olšák wrote:
>>
>> From: Marek Olšák <marek.olsak at amd.com>
>>
>> ---
>>   tests/spec/arb_program_interface_query/resource-query.c | 3 ++-
>>   1 file changed, 2 insertions(+), 1 deletion(-)
>>
>> diff --git a/tests/spec/arb_program_interface_query/resource-query.c
>> b/tests/spec/arb_program_interface_query/resource-query.c
>> index 1db5585..c131891 100755
>> --- a/tests/spec/arb_program_interface_query/resource-query.c
>> +++ b/tests/spec/arb_program_interface_query/resource-query.c
>> @@ -646,7 +646,8 @@ check_extensions(const struct subtest_t st)
>>              st.programInterface == GL_TESS_CONTROL_SUBROUTINE_UNIFORM ||
>>              st.programInterface == GL_TESS_EVALUATION_SUBROUTINE_UNIFORM
>> ||
>>              st.tcs_text || st.tes_text) &&
>> -            !piglit_is_extension_supported("GL_ARB_tessellation_shader"))
>> {
>> +           (!piglit_is_extension_supported("GL_ARB_shader_subroutine") ||
>> +
>> !piglit_is_extension_supported("GL_ARB_tessellation_shader"))) {
>
>
>
> This same check is already made a few lines above. It first checks against
> all enums that require subroutine and only then for tessellation.
>
>>                  return false;
>>          }
>>
>>
>
> // Tapani