[FFmpeg-devel] [PATCH] Frame rate emulation

[Michael Niedermayer]

Hi

On Thu, Aug 09, 2007 at 08:21:12PM -0300, Ramiro Ribeiro Polla wrote:
> Ramiro Ribeiro Polla wrote:
>> Michael Niedermayer wrote:
>>   
>>> Hi
>>>
>>> On Sun, Jun 17, 2007 at 06:59:30PM -0300, Ramiro Ribeiro Polla wrote:
>>>     
>
> [...]
>
>>>> @@ -270,8 +263,8 @@
>>>>      st->codec->codec_id = CODEC_ID_RAWVIDEO;
>>>>      st->codec->width = width;
>>>>      st->codec->height = height;
>>>> -    st->codec->time_base.den      = frame_rate;
>>>> -    st->codec->time_base.num = frame_rate_base;
>>>> +    st->codec->time_base.den = ap->time_base.den;
>>>> +    st->codec->time_base.num = ap->time_base.num;
>>>>           
>>> st->codec->time_base= ap->time_base;
>>>
>>> except these and a missing minor version increase the patch looks ok
>>> asuming you did test the changes grabing demxuers (AV sync amongth others
>>> should not get worse)
>>>
>>>       
>>
>> I won't apply this (at least for now) since the discussion seems to be 
>> pointing to be a better solution (or looking for a better solution).
>>
>>   
>
> Updated patches introduce AVRateEmu, and use it with AVFMT_FLAG_NONBLOCK in 
> v4l and x11grab. Tested with r9876 reverted, and works the same with v4l 
> and x11grab.

what about moving the av rate emu loop to av_read_frame_internal()
that would avoid duplicating it for every grab demuxer


[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

There will always be a question for which you do not know the correct awnser.
-------------- 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/20070811/b76dfc97/attachment.pgp>