[MPlayer-dev-eng] [PATCH] ao_dart

Michael Niedermayer michaelni at gmx.at
Sun Mar 1 18:33:18 CET 2009


On Sun, Mar 01, 2009 at 02:13:38PM +0100, Reimar Döffinger wrote:
> On Sun, Mar 01, 2009 at 02:10:58PM +0100, Diego Biurrun wrote:
> > On Sun, Mar 01, 2009 at 09:59:30PM +0900, KO Myung-Hun wrote:
> > > 
> > > Diego Biurrun wrote:
> > > >> But, now the closing bracket '}' of last case is placed on the same
> > > >> depth with the closing bracket '}' of switch. I think it's not good
> > > >> style.
> > > >
> > > > Just look at what K&R really does: Indent the opening brace after 'case'.
> > > 
> > > How do I that in this case ?
> > > 
> > >     case 1234:
> > >         {
> > >         if (true) {
> > >         }
> > >         break;
> > >         }
> > > 
> > > Just do that ?
> > 
> > Fine with me.
> 
> And that's supposed to be not ugly?

Its supposed to be asshatry.

Like ive said in the past, following K&R style in a way that is 5 magnitudes
more strict than K&R themselfs ever suggested is plain idiotic. AFIAK they
said something like pick a style and follow it not pick my style and follow
it in all corner cases that ive never thought about. But then i could be
mistaken ive not read the book just a few quotes of it ...

And besides its not K&R but a indent tool you try to follow. If you guys
where quoting K&R that at least had a hint of sanity ...

anyway, i wont disturb you guys further with this as iam busy with writing
software and fixing bugs. So i wish you alot of pleasure and fun being pets
of your dictators. Remember you MUST follow K&R style no matter what!

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

I know you won't believe me, but the highest form of Human Excellence is
to question oneself and others. -- Socrates
-------------- 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/mplayer-dev-eng/attachments/20090301/8bf9fa7e/attachment.pgp>


More information about the MPlayer-dev-eng mailing list