[Mesa-dev] [PATCH 1/3] main: silence missing return value warning in array_index_of_resource()

[Brian Paul]

On 04/21/2015 07:01 AM, Emil Velikov wrote:
> On 16/04/15 21:52, Brian Paul wrote:
>> ---
>>   src/mesa/main/shader_query.cpp | 1 +
>>   1 file changed, 1 insertion(+)
>>
>> diff --git a/src/mesa/main/shader_query.cpp b/src/mesa/main/shader_query.cpp
>> index b5f1d08..ad936e5 100644
>> --- a/src/mesa/main/shader_query.cpp
>> +++ b/src/mesa/main/shader_query.cpp
>> @@ -537,6 +537,7 @@ array_index_of_resource(struct gl_program_resource *res,
>>         return get_matching_index(RESOURCE_VAR(res), name);
>>      default:
>>         assert(!"support for resource type not implemented");
>> +      return 0;
> Both callers assume that 0 is a valid value. Perhaps return -1 so that
> the error paths can kick in ?

Yes, good point.  I'll fix that before pushing.

-Brian