[Bug 704950] videomixer: add colorspace conversion

[GStreamer (bugzilla.gnome.org)]

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

--- Comment #22 from Sebastian Dröge (slomo) <slomo at circular-chaos.org> 2013-07-29 09:58:37 UTC ---
(In reply to comment #20)
> (In reply to comment #17)
> > (From update of attachment 250230 [details] [details])
> > Looks good in general, but this *only* ever does conversion if we run into one
> > of these race conditions, right? E.g. If one pad is configured with RGBA
> > already, but another one tries to configure with I420 and checked what
> > videomixer can do before RGBA was configured?
> > 
> > Would it make sense to just always allow conversion now inside videomixer but
> > keep the configured format at the top of the caps that are returned by the CAPS
> > query?
> 
> I'm not sure I understand this comment, the conversion will not only take place
> in the race condition scenario, but as well when a first pad is configured with
> a format, buffers flow for a moment then after eg 1 second a new pad is added
> with a different format there will be conversion, and possibly renegotiation of
> the output format.

Well, that's exactly the race condition case. Because you can currently only
get a different format on the second pad if the race condition happens. Because
after one pad is configured right now the CAPS query will only return caps with
the format of that first 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.