[Bug 747449] adaptivedemux: multiple period case does not work well

[GStreamer (GNOME Bugzilla)]

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

Thiago Sousa Santos <thiagossantos at gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #301069|none                        |needs-work
             status|                            |

--- Comment #4 from Thiago Sousa Santos <thiagossantos at gmail.com> ---
Review of attachment 301069:
 --> (https://bugzilla.gnome.org/review?bug=747449&attachment=301069)

This causes another issue that is ignoring downstream returns of GST_FLOW_EOS.
Basically, downstream elements can return GST_FLOW_EOS to make upstream know
that the stream should end. For example, they might be configured with some
GstSegment that has a stop that is earlier than upstream knows. This would
cause this return to be ignored.

Perhaps we need to have 2 status variables here? One for our internal playlist
position and another relative to what downstream returned to us?

Otherwise nice catch, I can reproduce the bug here.

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