[MPlayer-dev-eng] [rfc] committing rules and other policies

Michael Niedermayer michaelni at gmx.at
Mon Feb 26 00:02:32 CET 2007


On Mon, Feb 26, 2007 at 12:01:27AM +0200, Uoti Urpala wrote:
> On Sun, 2007-02-25 at 15:30 -0500, Rich Felker wrote:
> > Then, there's a second inline asm syntax that keeps changing with
> > every "modern" release of gcc. Following this new, moving target is
> > akin to using the C++ language or writing Linux-kernel software. The
> > rules keep changing under you.
> >
> > Here the answer is much less clear. There's no standard to go by. What
> > MPlayer has been doing for x86, and what I believe is the correct
> > solution according to the principle that forced upgrades are bad, is
> > to follow the interface that's been stable for 10+ years, and restrict
> > using constructs that worked in that old interface but that have been
> > broken by recent instability.
> Portraying gcc-2.95 as the "stable interface" and everything else as
> "new, moving target" that "keeps changing with every release" is
> dishonest. For example the new feature that probably creates the biggest
> single improvement in asm readability and modifiability over gcc-2.95,
> named asm arguments, is unlikely to disappear in future versions.

as ive said before you can do named arguments with the preprocessor
and that works fine with gcc 2.95

this is all about weighting the advantages against the dissadvantages
and cpp named asm syntax vs. gcc 3 named asm syntax and randomly placing
your int i; does not seem to me like a huge improvement
the disadvantage is loosing gcc 2.95 support which alot of developers dont
seem to want to loose and possible causing problems for non gcc compilers
also some people dream of having mplayer compiled with tinycc or another
new compiler, moving all the source to the latest C99 and gcc 3 features
is not going to make that easier


Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Democracy is the form of government in which you can choose your dictator
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20070226/6c128fda/attachment.pgp>

More information about the MPlayer-dev-eng mailing list