[FFmpeg-devel] [PATCH] Replace hardcoded LP filter order with parameter in lsp.c

Måns Rullgård mans
Sun May 11 13:25:53 CEST 2008


Diego Biurrun <diego at biurrun.de> writes:

> On Sun, May 11, 2008 at 09:28:25AM +0700, Vladimir Voroshilov wrote:
>> 
>> This should be better.
>> 
>> --- a/libavcodec/lsp.c
>> +++ b/libavcodec/lsp.c
>> @@ -28,30 +28,30 @@
>>  
>> -    for(i=0; i<9; i++)
>> +    for(i=0; i<lp_order-1; i++)
>
> Nit: Can we maybe get an extra space after the ';' there? :)

There are already spaces after the ;.  Some of the other operators
could sure use some surrounding spaces though.

-- 
M?ns Rullg?rd
mans at mansr.com




More information about the ffmpeg-devel mailing list