On 25/02/13 18:09, Diego Biurrun wrote:
> On Mon, Feb 25, 2013 at 10:41:02AM +0100, Luca Barbato wrote:
>> The value is within 0 and 32.
>>
>> Remove an `array subscript is below array bounds` warning.
>> --- a/libavutil/lls.c
>> +++ b/libavutil/lls.c
>> @@ -48,7 +48,7 @@ void avpriv_update_lls(LLSModel *m, double *var, double 
>> decay)
>>  
>> -void avpriv_solve_lls(LLSModel *m, double threshold, int min_order)
>> +void avpriv_solve_lls(LLSModel *m, double threshold, unsigned short 
>> min_order)
> 
> Why not uint16_t?  It seemed to me that "short" was going out of fashion.
> 

too lazy to add the proper header to use it and short describes better
what you want.

let me first know if the concept is fine, then we could use uint16_t
instead of unsigned short.

lu

_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to