[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