[Cogl] [PATCH] Include cogl-texture-private.h from cogl-texture-gl.c

Robert Bragg robert at sixbynine.org
Tue Nov 27 10:43:30 PST 2012


This looks good to me:

Reviewed-by: Robert Bragg <robert at linux.intel.com>

thanks,
- Robert


On Tue, Nov 27, 2012 at 5:47 PM, Neil Roberts <neil at linux.intel.com> wrote:

> cogl-texture-gl.c needs cogl-texture-private.h to get the definition
> of the texture vtable struct. This header gets included via
> cogl-texture-pixmap-x11-private.h which is included from
> cogl-winsys-private.h and cogl-context-private.h. However if Cogl is
> being built without an X11 backend then it won't be included so the
> file should also explicitly include it. This was breaking building
> cogl with only the SDL winsys.
> ---
>  cogl/driver/gl/cogl-texture-gl.c | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/cogl/driver/gl/cogl-texture-gl.c
> b/cogl/driver/gl/cogl-texture-gl.c
> index a9f0d32..8d3aafe 100644
> --- a/cogl/driver/gl/cogl-texture-gl.c
> +++ b/cogl/driver/gl/cogl-texture-gl.c
> @@ -29,6 +29,7 @@
>  #include "cogl-internal.h"
>  #include "cogl-context-private.h"
>  #include "cogl-texture-gl-private.h"
> +#include "cogl-texture-private.h"
>  #include "cogl-util.h"
>
>  static inline int
> --
> 1.7.11.3.g3c3efa5
>
> _______________________________________________
> Cogl mailing list
> Cogl at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/cogl
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/cogl/attachments/20121127/a0e91b81/attachment.html>


More information about the Cogl mailing list