[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