[Bug 738416] decodebin: Fix autoplugging of parser elements

[GStreamer (bugzilla.gnome.org)]

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

--- Comment #7 from sreerenj <bsreerenj at gmail.com> 2014-10-20 11:35:10 UTC ---
The only problem is the computational overhead, I agree. But that is an enough
justification for pushing the patch which is avoiding this overhead IMHO.
In any circumstances decodebin shouldn't autoplug a second parser element of
same type. We cannot ask anyone/customer to something like "you are not allowed
to install a second plugin which is doing same thing" which seems to be weird
for me. It is not only about gstreamer-vaapi. The gstreamer-vaapi is just a use
case. Another example, i will install a different parser of same type for
codecanalyzer where the patches are not going to integrate to upstrem anytime
soon. I could install my own plugin which is doing other parsings too. In any
case we shouldn't allow decodebin to autoplug a second plugin. With my patch,
the only overhead is the initial checking for some substring, what we gain is
the computational gain during the full life time of playback.

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