[FFmpeg-devel] [PATCH][0/4]: MMS base and MMSH implementation

[Björn Axelsson]

On Sun, 2008-02-03 at 12:48 +0100, Reimar D?ffinger wrote:
> Hello,
> On Thu, Dec 13, 2007 at 11:03:46PM +0100, Michael Niedermayer wrote:
> > a totally hypothetical and probably not applicable variant would be
> > 1. a patch for just establishing the connection
> > 2. a patch for reading packets
> > 3. a patch which adds seeking
> > 4. a patch which adds pause support
> > 5. a patch which adds support to send back connection statistics to the servr

I have had some time between other projects to look into this again.

The attached mmsh patch has no state machine (which didn't make much
sense for mmsh anyway), and is reduced to only include points 1 and 2
from above.
Also, the HTTP parser and packet reader has been through a few more
iterations and cleanups.
Hopefully this brings it into the realm of reviewability...

A refreshed build patch is also included for anyone who wants to test
it.

> I wanted to see if I could do such splitting (actually just removing 3
> and 4 to see if the state machine can be get rid of), but I do not have
> a stream that works with it.
> I tried the first best thing that google could find:
> mmsh://wm.interoute.com/{98f50251-8adb-4429-af0f-60ee34e6cd57}/{d2f6a347-0e2f-4ac1-a375-1d9d6e295c0f}/qotsa_wieLive.wmv
> It plays with MPlayer but not with ffplay.
> Messages:
> > [mmsh @ 0x79ff00]read before play. Playing automatically.
> > [mmsh @ 0x79ff00]Data chunk larger than buffer (8948 > 8192)
> > [mmsh @ 0x79ff00]Got packet 0x81 in the wrong state: AWAITING_STREAM_ID_ACCEPTANCE
> 
> I might be missing a part of the patch though, not sure, I applied this
> one and the build system stuff.

No, it was just that the packet size of the stream was larger than the
hardcoded buffer size. I have increased the buffer size considerably in
this version, but ultimately it should be dynamically allocated. The
attached patch plays the stream above.

-- 
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
-------------- next part --------------
A non-text attachment was scrubbed...
Name: mms_base_and_http.diff
Type: text/x-patch
Size: 31501 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20080306/f26763d2/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: mmsh_build_enable.diff
Type: text/x-patch
Size: 1467 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20080306/f26763d2/attachment-0001.bin>