GNOME Bugzilla – Bug 786119
va.h already includes VA sub APIs
Last modified: 2017-08-16 10:10:09 UTC
These patches cleans up the include of VA sub APIs, because since VA-API 0.37 they are included in va.h, since there is no need to include them again.
Created attachment 357345 [details] [review] build: check for va_vpp.h Thus, in config.h the macro HAVE_VA_VA_VPP_H is defined. This will allow us to handle the inclusion of the header better.
Created attachment 357346 [details] [review] build: consolidate the VA sub API includes Include all VA sub APIs headers in a single point (gstvaapicompat.h), since they are all already included in va.h after VA-API 0.38.
Created attachment 357347 [details] [review] libs: encoder: remove va.h include Since it is already managed by gstvaapicompat.h
Created attachment 357348 [details] [review] libs: utils: remove va.h include in header And include gstvaapicompat.h in the C files, since the VA-API is not exposed in the headers.
Created attachment 357349 [details] [review] libs: utils: move gstvaapisurface.h to private headers Since the utils don't expose API defined in gstvaapisource.h, it is moved to their private headers where they are used.
Created attachment 357350 [details] [review] libs: remove unused header Since libgstvaapi is not distributed, there is no need to check for private header inclusion. Thus removing it.
Attachment 357345 [details] pushed as 1789c60 - build: check for va_vpp.h Attachment 357346 [details] pushed as 9891f1a - build: consolidate the VA sub API includes Attachment 357347 [details] pushed as aaca75f - libs: encoder: remove va.h include Attachment 357348 [details] pushed as 3f9ad1f - libs: utils: remove va.h include in header Attachment 357349 [details] pushed as 3b8b2ff - libs: utils: move gstvaapisurface.h to private headers Attachment 357350 [details] pushed as a07ff96 - libs: remove unused header