[FFmpeg-cvslog] r19695 - trunk/libavcodec/twinvq.c

Diego Biurrun diego
Wed Sep 2 11:19:58 CEST 2009


On Fri, Aug 28, 2009 at 01:11:53PM +0200, Michael Niedermayer wrote:
> On Tue, Aug 25, 2009 at 10:50:50AM +0200, Diego Biurrun wrote:
> > On Tue, Aug 25, 2009 at 04:16:37AM +0200, Vitor Sessak wrote:
> > > Diego Biurrun wrote:
> [...]
> > > >>--- trunk/libavcodec/twinvq.c	Mon Aug 24 15:43:33 2009 (r19694)
> > > >>+++ trunk/libavcodec/twinvq.c	Mon Aug 24 19:35:47 2009 (r19695)
> > > >>@@ -458,7 +458,7 @@ static void add_peak(int period, int wid
> > > >> 
> > > >>-    for (j=-width/2; j < (width+1)/2 && shape < shape_end; j++)
> > > >>+    for (j=-width/2; j < (width + 1)/2 && shape < shape_end; j++)
> > > >
> > > >There are still some operators on this line that could benefit from
> > > >whitespace, especially / and =.
> > > 
> > > I try to stick to K&R for most of the things but it goes a little too 
> > > far on extra whitespaces for my taste. I particularly hate doing
> > > 
> > > for (i = 0; i < 10; i++)
> > > 
> > > instead of
> > > 
> > > for (i=0; i < 10; i++)
> > 
> > That's the good thing about common styles: Nobody gets to be the
> > privileged person whose style gets picked.  Everybody has to adapt.
> 
> There is nothing good on "Everybody has to adapt"

You misunderstand.  It is much better to make everybody get used to a
common style instead of picking favorites and declaring that everybody
has to adopt the style used by one person like you or me or whomever.

> If it where we could go the full way to gnu style, for that people would
> have to adapt alot more and fight their better judgement also alot more.

That's why we are using a common style that has sensible defaults: K&R.
It's also one of or even the most commonly used style, so people have to
adapt less.

> > Rest assured that there are parts of K&R I dislike as well.  But it's no
> > use complaining.  After a while you adapt and don't consider it
> > obnoxious anymore.  Tastes *are* malleable.
> 
> But dislike is an indication that there are problems, and
> i claim ommiting the spaces around * / is more readable. Also in
> mathematics with pencil and paper there are no spaces around * and / and
> actually * is not written at all its 
> A = 4ab + cd
> not 
> A = 4 * a * b + c * d
> And i think there is little question which is more readable and convenient
> 4ab is a more "strongly" bound unit than 4ab + cd so it makes sense to also
> use spaces differently

In math it is clear because numbers and letters are different
namespaces.  In C both are just parts of a string.

With pencil and paper you do put space around / and/or make it exceed
the height of the letters.  Mostly you write fractions, then it's
absolutely clear.

Diego



More information about the ffmpeg-cvslog mailing list