[FFmpeg-devel] [PATCH] lavf/mov: Always prefer tfdt over sidx

[Gyan Doshi]

On 2021-11-03 03:16 pm, Thilo Borgmann wrote:
> Hi,
>
> this effectively reverts 071930de724166bfb90fc6d368c748771188fd94 and fixes the underlying issue by always preferring TFDT.
>
> Furthermore, the current solution fails if the -use_tfdt flag is given but no TFDT box is found in the stream.

Thanks for the heads-up.

The original impetus for my patch was a client's sample file which had 
rubbish sidx values but sane tfdt ones, likely due to a buggy muxer.
The motivation for an option was to give the user control.

I think hardcoding a reversed preference can lead to the same impasse. 
How about change the default value of use_tfdt and check for tfdt 
presence when enabled?

Regards,
Gyan