[Bug 687182] playbin: autoplugging s/w and h/w accelerated deinterlacers

[GStreamer (bugzilla.gnome.org)]

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

--- Comment #28 from Sebastian Dröge <slomo at circular-chaos.org> 2013-07-09 07:27:53 UTC ---
(In reply to comment #27)

> Another thing that we should consider is the playback starting delay of
> playbin. Gwenole might have some profiling information regarding this since he
> was complaining about this :) So it  would be good to avoid string comparisons.

Some profiler results for this would be useful indeed, yes :)


(In reply to comment #27)
> I guess there is no gstreamer design objection to add more than one
> classification for a single element.Something like
> "Filter/Effect/Video/Deinterlace, Filter/Converter/Video/Scaler,
> Filter/Effect/Video/Colorbalance" :)
> 
> Basically we only need to enable autoplug for deinterlacer in the current
> playbin at the moment. If I understood correctly all other code to enable the
> autoplugging of elements like videoconvert, videobalance etc will be untouched
> in this case. So just doing some elementfactory classification comparison +
> capsfeatures comparison could be okay, If we are not going for some generalized
> vpp_element autoplugging feature.
> But IMHO, it would be good to have vpp_interface if we need to enable
> autoplugging for each element like videobalance, videoconvert etc. 
> 
> @Gwenole: What do you think about vpp filter enable/disable option? I guess it
> is would be good to have such a feature since vappipostproc is doing many thing
> in one pass as you said before.

Ok, so maybe an interface like the one you proposed plus something to
enable/disable specific functions. And if additional configuration of a
postproc function is required a different interface is used (GstColorBalance
for example).

Instead of flags you could also use strings btw and make them GQuarks
internally. Which would be just a pointer comparison then. See the GstStructure
API for example.

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