After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 768899 - vaapidecodebin: simplify the code
vaapidecodebin: simplify the code
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gstreamer-vaapi
unspecified
Other All
: Normal normal
: 1.9.2
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on: 724352 734093
Blocks:
 
 
Reported: 2016-07-17 11:06 UTC by Víctor Manuel Jáquez Leal
Modified: 2016-07-27 13:47 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
vaapidecodebin: simplify the code (9.31 KB, patch)
2016-07-17 11:06 UTC, Víctor Manuel Jáquez Leal 		committed Details | Review

Description Víctor Manuel Jáquez Leal 2016-07-17 11:06:48 UTC 
After codec-split and only the registration of the available plugin's features, 
it is possible to simplify vaapidecobin, by instantiating a vaapidecode as a 
normal gobject, since it is not registered anymore.

@tim @slomo, what do you thing about this approach?
Comment 1 Víctor Manuel Jáquez Leal 2016-07-17 11:06:58 UTC 
Created attachment 331643 [details] [review]
vaapidecodebin: simplify the code

Since the elements dependant of the VA video processor are now only registered
if it is available, vaapidecodebin code can be simplified a lot, removing all
the code required to check if the VA video processor was available.
Comment 2 Víctor Manuel Jáquez Leal 2016-07-27 13:28:03 UTC 
Attachment 331643 [details] pushed as ca0c3fd - vaapidecodebin: simplify the code