[Bug 725187] Add new multiappsrc or dynappsrc element with multiple output streams

[GStreamer (GNOME Bugzilla)]

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

Julien Isorce <julien.isorce at gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #335491|none                        |reviewed
             status|                            |

--- Comment #54 from Julien Isorce <julien.isorce at gmail.com> ---
Review of attachment 335491:
 --> (https://bugzilla.gnome.org/review?bug=725187&attachment=335491)

Thx for the updated patch and the diagram. All looks good!
Is someone else interested to review it ?

::: tests/check/elements/multiappsrc.c
@@ +208,3 @@
+          source_id2, gst_buffer_new_and_alloc (4)) == GST_FLOW_OK,
+      "fail to push buffer");
+

Ok I just realized that the API gst_multi_appsrc_push_buffer takes ownership
whereas the action signal does not. Like GstAppSrc.

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