[Bug 734220] decodebin/uridecodebin/playbin support multiple srcpads in a parser

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=734220
  GStreamer | gst-plugins-base | git

Thiago Sousa Santos <thiagossantos> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |thiagossantos at gmail.com

--- Comment #3 from Thiago Sousa Santos <thiagossantos at gmail.com> 2014-08-29 21:07:41 UTC ---
(In reply to comment #2)
> (From update of attachment 282640 [details])
> This is not correct. Not every video parser can have multiple srcpads, and you
> still need to handle the parsers as parsers elsewhere in the decodebin code.
> This patch would break conversion of stream formats for example.
> 
> If you look elsewhere in decodebin you will see some parser specific code that
> still must be called for this.
>

I started implementing a dynamic-parser handling that is similar to the
demuxer, except that the no-more-pads signaling is ignored. It seems to work
and is quite simple.

> 
> Maybe there should be a separate demuxer element for extracting the ancilliary
> data instead of putting it into the parser? Might be an easier solution

Seems more elegant, but:
1) Will likely replicate parsing code in 2 elements
2) Autoplugging and avoiding loops will add extra complexity to decodebin

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