[FFmpeg-devel] [PATCH] QCELP decoder

Diego Biurrun diego
Thu Oct 30 19:50:10 CET 2008


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.

Diego




More information about the ffmpeg-devel mailing list