[MPlayer-dev-eng] [bikeshed] Coding style

[Michael Niedermayer]

On Tue, Jun 17, 2008 at 12:39:04AM +0200, Diego Biurrun wrote:
> On Mon, Jun 16, 2008 at 08:39:33PM +0200, Benjamin Zores wrote:
> > Uoti Urpala a écrit :
> > > On Mon, 2008-06-16 at 13:45 +0200, Luca Barbato wrote:
> > >> Given that some people seems to be fine with the cleanup weekend here 
> > >> something to start a nice flame:
> > >>
> > >> We could decide a coding style and try to be consistent as much as possible.
> > >>
> > >> I like trying to leave spaces when possible but I like better having 
> > >> *pointer, conditional blocks should keep the bracket in the same line, 
> > >> functions the line immediately after.
> > > 
> > > Standard K&R style as generated by the -kr option of the indent program,
> > > without any "personal" variations. Some deviations can be allowed but
> > > they should be viewed as that, deviations from the recommended style.
> > > Trying to create an MPlayer-specific style is a bad idea which will
> > > cause unnecessary arguments without any real benefit and with real harm
> > > such as making the style less obvious to people not familiar with
> > > MPlayer.
> > 
> > I tend to agree on that.
> > We might choose one well-known coding style because, obviously,
> > there is no chance that everyone agree on something to define an MPlayer 
> > one.
> > 
> > Sure, some might not like the chosen one but it's a matter of tradeoff
> > and can't be worse that the current situation.
> 
> Seconded (or rather thirded).

I object, we should agree on the style most developers like best. The style
we would choose is what people here will have to live with for quite some
time. If a overwhelming majority dislikes some detail of K&R it would be
simply wrong to ignore that.
Iam not saying iam against indent -kr style, iam just saying that it is
very wrong to not vote on each issue individually. But rather to bundle
them up in what the person in charge would prefer to see ...

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

When the tyrant has disposed of foreign enemies by conquest or treaty, and
there is nothing more to fear from them, then he is always stirring up
some war or other, in order that the people may require a leader. -- Plato
-------------- 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/20080617/cba7dc5a/attachment.pgp>