[Bug 707605] Need "reverse-funnel" element

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=707605
  GStreamer | gstreamer (core) | git

Sebastian Dröge (slomo) <slomo> changed:

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

--- Comment #21 from Sebastian Dröge (slomo) <slomo at circular-chaos.org> 2013-09-16 09:08:49 UTC ---
Review of attachment 254906:
 --> (https://bugzilla.gnome.org/review?bug=707605&attachment=254906)

Just took a short look, but:

::: plugins/elements/gststreamiddemux.c
@@ +331,3 @@
+    case GST_EVENT_STREAM_START:
+    {
+      if (gst_pad_get_stream_id (pad) != NULL) {

gst_event_parse_stream_start (event, &stream_id) will give you the stream-id.
You can then create a pad if necessary, and select it as the active one. From
this point on it will stay the active pad until the next stream-start event
arrives

@@ +341,3 @@
+    {
+      /* Send caps to all src pads */
+      res = gst_pad_event_default (pad, parent, event);

Only to the active pad, probably for all events except flush-start, flush-stop
and EOS

@@ +351,3 @@
+      }
+      /* Send newsegment to all src pads */
+      res = gst_pad_event_default (pad, parent, event);

No, only to the active pad

@@ +396,3 @@
+    }
+    default:
+      res = gst_pad_query_default (pad, parent, query);

Probably more queries should go to the currently active pad

-- 
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.