[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