[Ffmpeg-cvslog] r7617 - in trunk: libavcodec/allcodecs.c libavformat/allformats.c

[Reimar Doeffinger]

Hello,
On Tue, Jan 23, 2007 at 10:29:49AM +0100, Diego Biurrun wrote:
> On Mon, Jan 22, 2007 at 02:41:23PM +0000, M?ns Rullg?rd wrote:
> > Evgeniy Stepanov said:
> > > On Monday 22 January 2007 00:04, mru wrote:
> > >>
> > >> Modified:
> > >>    trunk/libavcodec/allcodecs.c
> > >>    trunk/libavformat/allformats.c
> > >>
> > >> Log:
> > >> remove no longer needed #ifdefs
> > >
> > > This change breaks mplayer compilation. Ifdef's are still used there.
> > 
> > Perhaps it's time mplayer started calling the ffmpeg configure script instead
> > of attempting to replicate the functionality.  Diego?
> 
> It's an idea (and one that looks increasingly appealing), but it would
> require including all of FFmpeg in MPlayer.  I think it would mean more
> than a megabyte of code.  It might be worth it, though...

Calling ffmpeg's configure with it's own autodetection seems likely to
cause quite a few bugs/weird behaviour to me.
On the other hand, maybe it would help to just reuse some parts of it?
Maybe even like splitting some parts into seperate files (preferable in
a subdir for easy use as external) would help...
Just an easy way to remove or add stuff to the list of codecs, demuxers
etc. to be built would be enough to at least allow me to make a "hack"
that makes MPlayer build again...

Greetings,
Reimar D?ffinger