[MPlayer-dev-eng] [PATCH 11/11] MPlayer <-> GeeXboX patches port : Single binary

Diego Biurrun diego at biurrun.de
Mon Mar 13 00:43:30 CET 2006


On Sun, Mar 12, 2006 at 01:44:38PM +0100, Benjamin Zores wrote:
> On Fri, 24 Feb 2006 16:54:29 +0100
> Diego Biurrun <diego at biurrun.de> wrote:
> 
> > On Fri, Feb 24, 2006 at 04:33:42PM +0100, Benjamin Zores wrote:
> > > 
> > > I've sent to this list some times ago (nearly 6 month) a patch for
> > > having MPlayer and MEncoder sharing a part of their code (mainly
> > > global variables) which resulted in having MPlayer and MEncoder being
> > > a shared single and unified binary to avoid code duplication.
> > >
> > > Is someone still interested in it ?
> > 
> > Send it in.
> 
> Sorry for the delay.
> Here it is.

I don't really like it.  This is more intrusive than necessary and makes
single-binary the default case.  I will only accept a patch that adds it
as an option, off by default.

Diego




More information about the MPlayer-dev-eng mailing list