tsdemux: Always issue a DTS even when it's equal to PTS
authorVivia Nikolaidou <vivia@ahiru.eu>
Fri, 11 Oct 2019 14:25:04 +0000 (17:25 +0300)
committerJan Schmidt <thaytan@noraisin.net>
Thu, 14 Nov 2019 12:16:04 +0000 (12:16 +0000)
Currently tsdemux timestamps only the PTS, and only issues the DTS if
it's different. In that case, parsers tend to estimate the next DTS
based on the previous DTS and the duration, which can accumulate
rounding errors.

gst/mpegtsdemux/tsdemux.c

index 53f92e3..c50f811 100644 (file)
@@ -3103,7 +3103,8 @@ gst_ts_demux_push_pending_data (GstTSDemux * demux, TSDemuxStream * stream,
     buffer = gst_buffer_list_get (buffer_list, 0);
 
   if (GST_CLOCK_TIME_IS_VALID (stream->pts))
-    GST_BUFFER_PTS (buffer) = stream->pts;
+    GST_BUFFER_PTS (buffer) = GST_BUFFER_DTS (buffer) = stream->pts;
+  /* DTS = PTS by default, we override it if there's a real DTS */
   if (GST_CLOCK_TIME_IS_VALID (stream->dts))
     GST_BUFFER_DTS (buffer) = stream->dts;