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

Martin Storsjö martin
Fri Sep 24 08:54:27 CEST 2010


On Fri, 3 Sep 2010, Martin Storsj? wrote:

> 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.

Ping, Ronald and Luca B, do you think this looks sane? In particular, the 
code for enforcing a maximum reordering delay, is that ok?

Michael, is the new added AVOption for max_delay acceptable (patch #4)?

// Martin



More information about the ffmpeg-devel mailing list