[FFmpeg-devel] [PATCH] RDT/Realmedia patches #2
Mon Nov 3 18:42:27 CET 2008
On Sat, Oct 11, 2008 at 08:55:57AM -0400, Ronald S. Bultje wrote:
> 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
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
Size: 189 bytes
Desc: Digital signature
More information about the ffmpeg-devel