After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 734443 - qtdemux: forward DISCONT from upstream to the output streams
qtdemux: forward DISCONT from upstream to the output streams
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-good
git master
Other Linux
: Normal normal
: 1.5.1
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2014-08-07 15:52 UTC by George Kiagiadakis
Modified: 2014-08-11 08:28 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
qtdemux: forward DISCONT from upstream to the output streams (1.23 KB, patch)
2014-08-07 15:52 UTC, George Kiagiadakis 		committed Details | Review

Description George Kiagiadakis 2014-08-07 15:52:23 UTC 
Created attachment 282826 [details] [review]
qtdemux: forward DISCONT from upstream to the output streams

In DASH reverse playback, when dashdemux starts downloading a previous dash segment from the one that is currently playing, it marks its start with a DISCONT, which needs to be forwarded downstream so that downstream actually flushes out the already decoded buffers and continues with the new segment.

Currently this doesn't happen, so the stream stucks. This patch attempts to fix that.
Comment 1 Sebastian Dröge (slomo) 2014-08-11 08:28:52 UTC 
commit 9dd48c503c5e6b78dd5834a4821d23da5ff05c87
Author: George Kiagiadakis <george.kiagiadakis@collabora.com>
Date:   Thu Aug 7 17:12:38 2014 +0300

    qtdemux: forward DISCONT from upstream to the output streams
    
    This makes sense in DASH reverse playback, where the upstream dashdemux
    will download DASH segments in reverse order, but push their buffers
    forward to qtdemux and mark each segment start as DISCONT. This needs
    to be forwarded downstream to the parser/decoder, otherwise it won't work.
    
    https://bugzilla.gnome.org/show_bug.cgi?id=734443