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

[Reimar Döffinger]

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)...
ndex: stream/stream_ffmpeg.c
===================================================================
--- stream/stream_ffmpeg.c	(revision 34832)
+++ stream/stream_ffmpeg.c	(working copy)
@@ -101,6 +101,13 @@
         goto out;
     }
 
+#ifdef AVIO_FLAG_DIRECT
+    flags |= AVIO_FLAG_DIRECT;
+#else
+    mp_msg(MSGT_OPEN, MSGL_WARN, "[ffmpeg] No support for AVIO_FLAG_DIRECT, might cause performance and other issues."
+                                 "Please update to and rebuild against a supported FFmpeg version.\n");
+#endif
+
     if (stream->url)
         filename = stream->url;
     else {