libs: check that private headers remain private.

Make sure that libgstvaapi private headers remain internally used to
build libgstvaapi libraries only. All header dependencies were reviewed
and checks for IN_LIBGSTVAAPI definition were added accordingly.

Also rename GST_VAAPI_CORE definition to IN_LIBGSTVAAPI_CORE to keep
consistency.

diff --git a/gst-libs/gst/vaapi/Makefile.am b/gst-libs/gst/vaapi/Makefile.am
index 390d91e..11efb4b 100644 (file)
--- a/gst-libs/gst/vaapi/Makefile.am
+++ b/gst-libs/gst/vaapi/Makefile.am
@@ -20,7 +20,8 @@ libgstvaapi_includedir = \
        $(includedir)/gstreamer-$(GST_API_VERSION)/gst/vaapi
 
 libgstvaapi_cflags =                           \
-       -DGST_VAAPI_CORE                        \
+       -DIN_LIBGSTVAAPI                        \
+       -DIN_LIBGSTVAAPI_CORE                   \
        -DGST_USE_UNSTABLE_API                  \
        -I$(top_srcdir)/gst-libs                \
        -I$(top_builddir)/gst-libs              \
@@ -130,6 +131,7 @@ libgstvaapi_source_priv_h =                 \
        gstvaapivideopool_priv.h                \
        gstvaapiwindow_priv.h                   \
        gstvaapiworkarounds.h                   \
+       libgstvaapi_priv_check.h                \
        sysdeps.h                               \
        $(NULL)
 
@@ -291,6 +293,7 @@ libgstvaapi_drm_@GST_API_VERSION@includedir =       \
        $(libgstvaapi_includedir)
 
 libgstvaapi_drm_@GST_API_VERSION@_la_CFLAGS =  \
+       -DIN_LIBGSTVAAPI                        \
        -DGST_USE_UNSTABLE_API                  \
        -I$(top_srcdir)/gst-libs                \
        $(GLIB_CFLAGS)                          \
@@ -327,6 +330,7 @@ libgstvaapi_x11_@GST_API_VERSION@includedir =       \
        $(libgstvaapi_includedir)
 
 libgstvaapi_x11_@GST_API_VERSION@_la_CFLAGS =  \
+       -DIN_LIBGSTVAAPI                        \
        -DGST_USE_UNSTABLE_API                  \
        -I$(top_srcdir)/gst-libs                \
        $(GLIB_CFLAGS)                          \
@@ -365,6 +369,7 @@ libgstvaapi_glx_@GST_API_VERSION@includedir =       \
        $(libgstvaapi_includedir)
 
 libgstvaapi_glx_@GST_API_VERSION@_la_CFLAGS =  \
+       -DIN_LIBGSTVAAPI                        \
        -DGST_USE_UNSTABLE_API                  \
        -I$(top_srcdir)/gst-libs                \
        $(GLIB_CFLAGS)                          \
@@ -402,6 +407,7 @@ libgstvaapi_wayland_@GST_API_VERSION@includedir = \
        $(libgstvaapi_includedir)
 
 libgstvaapi_wayland_@GST_API_VERSION@_la_CFLAGS = \
+       -DIN_LIBGSTVAAPI                        \
        -DGST_USE_UNSTABLE_API                  \
        -I$(top_srcdir)/gst-libs                \
        $(GLIB_CFLAGS)                          \

diff --git a/gst-libs/gst/vaapi/gstvaapidecoder_unit.h b/gst-libs/gst/vaapi/gstvaapidecoder_unit.h
index 7ee77a0..32fa9bb 100644 (file)
--- a/gst-libs/gst/vaapi/gstvaapidecoder_unit.h
+++ b/gst-libs/gst/vaapi/gstvaapidecoder_unit.h
@@ -23,6 +23,8 @@
 #ifndef GST_VAAPI_DECODER_UNIT_H
 #define GST_VAAPI_DECODER_UNIT_H
 
+#include "libgstvaapi_priv_check.h"
+
 G_BEGIN_DECLS
 
 typedef struct _GstVaapiDecoderUnit             GstVaapiDecoderUnit;

diff --git a/gst-libs/gst/vaapi/gstvaapidisplay_priv.h b/gst-libs/gst/vaapi/gstvaapidisplay_priv.h
index 1d13964..92d466a 100644 (file)
--- a/gst-libs/gst/vaapi/gstvaapidisplay_priv.h
+++ b/gst-libs/gst/vaapi/gstvaapidisplay_priv.h
@@ -227,7 +227,7 @@ gst_vaapi_display_get_display_types (GstVaapiDisplay * display)
 }
 
 /* Inline reference counting for core libgstvaapi library */
-#ifdef GST_VAAPI_CORE
+#ifdef IN_LIBGSTVAAPI_CORE
 #define gst_vaapi_display_ref_internal(display) \
     ((gpointer)gst_vaapi_mini_object_ref(GST_VAAPI_MINI_OBJECT(display)))
 

diff --git a/gst-libs/gst/vaapi/gstvaapidisplaycache.h b/gst-libs/gst/vaapi/gstvaapidisplaycache.h
index bbd2436..62f7b90 100644 (file)
--- a/gst-libs/gst/vaapi/gstvaapidisplaycache.h
+++ b/gst-libs/gst/vaapi/gstvaapidisplaycache.h
@@ -23,6 +23,7 @@
 #ifndef GSTVAAPIDISPLAYCACHE_H
 #define GSTVAAPIDISPLAYCACHE_H
 
+#include "libgstvaapi_priv_check.h"
 #include <gst/vaapi/gstvaapidisplay.h>
 
 typedef struct _GstVaapiDisplayCache            GstVaapiDisplayCache;

diff --git a/gst-libs/gst/vaapi/gstvaapiminiobject.h b/gst-libs/gst/vaapi/gstvaapiminiobject.h
index 5f5ada9..058378d 100644 (file)
--- a/gst-libs/gst/vaapi/gstvaapiminiobject.h
+++ b/gst-libs/gst/vaapi/gstvaapiminiobject.h
@@ -24,6 +24,7 @@
 #define GST_VAAPI_MINI_OBJECT_H
 
 #include <glib.h>
+#include "libgstvaapi_priv_check.h"
 
 G_BEGIN_DECLS
 

diff --git a/gst-libs/gst/vaapi/gstvaapiobject_priv.h b/gst-libs/gst/vaapi/gstvaapiobject_priv.h
index 4c91149..f6f3a58 100644 (file)
--- a/gst-libs/gst/vaapi/gstvaapiobject_priv.h
+++ b/gst-libs/gst/vaapi/gstvaapiobject_priv.h
@@ -219,7 +219,7 @@ gst_vaapi_object_new(const GstVaapiObjectClass *klass,
     GstVaapiDisplay *display);
 
 /* Inline reference counting for core libgstvaapi library */
-#ifdef GST_VAAPI_CORE
+#ifdef IN_LIBGSTVAAPI_CORE
 static inline gpointer
 gst_vaapi_object_ref_internal(gpointer object)
 {

diff --git a/gst-libs/gst/vaapi/gstvaapipixmap_priv.h b/gst-libs/gst/vaapi/gstvaapipixmap_priv.h
index 577fda8..9219bf6 100644 (file)
--- a/gst-libs/gst/vaapi/gstvaapipixmap_priv.h
+++ b/gst-libs/gst/vaapi/gstvaapipixmap_priv.h
@@ -109,7 +109,7 @@ gst_vaapi_pixmap_new_from_native(const GstVaapiPixmapClass *pixmap_class,
     GstVaapiDisplay *display, gpointer native_pixmap);
 
 /* Inline reference counting for core libgstvaapi library */
-#ifdef GST_VAAPI_CORE
+#ifdef IN_LIBGSTVAAPI_CORE
 #define gst_vaapi_pixmap_ref_internal(pixmap) \
     ((gpointer)gst_vaapi_mini_object_ref(GST_VAAPI_MINI_OBJECT(pixmap)))
 

diff --git a/gst-libs/gst/vaapi/gstvaapiutils.h b/gst-libs/gst/vaapi/gstvaapiutils.h
index 21d13c2..9fcab47 100644 (file)
--- a/gst-libs/gst/vaapi/gstvaapiutils.h
+++ b/gst-libs/gst/vaapi/gstvaapiutils.h
@@ -26,6 +26,7 @@
 #define GST_VAAPI_UTILS_H
 
 #include "config.h"
+#include "libgstvaapi_priv_check.h"
 #include <glib.h>
 #include <va/va.h>
 

diff --git a/gst-libs/gst/vaapi/gstvaapiutils_glx.h b/gst-libs/gst/vaapi/gstvaapiutils_glx.h
index 5d4f75a..3df92ae 100644 (file)
--- a/gst-libs/gst/vaapi/gstvaapiutils_glx.h
+++ b/gst-libs/gst/vaapi/gstvaapiutils_glx.h
@@ -26,6 +26,7 @@
 #define GST_VAAPI_UTILS_GLX_H
 
 #include "config.h"
+#include "libgstvaapi_priv_check.h"
 #include <GL/gl.h>
 #include <GL/glext.h>
 #include <GL/glx.h>

diff --git a/gst-libs/gst/vaapi/gstvaapiutils_h264_priv.h b/gst-libs/gst/vaapi/gstvaapiutils_h264_priv.h
index 5bfbf87..85eb184 100644 (file)
--- a/gst-libs/gst/vaapi/gstvaapiutils_h264_priv.h
+++ b/gst-libs/gst/vaapi/gstvaapiutils_h264_priv.h
@@ -24,6 +24,7 @@
 #define GST_VAAPI_UTILS_H264_PRIV_H
 
 #include "gstvaapiutils_h264.h"
+#include "libgstvaapi_priv_check.h"
 
 G_BEGIN_DECLS
 

diff --git a/gst-libs/gst/vaapi/gstvaapiutils_mpeg2_priv.h b/gst-libs/gst/vaapi/gstvaapiutils_mpeg2_priv.h
index 179fed3..3f96eca 100644 (file)
--- a/gst-libs/gst/vaapi/gstvaapiutils_mpeg2_priv.h
+++ b/gst-libs/gst/vaapi/gstvaapiutils_mpeg2_priv.h
@@ -24,6 +24,7 @@
 #define GST_VAAPI_UTILS_MPEG2_PRIV_H
 
 #include "gstvaapiutils_mpeg2.h"
+#include "libgstvaapi_priv_check.h"
 
 G_BEGIN_DECLS
 

diff --git a/gst-libs/gst/vaapi/gstvaapiutils_x11.h b/gst-libs/gst/vaapi/gstvaapiutils_x11.h
index 1229b34..f6faf25 100644 (file)
--- a/gst-libs/gst/vaapi/gstvaapiutils_x11.h
+++ b/gst-libs/gst/vaapi/gstvaapiutils_x11.h
@@ -26,6 +26,7 @@
 #define GST_VAAPI_UTILS_X11_H
 
 #include "config.h"
+#include "libgstvaapi_priv_check.h"
 #include <X11/Xlib.h>
 #include <glib.h>
 

diff --git a/gst-libs/gst/vaapi/gstvaapiwindow_priv.h b/gst-libs/gst/vaapi/gstvaapiwindow_priv.h
index f0dc9a5..3abf421 100644 (file)
--- a/gst-libs/gst/vaapi/gstvaapiwindow_priv.h
+++ b/gst-libs/gst/vaapi/gstvaapiwindow_priv.h
@@ -108,7 +108,7 @@ gst_vaapi_window_new_from_native(const GstVaapiWindowClass *window_class,
     GstVaapiDisplay *display, gpointer native_window);
 
 /* Inline reference counting for core libgstvaapi library */
-#ifdef GST_VAAPI_CORE
+#ifdef IN_LIBGSTVAAPI_CORE
 #define gst_vaapi_window_ref_internal(window) \
     ((gpointer)gst_vaapi_mini_object_ref(GST_VAAPI_MINI_OBJECT(window)))