[MPlayer-dev-eng] [PATCH] ad_ffmpeg does not honor log level

Diego Biurrun diego at biurrun.de
Sat Sep 4 10:11:35 CEST 2010


On Sun, Aug 22, 2010 at 08:16:52AM +0200, Reimar Döffinger wrote:
> On Sun, Aug 22, 2010 at 12:24:28AM -0400, Vlad Seryakov wrote:
> > On Aug 21, 2010, at 2:27 PM, Reimar Döffinger wrote:
> > 
> > > Please don't spend more time on this, it is a really bad quick hack.
> > > We need a proper "intialize ffmpeg" function that is called everywhere,
> > > not this piecemeal "initialize a bit here, a bit there", then our own
> > > callback can be installed from there (like vd_ffmpeg already does,
> > > but obviously only if we use an FFmpeg video decoder) and fix this properly.
> > > 
> > 
> > Will this work?
> 
> I would think so (though it still leaves the same issue for libavformat for the
> demuxer lavf and ffmpeg:// stream messages), but of course it needs testing.
> In the long term I think it would be better to move the initialization
> function to a global location, call it once from MPlayer and mencoder
> and initialize both libavformat and libavcodec, but I don't mind this
> as a first hack.

Applied a fixed version.

Diego


More information about the MPlayer-dev-eng mailing list