[Bug 668471] [h264parse] Negotiation regression for autoplugging

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=668471
  GStreamer | gst-plugins-bad | 0.11.x

--- Comment #4 from Sebastian Dröge <slomo at circular-chaos.org> 2012-01-23 11:09:23 UTC ---
(In reply to comment #3)
> So far it has not been specified that a get_caps result should necessarily be
> treated in "first order", and as such not clear/certain it should do conversion
> if not doing so would also work for downstream (as indicated in caps), and so
> relying upon such in responding to get_caps is borderline.

No, that was never defined. Correct :) But that's the behaviour all parsers
used in the past and that what the special handling of parsers in decodebin2
expects parsers to do.

The alternative would be to intercept gst_pad_push() and gst_pad_set_caps()
calls before the parser to catch GST_FLOW_NOT_NEGOTIATED and something to catch
error messages from the parser. Or some new API for element negotiation that
somehow allows explicit negotiation of elements.

> Besides that it seems like the rest of the negotiation in h264parse may need
> some fixing since it not set to handle cases like stream-format = {
> byte-stream, avc }.  And in this case the set {} there makes it not clear what
> is 'first' anyway.

Agreed

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