[Bug 682629] decodebin: won't expose unknown pads even with expose-all-streams=true

[GStreamer (bugzilla.gnome.org)]

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

Reynaldo H. Verdejo Pinochet <reynaldo> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |NEEDINFO
                 CC|                            |reynaldo at opendot.cl

--- Comment #6 from Reynaldo H. Verdejo Pinochet <reynaldo at opendot.cl> 2013-11-04 16:22:40 UTC ---
Seems we are still coming short for what expose-all-streams seems
to mean. The current code has at least two seemingly analogous
do-not-expose conditions:

unknown_type: Type unknown, caps could not be fixated to a compatible
set (missing decoder ...)
discarded_type: Type known, no final caps due to failed nego (decoder
present, but stream variant not supported)

if any of these conditions are hit, the pad/stream is not exposed.
And this is where I think we have a problem. expose-all sounds like both
should be exposed no matter what, whereas the current patch only fixes(*)
the behavior for the unknown_type case.

As I see it there are likely 2 ways forward. Either we complete/fix the
patch as it is and change the expose-all-streams name to something more
intuitive like expose-unknown-streams, or we make sure the fix also
handles the discarded_type condition.

What do you think?

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