[FFmpeg-devel] [PATCH] MMS wrapper

[Stefan Gehrer]

On 07/14/2012 01:49 PM, Geek.Song wrote:

> On Sat, Jul 14, 2012 at 4:41 AM, Stefan Gehrer <stefan.gehrer at gmx.de> wrote:
>> attached patch implements a simple wrapper for handling mms:// URLs.
>> It first tries to open using mmst protocol and on failure tries
>> using mmsh protocol.
>> I found some streams on tunein.com which either support only
>> mmst or only mssh, with this patch applied both kinds can be
>> played back with just specifying mms:// as protocol.
>> This should fix ticket #998, the mms URL given there also works.
>> mms://live1.wm.skynews.servecast.net/skynews_wmlz_live300k
 >
 > Dear:
 >
 > I think you over-design the implementation, I just modify the
 > avio_open2() to probe mmsh or mmst.
 >

I am all ears for solutions with the least amount of code, so
I formed your suggestion into a patch (with slight modifications)
and attached. If in the future you post code to this list, and I
hope you do, please also do so in patch form (unified diff, generated
by diff -u).

The disadvantage of your solution is that protocol-specific code is
put into generic parts which may make it unacceptable, but that is
not my decision.

Stefan

P.S. please do not top-post, but instead put your reply at the bottom
of the text you are replying to.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: mms_switch.diff
Type: text/x-patch
Size: 1689 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20120714/1986fe4d/attachment.bin>