[FFmpeg-devel] [PATCH] QCELP decoder
Kenan Gillet
kenan.gillet
Thu Oct 30 20:11:01 CET 2008
On Oct 30, 2008, at 11:50 AM, Diego Biurrun wrote:
> On Thu, Oct 30, 2008 at 11:44:24AM -0700, Kenan Gillet wrote:
>>
>> 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_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.
>
> You formatted it as
>
> for(i = 0; i < length; i++)
>
> in other places, so you should be consistent. And yes, I find the
> latter version more readable.
I do like some spaces too, so changed
and I like my bikeshed painted orange ;)
Kenan
More information about the ffmpeg-devel
mailing list