[MPlayer-dev-eng] [PATCH] automatic fps for mpeg4 raw/in TS

[Nico Sabbi]

Nico Sabbi wrote:

> Nico Sabbi wrote:
>
>> Michael Niedermayer wrote:
>>
>>> Hi
>>>
>>> On Wednesday 05 January 2005 23:12, Nico Sabbi wrote:
>>>  
>>>
>>>> Michael Niedermayer wrote:
>>>>  
>>>>
>>>>> Hi
>>>>>
>>>>> On Wednesday 05 January 2005 22:09, Nico Sabbi wrote:
>>>>>    
>>>>>
>>>>>> Hi,
>>>>>> this patch permits to determine automatically the initial fps of 
>>>>>> mpeg4
>>>>>> video in the VOL header,
>>>>>> so there's no need to specify it with -fps.
>>>>>>
>>>>>> Michael, can you please check it?
>>>>>>       
>>>>>
>>>>>
>>>>> did u try it with a few different framerates like a 24000/1001 
>>>>> .m4v stream
>>>>> generated by ffmpeg? i doubt it will work
>>>>>
>>>>> [...]
>>>>>     
>>>>
>>>>
>>>> I checked some fancy fps but not the fractional ones; now I see they
>>>> aren't recognized
>>>> correctly.
>>>> If I understand correctly the specs the fixed_timeinc is given by the
>>>> smallest
>>>> number of bits (up to 16) to represent   timeinc_resolution-1, 
>>>> correct ?
>>>>   
>>>
>>>
>>>
>>> IIRC, yes
>>>
>>>
>>>  
>>>
>>>> What's wrong? :)
>>>>   
>>>
>>>
>>>
>>> many mpeg4 files have no fixed_timeinc
>>> [...]
>>>  
>>>
>>
>> This one works correctly with all fps I tried, both with lavc and 
>> with xvid, in raw files and in TS.
>> Is it ok ?
>>
>>    Nico
>
>
>
> sorry, this doesn't work with b-frames :(
> next time ...
>

given that 3 consecutive frames are always enough to determine the 
smallest delta_timeinc
(or I'm mistaken?) this patch should work correctly (it does for me, 
even seeking with -sb).
Michael, what's your opinion?

-------------- next part --------------
A non-text attachment was scrubbed...
Name: mpeg4fps.diff
Type: text/x-patch
Size: 7112 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20050107/549f4e9f/attachment.bin>