[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