[Bug 784779] gl: public gstgl headers should not include GL headers

GStreamer (GNOME Bugzilla) bugzilla at gnome.org
Tue Aug 22 08:08:07 UTC 2017


https://bugzilla.gnome.org/show_bug.cgi?id=784779

Matthew Waters (ystreet00) <ystreet00 at gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #357732|none                        |reviewed
             status|                            |

--- Comment #16 from Matthew Waters (ystreet00) <ystreet00 at gmail.com> ---
Review of attachment 357732:
 --> (https://bugzilla.gnome.org/review?bug=784779&attachment=357732)

Looks ok to me.  Feel free to push after changing the comment.

Are you going to look into the other platform-specific headers?

::: tests/check/libs/gstglheaders.c
@@ +26,3 @@
+
+/* This test check that public gstgl headers does not include any
+ * GL headers. Except for gst/gl/gstglfuncs.h and gst/gl/gl.h */

Comment is not quite correct anymore.

gst/gl/gl.h is included

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