[Bug 682110] qtdemux: discont flag set on multiple buffers in push mode

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=682110
  GStreamer | gst-plugins-good | 0.11.x

Tim-Philipp Müller <t.i.m> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
         Resolution|                            |FIXED
   Target Milestone|1.1.x                       |1.0.8
            Summary|[isomp4/qtdemux] Discont    |qtdemux: discont flag set
                   |flag set on multiple        |on multiple buffers in push
                   |buffers in push mode        |mode

--- Comment #6 from Tim-Philipp Müller <t.i.m at zen.co.uk> 2013-05-09 07:56:42 UTC ---
I suspect this fixes it, if not please re-open:

 commit 725faab590353610caf372e2bbe51d27d0a0275f
 Author: Thiago Santos <thiago.sousa.santos at collabora.com>
 Date:   Wed Apr 17 16:54:22 2013 -0300

    qtdemux: do not push discont buffers if they aren't discont

    qtdemux takes its buffers from a GstAdapter. Those buffers are created
    from the larger buffer that it obtained from upstream and they carry
    the same flags, including DISCONT if it is set. In these cases, all
    buffers that qtdemux is going to push would be marked as DISCONT.

    This scenario can make parsers/decoders flush on every buffer leading
    to no decoding at all hapenning. This patch prevents this by unsetting
    the flag if it shouldn't be set.

-- 
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
You are the assignee for the bug.