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)
committerTim-Philipp Müller <tim@centricular.com>
Thu, 14 Nov 2019 14:04:04 +0000 (14:04 +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 9f696f0..c28516e 100644 (file)
@@ -3040,7 +3040,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;