[FFmpeg-devel] [PATCH/RFC] Ryan Martell's native mms code

[Björn Axelsson]

On Thu, 2007-08-09 at 14:35 +0200, Reimar D?ffinger wrote:
> Hello,
> On Thu, Aug 09, 2007 at 02:24:08PM +0200, Bj?rn Axelsson wrote:
> > I have merged Ryan's native_mms work mentioned in an earlier mail with
> > the SVN trunk, to produce an easily reviewed patch. I have tried not to
> > change anything from Ryan's code except what was needed to make it
> > compile.
> > 
> > native_mms.c is mostly and will still require substantial work until it
> > is acceptable. It should probably also be split up into three files: one
> > for common functions, one for tcp/udp and one for http. 
> 
> Actually I don't see why it needs to be a demuxer at all instead of only
> a protocol. At least in MPlayer splitting it this way works (though
> stream selection admittedly is ugly and possibly not suitable in this
> way for ffmpeg).

I don't have enough knowledge about the ffmpeg internals to say whether
it is possible or not to implement it as a protocol instead. However, it
seems that rtsp is implemented as a demuxer and I would guess that
native_mms.c is based in part on rtsp.c.
If mms was implemented as a protocol, the asf demuxer would have to take
over the application interface for much of the functionality such as
playing, pausing and seeking with different cases for file and mms
access. If that is acceptable I'll give it a try and see if I run into
any problems.

> But more importantly I consider asx parsing implemented as demuxer
> unacceptable, and to be honest I think currently playlist parsing has no
> place at all in ffmpeg. If it is desperately wanted it needs a proper
> API first though.

The asx parsing currently implemented in native_mms.c is only meant to
handle redirection files. I'd say it is necessary, because many web
sites use .asx files to redirect to mms streams. It allows the browser
to download the .asx file and run e.g. ffplay on it without requiring a
mms: plugin for the browser.

-- 
Bj?rn Axelsson                    Phone: +46-(0)90-18 98 97
Intinor AB                          Fax: +46-(0)920-757 10
www.intinor.se
Interactive Television & Digital Media Distribution