[Bug 760408] #750013 (streamsynchronizer patches) broke some use cases in GES

[GStreamer (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=760408

Sebastian Dröge (slomo) <slomo at coaxion.net> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|---                         |FIXED
   Target Milestone|git master                  |1.7.2

--- Comment #2 from Sebastian Dröge (slomo) <slomo at coaxion.net> ---
commit 2717f4a86f3afa8a1cc5d4399b11e4cbab631944
Author: Mathieu Duponchelle <mathieu.duponchelle at opencreed.com>
Date:   Sat Jan 9 04:35:23 2016 +0100

    streamsynchronizer: Ignore flushing streams [..]

    [..] when resetting group start time. In GES, we are usually connected
    to the streamsynchronizer on one audio and one video pad.

    When seeking the timeline, both nlecompositions often output their
flush_start
    before any of them has output its flush_stop.

    The current code, when receiving the first flush stop was using the
    running time of the start of the second composition, which could
    be pretty much anything, and means nothing at that point.

    This patch is thread-safe, as STREAM_SYNCHRONIZER_LOCK is taken
    both when setting flushing and when checking it.

    https://bugzilla.gnome.org/show_bug.cgi?id=750013

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.