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

Luca Barbato lu_zero
Tue May 18 22:25:00 CEST 2010


On 05/18/2010 12:52 PM, Martin Storsj? wrote:
> On Tue, 18 May 2010, Martin Storsj? wrote:
> 
>> On Tue, 18 May 2010, Luca Barbato wrote:
>>
>>> On 05/17/2010 09:34 PM, Martin Storsj? wrote:
>>>> +            rt->lower_transport == RTSP_LOWER_TRANSPORT_TCP ? 0 : 10);
>>>
>>> 10?
>>
>> 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...

lu

-- 

Luca Barbato
Gentoo/linux
http://dev.gentoo.org/~lu_zero




More information about the ffmpeg-devel mailing list