mp4 robust muxing: improve documentation and logging
authorMathieu Duponchelle <mathieu@centricular.com>
Thu, 16 Aug 2018 16:55:29 +0000 (18:55 +0200)
committerMathieu Duponchelle <mathieu@centricular.com>
Thu, 16 Aug 2018 17:43:50 +0000 (19:43 +0200)
gst/isomp4/gstqtmux.c
gst/multifile/gstsplitmuxsink.c

index 4733bb6..cf8148d 100644 (file)
@@ -2968,6 +2968,10 @@ gst_qt_mux_start_file (GstQTMux * qtmux)
             (NULL));
         return GST_FLOW_ERROR;
       }
+      if (qtmux->reserved_moov_update_period == GST_CLOCK_TIME_NONE) {
+        GST_WARNING_OBJECT (qtmux,
+            "Robust muxing requires reserved-moov-update-period to be set");
+      }
       break;
     case GST_QT_MUX_MODE_FAST_START:
     case GST_QT_MUX_MODE_FRAGMENTED_STREAMABLE:
index 260e8e3..0568c65 100644 (file)
@@ -338,7 +338,8 @@ gst_splitmux_sink_class_init (GstSplitMuxSinkClass * klass)
           "reserved-duration-remaining properties and use them if so. "
           "(Only present on qtmux and mp4mux for now). splitmuxsink may then also "
           " create new fragments if the reserved header space is about to overflow. "
-          "Note this does not set reserved-moov-update-period - apps should do that manually",
+          "Note that for mp4mux and qtmux, reserved-moov-update-period must be set "
+          "manually by the app to a non-zero value for robust muxing to have an effect.",
           DEFAULT_USE_ROBUST_MUXING,
           G_PARAM_READWRITE | G_PARAM_STATIC_STRINGS));