[FFmpeg-devel] Realmedia patch

Ronald S. Bultje rsbultje
Sun Sep 7 03:36:46 CEST 2008


Hi Luca,

On Fri, Sep 5, 2008 at 3:59 PM, Luca Abeni <lucabe72 at email.it> wrote:
> Ronald S. Bultje wrote:
> [...]
>> (only rtsp-data_parsing.patch to go and we have basic playback in svn!)
>
> I think the only controversial part of this patch was the
> +    if (rt->server_type == RTSP_SERVER_RDT) {
> +        int idx = (buf[0] < 0x40 || buf[0] > 0x42) ? 9 : 0;
> +        id = (buf[idx] >> 1) & 0x1f;
> +    }
> hunk, right?
>
> And if I understand well basic (audio only) playback will work even
> without this patch?
> If yes, I'd suggest to leave this chunk out for the moment.

So it's applied with it, basic playback of audio RTSP/Realmedia
streams works now.

I rewrote the hunk above locally as $attached, so the actual
bithandling is in rdt.c in a shared function (that is also used in
ff_rdt_parse_packet()). Take your time for review, like I said, I'll
work on MMS/QuicktimeRTSP for a while. Other patches are on my
website.

Later,
Ronald
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: rtsp-proper_rdt_idx.patch
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20080906/076ee9fe/attachment.asc>



More information about the ffmpeg-devel mailing list