[Mesa-dev] [PATCH 4/8] st/va: fix unused variable warning

Christian König deathsimple at vodafone.de
Tue Dec 15 02:26:09 PST 2015


On 15.12.2015 11:08, Emil Velikov wrote:
> On 11 December 2015 at 12:33, Christian König <deathsimple at vodafone.de> wrote:
>> From: Christian König <christian.koenig at amd.com>
>>
>> Signed-off-by: Christian König <christian.koenig at amd.com>
>> ---
>>   src/gallium/state_trackers/va/picture.c | 1 -
>>   1 file changed, 1 deletion(-)
>>
>> diff --git a/src/gallium/state_trackers/va/picture.c b/src/gallium/state_trackers/va/picture.c
>> index 8623139..7b30bf8 100644
>> --- a/src/gallium/state_trackers/va/picture.c
>> +++ b/src/gallium/state_trackers/va/picture.c
>> @@ -92,7 +92,6 @@ vlVaGetReferenceFrame(vlVaDriver *drv, VASurfaceID surface_id,
>>   static VAStatus
>>   handlePictureParameterBuffer(vlVaDriver *drv, vlVaContext *context, vlVaBuffer *buf)
>>   {
>> -   unsigned int i;
>>      VAStatus vaStatus = VA_STATUS_SUCCESS;
>>
> Can I bribe you to also remove the "set once, used once" variable vaStatus ?

Unfortunately I already pushed this one yesterday after Julien gave me 
his rb. But going to keep that in mind when I touch the function the 
next time.

Regards,
Christian.

>
> Either way
> Reviewed-by: Emil Velikov <emil.l.velikov at gmail.com>
>
> -Emil



More information about the mesa-dev mailing list