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

Reimar Döffinger Reimar.Doeffinger
Thu Aug 9 14:35:01 CEST 2007


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).
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.

Greetings,
Reimar D?ffinger




More information about the ffmpeg-devel mailing list