[FFmpeg-devel] [PATCH] RTP reordering, again
Martin Storsjö
martin
Fri Sep 24 20:06:05 CEST 2010
On Fri, 24 Sep 2010, Michael Niedermayer wrote:
> On Fri, Sep 03, 2010 at 12:11:09PM +0300, Martin Storsj? wrote:
> > Hi,
> >
> > This is a respin of the RTP reordering patches from May. Ronald was quite
> > ok with the actual reordering logic (as far as I remember), but Michael
> > wanted to make sure we could set a maximum delay for the reordering. Back
> > then, I made one attempt at this, but I've rethought the max delay logic
> > now, and I'd say this new attempt is both better and much more straight
> > forward than the previous one.
> >
> > In order to set max_delay for demuxers, I add an AVOption for that field.
> > Without setting it (when max_delay is 0), no packet reordering will be
> > done, packets are passed straight through.
> >
> > Attached is also a patch (not for review) for making rtpenc send packets
> > in non-linear order, just for testing this. I'm happy to say that our code
> > manages to decode the packets sent in non-linear order just fine with this
> > reordering code, as does QuickTime. VLC, with their current RTSP lib,
> > didn't work perfectly, though.
> >
> > // Martin
> [...]
>
>
> > options.c | 1 +
> > 1 file changed, 1 insertion(+)
> > 4d86fc74ea809aab738ec0d01f72d65a6a20af20 0004-Add-an-AVOption-max_delay-for-AVFormatContext-max_de.patch
> > From 52a0335b0b3dfd617a6dea514e485c7c402edaa5 Mon Sep 17 00:00:00 2001
> > From: Martin Storsjo <martin at martin.st>
> > Date: Fri, 3 Sep 2010 11:11:58 +0300
> > Subject: [PATCH 4/4] Add an AVOption max_delay for AVFormatContext->max_delay
> >
> > This can currently also be set via -muxdelay in ffmpeg for muxers,
> > but not for demuxers (nor for demuxers in ffplay) - this patch
> > allows it to be set in all those cases.
>
> ok
Thanks, applied.
> and sorry for the delay, i saw "not for review" in the mail and skiped it i
> think
Ok, no problem. :-)
// Martin
More information about the ffmpeg-devel
mailing list