[FFmpeg-devel] [PATCH] avcodec/setts_bsf: set the output packet duration to 0

James Almer jamrial at gmail.com
Sun Feb 13 19:28:07 EET 2022



On 2/13/2022 2:22 PM, Andreas Rheinhardt wrote:
> James Almer:
>> On 2/13/2022 2:14 PM, Paul B Mahol wrote:
>>> Too soon.
>>>
>>> Wait more,
>>
>> Ok.
>>
>>> does this breaks something?
>>
>> There are no fate tests, and 0 is used when the duration is unknown, so
>> it shouldn't break anything.
>>
> 
> It breaks the simple use-case where this filter is used to just shift
> the timestamps.

How so? It's a documented value that means unknown. What breaks because 
of it? Wouldn't it be revealing a bug if so?

And in non simple use cases that completely replace or rescale the 
timestamps, the old duration value is no longer valid, and something 
definitely worse to have than 0.

> 
> - Andreas
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
> 
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request at ffmpeg.org with subject "unsubscribe".


More information about the ffmpeg-devel mailing list