[FFmpeg-devel] [PATCH] mov: Reduce the verbosity of the warning about fragmented MP4 vs advanced edit lists

Derek Buitenhuis derek.buitenhuis at gmail.com
Mon Jan 23 15:59:21 EET 2023


On 1/17/2023 9:31 AM, Martin Storsjö wrote:
> Only warn if the advanced_editlist option is enabled (it is enabled
> by default though) so we don't print one warning for each track, and
> demote the warning to AV_LOG_LEVEL_VERBOSE; this message does get
> generated whenever parsing a fragmented MP4 file, regardless of
> whether the file actually uses multiple edits or not.
> 
> Later when parsing the mov structures, the demuxer does warn if
> the file did contain multiple edits which would require the
> advanced_editlist option enabled for decoding correctly.
> 
> Adjust the warning message for the case when the file seemed like it
> actually would have needed handling of advanced edit lists, to
> reflect the fact that it doesn't help to try set the option as
> it has been automatically disabled.
> ---
>  libavformat/isom.h |  1 +
>  libavformat/mov.c  | 20 ++++++++++++++------
>  2 files changed, 15 insertions(+), 6 deletions(-)

LGTM.

- Derek


More information about the ffmpeg-devel mailing list