[FFmpeg-devel] [PATCH] QCELP decoder

Kenan Gillet kenan.gillet
Thu Oct 30 19:44:24 CET 2008


On Oct 30, 2008, at 6:21 AM, Michael Niedermayer wrote:

> On Thu, Oct 30, 2008 at 10:59:56AM +0100, Diego Biurrun wrote:
> [...]
>>> --- libavcodec/celp_filters.c	(revision 15692)
>>> +++ libavcodec/celp_filters.c	(working copy)
>>> @@ -84,3 +84,24 @@
>>>
>>> +    for(n=0; n<buffer_length; n++)
>>> +    {
>>
>> You put the opening brace on the same line in other places.  And  
>> the for
>> expression could use some whitespace.
>>
>>> +        out[n] = in[n];
>>> +        for(i=1; i<filter_length; i++)
>>
>> ditto
>>
>>> --- libavcodec/celp_math.c	(revision 15692)
>>> +++ libavcodec/celp_math.c	(working copy)
>>> @@ -195,3 +195,14 @@
>>>
>>> +    for(i=0; i<length; i++)
>>
>> Some more whitespace will earn you extra good karma here.
>
> i do like the whitespace in the for as it is. Though i agree the {
> would look a little nicer in the same line as for.
> </bikeshed>

I have no strong opinion on this, I just tried to stay consistent with  
the code in the celp_*.[ch]
Let me know what you would prefer.

Kenan





More information about the ffmpeg-devel mailing list