[FFmpeg-devel] [PATCH] Handle MP3ADU in RealRTSP, restructure rtpdec/rtsp handling of AVStream time_base

Luca Barbato lu_zero
Tue Dec 7 12:15:31 CET 2010


On 12/07/2010 11:29 AM, Martin Storsj? wrote:
> On Mon, 6 Dec 2010, Martin Storsj? wrote:
> 
>> On Mon, 6 Dec 2010, Luca Barbato wrote:
>>
>>> On 12/5/10 12:59 PM, Martin Storsj? wrote:
>>>> I've tested this change with quite a few different streams, and didn't see
>>>> any regression anywhere, but please do check if you know of any weird
>>>> stream that might break.
>>>
>>> It looks safe, from what I read the default if anything is present is missing
>>> (I just woke up so I can be wrong), but that shouldn't be an issue.
>>
>> Yes, I don't set the default explicitly any longer, but it's set to the 
>> same, 90 kHz, in av_new_stream anyway. If it would make things better, I 
>> could add it to be explicitly set within rtsp.c after creating the 
>> streams, so that we don't rely on defaults set anywhere else.
> 
> Ok to commit?

Looks like the email got lost between switches to and from macosx...

Ok from my side.

Ronald?

lu

-- 

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




More information about the ffmpeg-devel mailing list