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

[Martin Storsjö]

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

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

Rebased onto the latest version.

// Martin
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Split-out-the-part-of-rtp_parse_packet-that-does-the.patch
Type: text/x-diff
Size: 5117 bytes
Desc: 
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100926/4f9c7d62/attachment.patch>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0002-Reorder-received-RTP-packets.patch
Type: text/x-diff
Size: 9994 bytes
Desc: 
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100926/4f9c7d62/attachment-0001.patch>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0003-Return-a-queued-packet-if-it-has-been-in-the-queue-f.patch
Type: text/x-diff
Size: 3531 bytes
Desc: 
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100926/4f9c7d62/attachment-0002.patch>