[MPlayer-cvslog] CVS: main/libmpdemux ebml.c,1.4,1.5

[Alexander Strasser]

Rich Felker wrote:
> On Wed, Jul 20, 2005 at 10:35:57AM +0200, Alexander Strasser wrote:
> > Reimar Döffinger wrote:
> > > On Wed, Jul 20, 2005 at 04:05:44AM +0200, Diego Biurrun wrote:
> > > > On Tue, Jul 19, 2005 at 11:14:46PM +0200, Reimar Döffinger wrote:
> > > > > Since internal libavcodec still is not mandatory
> > > > 
> > > > I would consider that news...
> > > 
> > > Well, we are still testing for it, aren't we? And I can't see any other
> > > reason why we have all these
> > > #if #if LIBAVCODEC_BUILD < ...
> > > things around.
> > > Also I said _internal_ specifically, using the .so might give a few
> > > problems as well (mostly because of non-matching versions).
> > > I think Ivan was suggested making it mandatory some time ago, but
> > > somebody else disagreed.
> > 
> >   I don't consider it news, it is reality and mplayer does really
> > `work' without libavcodec! I was suggesting making it mandatory
> > short time ago, and had a short discussion with some devels on IRC
> > about it, but it didn't came to a real conclusion anyways.
> >   Maybe we should reconsider this again, as more and more parts of
> > libavcodec would be needed in mplayer to avoid code duplication.
> >   Though i still wonder sometimes if it wouldn't be possible to
> > split some of the more generic parts of libavcodec into e.g. an
> > independent subdir, so we could just make that part mandatory.
> > Comments welcome. (BTW mandatory means configure fails when the
> > object in question could not be found.)
> 
> I'm mildly against this. If we want to make parts of lavc mandatory we
> should just make the whole thing mandatory. MPlayer is basically
> useless without it anyway..

  I have no strong oppinion on this. I don't have a real problem
with making lavc mandatory as it is becoming more and more modular
anyway and one is/will be able to deactivate most things.

  Anyway I am already working on splitting some parts as i consider
it a good thing as it improves structure and it might be useful for
other projects.

  Alex (beastd)