graesslin requested changes to this revision.
graesslin added inline comments.
This revision now requires changes to proceed.
Restricted Application edited projects, added KWin; removed Plasma.
> #include "kwinglplatform.h"
> +#include "kwinglutils_funcs.h"
> #include <epoxy/gl.h>
why include kwinglutils_funcs.h? If it's truly needed, please add a comment so
that it doesn't get removed with cleanup
> #include <epoxy/egl.h>
> +// qopengl.h declares GLdouble as a typedef of float when Qt is built
> +// with GLES support. This conflicts with the epoxy/gl_generated.h
> +// declaration, so we have to prevent the Qt header from being #included.
> +#define QOPENGL_H
Instead of copy pasting this all over the place: let's add one header file
which does the epoxy include and the ifdef mess.
To: bshah, #plasma, graesslin
Cc: plasma-devel, kwin, #kwin, iodelay, bwowk, ZrenBot, lesliezhai,
ali-mohamed, hardening, jensreuterberg, abetts, sebas, apol, mart