[FFmpeg-cvslog] dashenc: Don't output frameRates at both AS and Representation-level. Only output maxFrameRate at AS-level.

Anton Schubert git at videolan.org
Sun Oct 29 19:30:39 EET 2017


ffmpeg | branch: master | Anton Schubert <ischluff at mailbox.org> | Sun Oct 29 16:10:35 2017 +0100| [fe334712ddc633dba3774beb56579381a00d59bf] | committer: Michael Niedermayer

dashenc: Don't output frameRates at both AS and Representation-level. Only output maxFrameRate at AS-level.

Fix conformance regarding section "3.2.4. Presence of Attributes and
Element" of the "Guidelines for Implementation: DASH-IF Interoperability
Points V4.1" (http://dashif.org/guidelines/)

Signed-off-by: Anton Schubert <ischluff at mailbox.org>
Signed-off-by: Peter Große <pegro at friiks.de>
Signed-off-by: Michael Niedermayer <michael at niedermayer.cc>

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

 libavformat/dashenc.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/libavformat/dashenc.c b/libavformat/dashenc.c
index c55106c11d..82381b67a9 100644
--- a/libavformat/dashenc.c
+++ b/libavformat/dashenc.c
@@ -393,8 +393,8 @@ static int write_adaptation_set(AVFormatContext *s, AVIOContext *out, int as_ind
 
     avio_printf(out, "\t\t<AdaptationSet id=\"%s\" contentType=\"%s\" segmentAlignment=\"true\" bitstreamSwitching=\"true\"",
                 as->id, as->media_type == AVMEDIA_TYPE_VIDEO ? "video" : "audio");
-    if (as->media_type == AVMEDIA_TYPE_VIDEO && as->max_frame_rate.num && !as->ambiguous_frame_rate)
-        avio_printf(out, " %s=\"%d/%d\"", (av_cmp_q(as->min_frame_rate, as->max_frame_rate) < 0) ? "maxFrameRate" : "frameRate", as->max_frame_rate.num, as->max_frame_rate.den);
+    if (as->media_type == AVMEDIA_TYPE_VIDEO && as->max_frame_rate.num && !as->ambiguous_frame_rate && av_cmp_q(as->min_frame_rate, as->max_frame_rate) < 0)
+        avio_printf(out, " maxFrameRate=\"%d/%d\"", as->max_frame_rate.num, as->max_frame_rate.den);
     lang = av_dict_get(as->metadata, "language", NULL, 0);
     if (lang)
         avio_printf(out, " lang=\"%s\"", lang->value);



More information about the ffmpeg-cvslog mailing list