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

[Michael Niedermayer]

On Sat, Oct 11, 2008 at 08:55:57AM -0400, Ronald S. Bultje wrote:
> Hi,
> 
> On Mon, Oct 6, 2008 at 10:59 PM, Ronald S. Bultje <rsbultje at gmail.com> wrote:
> > On Sat, Oct 4, 2008 at 12:39 AM, Ronald S. Bultje <rsbultje at gmail.com> wrote:
> >> Updated against SVN trunk after I committed all other patches.
> >
> > Ping. :-).
> 
> Not very succesful, it seems, so I'll try to split it, hoping it'll
> get reviewed. Attached is 2.4kb patch to implement rule-number
> parsing.

[...]

replying something because noone else does ...


> Index: ffmpeg-svn/libavformat/rdt.c
> ===================================================================
> --- ffmpeg-svn.orig/libavformat/rdt.c	2008-10-11 08:47:36.000000000 -0400
> +++ ffmpeg-svn/libavformat/rdt.c	2008-10-11 08:52:03.000000000 -0400
> @@ -39,7 +39,8 @@
>      AVStream *st;
>      void *dynamic_protocol_context;
>      DynamicPayloadPacketHandlerProc parse_packet;

> -    uint32_t prev_sn, prev_ts;
> +    uint32_t prev_sn, prev_ts, prev_rn;
> +    int rule;
>  };

what are all thouse variables? i can guess the meaning of prev ;) but
what is sn ts and rn?

and what is rule that is added?

(yes iam asking silly questions but by clarifying these it might become
 easier to review the code for me or others without having to spend time
 figuring out what everything is, also the code should become more readable
)

also what exactly does the patch do?

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

I have never wished to cater to the crowd; for what I know they do not
approve, and what they approve I do not know. -- Epicurus
-------------- 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/20081103/4848b860/attachment.pgp>