From 725da8579bae51f256812666651555f3f0f1be50 Mon Sep 17 00:00:00 2001 From: Edward Hervey Date: Sun, 28 Jun 2009 17:04:00 +0200 Subject: [PATCH] asfdemux: Use the average frame duration for buffers without a duration. This will still cause some timestamp jitter, but giving a hint as to the duration rather than nothing seems to be a better idea. Also, this allows some scenarios (like remuxing with asfmux) to estimate the total duration using the accumulated packet duration (which will be correct). --- gst/asfdemux/gstasfdemux.c | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/gst/asfdemux/gstasfdemux.c b/gst/asfdemux/gstasfdemux.c index 3debb46102..8300b0a234 100644 --- a/gst/asfdemux/gstasfdemux.c +++ b/gst/asfdemux/gstasfdemux.c @@ -1314,7 +1314,11 @@ gst_asf_demux_push_complete_payloads (GstASFDemux * demux, gboolean force) * position reporting if a live src is playing not so live content * (e.g. rtspsrc taking some time to fall back to tcp) */ GST_BUFFER_TIMESTAMP (payload->buf) = payload->ts + demux->in_gap; - GST_BUFFER_DURATION (payload->buf) = payload->duration; + if (payload->duration == GST_CLOCK_TIME_NONE) + GST_BUFFER_DURATION (payload->buf) = + stream->ext_props.avg_time_per_frame * 100; + else + GST_BUFFER_DURATION (payload->buf) = payload->duration; /* FIXME: we should really set durations on buffers if we can */