[FFmpeg-devel] [PATCH] RTP reordering, again

Michael Niedermayer michaelni
Fri Sep 24 19:48:44 CEST 2010


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
and sorry for the delay, i saw "not for review" in the mail and skiped it i
think

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

If a bugfix only changes things apparently unrelated to the bug with no
further explanation, that is a good sign that the bugfix is wrong.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100924/df49c597/attachment.pgp>



More information about the ffmpeg-devel mailing list