[FFmpeg-cvslog] r11437 - trunk/libavformat/rmdec.c

[Michael Niedermayer]

On Mon, Jan 07, 2008 at 01:05:01AM +0100, Aurelien Jacobs wrote:
> M?ns Rullg?rd wrote:
> 
> > diego <subversion at mplayerhq.hu> writes:
> > 
> > > Author: diego
> > > Date: Sun Jan  6 23:50:09 2008
> > > New Revision: 11437
> > >
> > > Log:
> > > cosmetics: Add another pair of parentheses to aid readability.
> > >
> > > Modified:
> > >    trunk/libavformat/rmdec.c
> > >
> > > Modified: trunk/libavformat/rmdec.c
> > > ==============================================================================
> > > --- trunk/libavformat/rmdec.c	(original)
> > > +++ trunk/libavformat/rmdec.c	Sun Jan  6 23:50:09 2008
> > > @@ -716,7 +716,7 @@ resync:
> > >          if (ff_rm_parse_packet (s, st, len, pkt, &seq, &flags,
> > > &timestamp) < 0) goto resync;
> > >
> > > -        if(flags&2 && (seq&0x7F) == 1)
> > > +        if( (flags&2) && (seq&0x7F) == 1 )
> > 
> > Aaaarrgghh!  PLEASE, don't put whitespace right after ( or right
> > before ).  It looks so hideous.
> 
> +1

me too ;)

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

I have never wished to cater to the crowd; for what I know they do not
approve, and what they approve I do not know. -- Epicurus
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-cvslog/attachments/20080107/a19ef165/attachment.pgp>