dashdemux: Implement a default presentation delay
authorEdward Hervey <edward@centricular.com>
Tue, 30 May 2017 13:27:37 +0000 (15:27 +0200)
committerEdward Hervey <bilboed@bilboed.com>
Thu, 13 Jul 2017 06:30:50 +0000 (08:30 +0200)
Have a fallback presentation delay is great (if not present in the
manifest), having an actual default value is better.

https://bugzilla.gnome.org/show_bug.cgi?id=783244

ext/dash/gstdashdemux.c

index 839a0b3..83bcb03 100644 (file)
@@ -205,7 +205,7 @@ enum
 #define DEFAULT_MAX_VIDEO_HEIGHT          0
 #define DEFAULT_MAX_VIDEO_FRAMERATE_N     0
 #define DEFAULT_MAX_VIDEO_FRAMERATE_D     1
-#define DEFAULT_PRESENTATION_DELAY     NULL     /* zero */
+#define DEFAULT_PRESENTATION_DELAY     "10s"    /* 10s */
 
 /* Clock drift compensation for live streams */
 #define SLOW_CLOCK_UPDATE_INTERVAL  (1000000 * 30 * 60) /* 30 minutes */
@@ -530,7 +530,7 @@ gst_dash_demux_init (GstDashDemux * demux)
   demux->max_video_height = DEFAULT_MAX_VIDEO_HEIGHT;
   demux->max_video_framerate_n = DEFAULT_MAX_VIDEO_FRAMERATE_N;
   demux->max_video_framerate_d = DEFAULT_MAX_VIDEO_FRAMERATE_D;
-  demux->default_presentation_delay = DEFAULT_PRESENTATION_DELAY;
+  demux->default_presentation_delay = g_strdup (DEFAULT_PRESENTATION_DELAY);
 
   g_mutex_init (&demux->client_lock);