[MPlayer-dev-eng] [PATCH] factorize print_version()

Diego Biurrun diego at biurrun.de
Sun Jan 25 17:03:56 CET 2009


On Tue, Jan 13, 2009 at 09:44:17AM +0100, Diego Biurrun wrote:
> On Tue, Jan 13, 2009 at 02:10:23AM +0200, Uoti Urpala wrote:
> > On Tue, 2009-01-13 at 00:36 +0100, Diego Biurrun wrote:
> > > On Tue, Jan 13, 2009 at 01:25:44AM +0200, Uoti Urpala wrote:
> > > > On Mon, 2009-01-12 at 23:51 +0100, Diego Biurrun wrote:
> > > > > On Tue, Jan 13, 2009 at 12:37:06AM +0200, Uoti Urpala wrote:
> > > > > > The code can be moved back later reasonably easily, but is it really
> > > > > > beneficial even in the short term? MEncoder is quite rotten already. As
> > > > > > long as MEncoder is not really maintained MPlayer quality is what
> > > > > > matters, and IMO it's worsened by changes like this.
> > > > > 
> > > > > I think the change is worth the trouble.
> > > > > 
> > > > > It will make my life easier when I fix bug 1378 and it will make my
> > > > > life easier when I update the copyright notice next year.  So I think
> > > > > it's beneficial in the short term and not a big deal in the long run.
> > > > 
> > > > That sounds like less benefit than even just the work needed to do the
> > > > change itself...
> > > 
> > > The work is done so the point is moot IMO.
> > > And to me it keeps sounding like the possible future trouble of moving
> > > code around again is even smaller than the current benefit.
> > 
> > Did you understand what I wrote about the negative effects?
> 
> I understood you and I disagree.  Do we have opinions from others?

Anybody?  I'd still like to commit this.  I'm also open for moving this
to an entirely different file.

Diego



More information about the MPlayer-dev-eng mailing list