[vlc-devel] [PATCH] avcodec: repeat the pictures marked as repeating

Francois Cartegnie fcvlcdev at free.fr
Mon Nov 16 15:10:17 CET 2020


Le 16/11/2020 à 14:52, Steve Lhomme a écrit :
> On 2020-11-16 14:35, Francois Cartegnie wrote:
>> Le 16/11/2020 à 14:12, Steve Lhomme a écrit :
>>
>>> It should make it easier to remux or reencode on containers with a 
>>> fixed frame
>>> rate that don't handle gaps between pictures. Each picture has the 
>>> same base
>>> duration.
>>>
>>> In addition the interpolate_next_pts() used the repeat_pict 
>>> incorrectly as the
>>> value is 2 per progressive picture. So every additional picture was 
>>> counted as
>>> 2 ticks.
>> No, it was correct.
>> All date_t in video have been fixed to use field rate.
>> 1 field time + 1 repeat == progressive
> 
> Ah yes, ticks_per_frame is 2 for progressive. And the p_sys->pts date is 
> also using this to adjust the frame rate.

If we break the ABI, an enum is still a better alternative

-- 
Francois Cartegnie
VideoLAN - VLC Developer


More information about the vlc-devel mailing list