[Bug 698712] playbin: autoplug video decoder and sink based on caps features

[GStreamer (bugzilla.gnome.org)]

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

--- Comment #40 from Sebastian Dröge <slomo at circular-chaos.org> 2013-04-26 16:23:34 UTC ---
(In reply to comment #39)
> Created an attachment (id=242571)
 View: https://bugzilla.gnome.org/attachment.cgi?id=242571
 Review: https://bugzilla.gnome.org/review?bug=698712&attachment=242571

> playbin: autoplug the audio/video decoders and sinks based on capsfeatures.
> 
> This one maintains the global list of dec-sink combination irrespective of the
> caps..
> 
> Do we need to lock with playbin->elements_lock with in the autoplug_select
> callback ?

Yes, otherwise looks good now.


One way to keep this decoder-sink combination list much smaller would be to
only include combinations here that have more than one caps feature in common
(i.e. not only handle system memory but also something else special). For all
other decoder-sink combinations we already know that they have exactly on caps
feature in common (system memory), and thus don't need to store that
information.

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