[FFmpeg-devel] [RFC] rtpdec: Reordering RTP packets

Michael Niedermayer michaelni
Wed May 19 02:17:19 CEST 2010


On Tue, May 18, 2010 at 11:37:57PM +0300, Martin Storsj? wrote:
> On Tue, 18 May 2010, Luca Barbato wrote:
> 
> > >> Default reorder queue size - the idea was to move the decision on the 
> > >> queue size out of rtpdec, so that it could be dynamically choosable.
> > >> For now there's no such code, but the decision on the queue size is 
> > >> outside of rtpdec at least.
> > >>
> > >> This perhaps still could be a define (where should it be in that case, 
> > >> rtpdec.h?) even if it isn't hardcoded as a static array in RTPDemuxContext 
> > >> as in the previous attempt.
> > > 
> > > Changed to a define now.
> > 
> > I'd consider it as rtp proto param, but I know will be annoying
> > forwarding it from rtsp://path/to/resource?buffer=10&tcp to it but might
> > be something nice for downstream users. Otherwise use the AVOption...
> 
> Yes, Ronald suggested something such, too. Although, I see that as a 
> separate feature that can be added later, after the general reordering 
> support within rtpdec...

iam scared

the user cares about 1 thing and that is delay, not packet number which is
just a meaningless number, unfit to be constant by default

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Democracy is the form of government in which you can choose your dictator
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100519/bcae26f5/attachment.pgp>



More information about the ffmpeg-devel mailing list