[FFmpeg-cvslog] avformat/movenc: remove unnecessary HDR metadata log messages

Jan Ekström git at videolan.org
Wed Feb 10 21:51:50 EET 2021


ffmpeg | branch: master | Jan Ekström <jeebjp at gmail.com> | Wed Feb 10 01:15:42 2021 +0200| [752f14ff942576dd8697320fb6ba055593a2de9d] | committer: Jan Ekström

avformat/movenc: remove unnecessary HDR metadata log messages

These made sense before 3ebf449766d4b69a7774c4990fb24fe71c848a3e
when the creation of these boxes was specifically requested by the
user, but now they have become unnecessary as they are just like
many other boxes: If the input has the information required, they
are written, otherwise they are not.

They were moved to verbose verbosity level (which happens to be
the last level still relatively usable), and now appear either once
(normal MP4 muxing), or thrice (with the faststart flag set) in
any normal MP4 usage, without giving much useful information. Thus,
remove them in their current form.

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=752f14ff942576dd8697320fb6ba055593a2de9d
---

 libavformat/movenc.c | 2 --
 1 file changed, 2 deletions(-)

diff --git a/libavformat/movenc.c b/libavformat/movenc.c
index 372c04295d..62192e5c24 100644
--- a/libavformat/movenc.c
+++ b/libavformat/movenc.c
@@ -2025,7 +2025,6 @@ static int mov_write_clli_tag(AVIOContext *pb, MOVTrack *track)
 
     side_data = av_stream_get_side_data(track->st, AV_PKT_DATA_CONTENT_LIGHT_LEVEL, NULL);
     if (!side_data) {
-        av_log(NULL, AV_LOG_VERBOSE, "Not writing 'clli' atom. No content light level info.\n");
         return 0;
     }
     content_light_metadata = (const AVContentLightMetadata*)side_data;
@@ -2052,7 +2051,6 @@ static int mov_write_mdcv_tag(AVIOContext *pb, MOVTrack *track)
     side_data = av_stream_get_side_data(track->st, AV_PKT_DATA_MASTERING_DISPLAY_METADATA, NULL);
     metadata = (const AVMasteringDisplayMetadata*)side_data;
     if (!metadata || !metadata->has_primaries || !metadata->has_luminance) {
-        av_log(NULL, AV_LOG_VERBOSE, "Not writing 'mdcv' atom. Missing mastering metadata.\n");
         return 0;
     }
 



More information about the ffmpeg-cvslog mailing list