[gstreamer-bugs] [Bug 532807] [rmdemux] Doesn't properly aggregate flow returns

[GStreamer (bugzilla.gnome.org)]

If you have any questions why you received this email, please see the text at
the end of this email. Replies to this email are NOT read, please see the text
at the end of this email. You can add comments to this bug at:
  http://bugzilla.gnome.org/show_bug.cgi?id=532807

  GStreamer | gst-plugins-ugly | Ver: HEAD CVS

Sebastian Dröge changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |slomo at circular-chaos.org
 Attachment #110783|none                        |needs-work
               Flag|                            |




------- Comment #2 from Sebastian Dröge  2008-05-13 07:20 UTC -------
Looks at least more correct than before to me but shouldn't the flow return be
checked after each pushed buffer to handle errors more instant instead of just
pushing more and more buffers and just taking the last flow return?

One case that comes to my mind where the current behaviour is, when all except
one pad are linked and the last buffer that is pushed is pushed to the unlinked
one. In that case we will error out with GST_FLOW_NOT_LINKED instead of just
continuing.

So, IMHO this patch needs some more work


-- 
See http://bugzilla.gnome.org/page.cgi?id=email.html for more info about why you received
this email, why you can't respond via email, how to stop receiving
emails (or reduce the number you receive), and how to contact someone
if you are having problems with the system.

You can add comments to this bug at http://bugzilla.gnome.org/show_bug.cgi?id=532807.