[Bug 784779] gl: public gstgl headers should not include GL headers
GStreamer (GNOME Bugzilla)
bugzilla at gnome.org
Fri Jul 28 16:20:40 UTC 2017
https://bugzilla.gnome.org/show_bug.cgi?id=784779
--- Comment #10 from Julien Isorce <julien.isorce at gmail.com> ---
(In reply to Matthew Waters (ystreet00) from comment #7)
> Review of attachment 355741 [details] [review]:
>
> Ok there seems to a mismatch on the expected behaviour. gst/gl/gl.h is the
> top-level header for everybody to include and is also the single include
> used for gobject-introspection as #include-ing random gstgl headers has
> gotten us into trouble before (the glmemory includes are specially crafted)
> so changing that behaviour to require this scenario for not using GL
> types/functions probably isn't wise.
>
> I think the best route is that gst/gl/gstglfuncs.h should not be #include-d
> in gst/gl/gl.h
Done in patch i just attached patch "gl: remove include of gstglfuncs.h from
gst/gl/gl.h"
I keep it split so that we can see the diff. But the plan is to squash it.
> The added typedef's are also problematic if the type doesn't match the
> system provided headers and/or GL/gl.h or GLES2/gl2.h aren't in system paths.
typedef void *EGLImageKHR;
typedef void *EGLSurface;
typedef void *EGLContext;
typedef void *EGLDisplay;
Yup I am not sure, note that this is only for egl and are not platform
dependent, i.e. not in eglplatform.h . So this should be always void* .
> +typedef void *EGLImageKHR;
>
> This may cause redefinition type warnings. What's the plan for dealing with
> these?
The best I can do is to guard it with #ifndef / #endif
> The added test looks good.
Cool! \o/
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
More information about the gstreamer-bugs
mailing list