[MPlayer-cvslog] r34824 - trunk/stream/stream_ffmpeg.c

[Reimar Döffinger]

On Wed, Apr 04, 2012 at 04:48:39PM +0200, Diego Biurrun wrote:
> On Tue, Mar 27, 2012 at 09:55:57PM +0200, Reimar Döffinger wrote:
> > On Tue, Mar 27, 2012 at 07:39:39PM +0200, Diego Biurrun wrote:
> > > On Sat, Mar 24, 2012 at 08:59:34PM +0100, Reimar Döffinger wrote:
> > > > And sorry for the grumpiness, but I've mentioned the issues with the
> > > > new official API several times already and for FFmpeg all the necessary
> > > > url_ symbols are exported again (though the header needs to be copied
> > > > manually still). So I am annoyed about it being applied with only
> > > > about a day's notice.
> > > 
> > > No worries, I will revert it shortly, was busy the past few days.
> > 
> > I've been working on a flag on the FFmpeg side that should disable
> > the buffering and might make this acceptable.
> > Problem is I need to come up with a way to test it properly.
> > The patch required on MPlayer side would be the below, though
> > the write function still needs to be fixed (needs to return error,
> > explicitly call flush for FFmpeg versions that don't have the flag)...
> 
> Revert or not then?

I'd say it's ok to leave it be as-is for a little bit longer.