[FFmpeg-devel] [PATCH] RDT/Realmedia patches #2

Michael Niedermayer michaelni
Sun Nov 30 22:15:35 CET 2008


On Sun, Nov 30, 2008 at 03:43:26PM -0500, Ronald S. Bultje wrote:
> Hi,
> 
> On Sun, Nov 23, 2008 at 11:53 AM, Ronald S. Bultje <rsbultje at gmail.com> wrote:
> > On Sun, Nov 23, 2008 at 11:17 AM, Ronald S. Bultje <rsbultje at gmail.com> wrote:
> >> On Sun, Nov 23, 2008 at 11:16 AM, Ronald S. Bultje <rsbultje at gmail.com> wrote:
> >>> This patch changes the prototype of rdt_demux_context_open() so that
> >>> it takes an index integer to the first AVStream pointed to by the
> >>> RTSPStream, matching the fact that one RTSPStream (in RDT) can have
> >>> multiple contained (AV)streams.
> >>
> >> And this patch uses that to remove the RDTDemuxContext->st and replace
> >> it by streams and n_streams to indicate that fact. I hope this fixes
> >> your worries about the st variable in RDTDemuxContext.
> >
> > Actually, I can do this without the extra "n" variable... See attached.
> 
> Attached is one updated against latest SVN. Same idea, it replaces
> RDTDemuxContext->st by an array of streams to indicate that one RDT
> set can contain multiple streams.

iam fine with the patch, iam also fine without the patch ...
aka apply if you like it better, dont if you dont

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Asymptotically faster algorithms should always be preferred if you have
asymptotical amounts of data
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20081130/074d2afa/attachment.pgp>



More information about the ffmpeg-devel mailing list