[Mesa-dev] [PATCH 1/3] egl: use util/macros.h

[Emil Velikov]

On 2 November 2016 at 18:07, Marek Olšák <maraeo at gmail.com> wrote:
> From: Marek Olšák <marek.olsak at amd.com>
>
> I need the definition of PUBLIC.
>
> Cc: 12.0 13.0 <mesa-stable at lists.freedesktop.org>
> ---
>  src/egl/drivers/dri2/egl_dri2.h | 2 --
>  src/egl/main/egldefines.h       | 5 ++---
>  2 files changed, 2 insertions(+), 5 deletions(-)
>
> diff --git a/src/egl/drivers/dri2/egl_dri2.h b/src/egl/drivers/dri2/egl_dri2.h
> index 85d4b85..0020a5b 100644
> --- a/src/egl/drivers/dri2/egl_dri2.h
> +++ b/src/egl/drivers/dri2/egl_dri2.h
> @@ -73,22 +73,20 @@
>  #include "eglconfig.h"
>  #include "eglcontext.h"
>  #include "egldisplay.h"
>  #include "egldriver.h"
>  #include "eglcurrent.h"
>  #include "egllog.h"
>  #include "eglsurface.h"
>  #include "eglimage.h"
>  #include "eglsync.h"
>
> -#define ARRAY_SIZE(a) (sizeof(a) / sizeof((a)[0]))
> -
>  struct wl_buffer;
>
>  struct dri2_egl_driver
>  {
>     _EGLDriver base;
>
>     void *handle;
>     _EGLProc (*get_proc_address)(const char *procname);
>     void (*glFlush)(void);
>  };
> diff --git a/src/egl/main/egldefines.h b/src/egl/main/egldefines.h
> index 13a7563..e8157da 100644
> --- a/src/egl/main/egldefines.h
> +++ b/src/egl/main/egldefines.h
> @@ -31,28 +31,27 @@
>   */
>
>
>  #ifndef EGLDEFINES_INCLUDED
>  #define EGLDEFINES_INCLUDED
>
>  #ifdef __cplusplus
>  extern "C" {
>  #endif
>
> +#include "util/macros.h"
> +
Include must be before the extern C.

With that the series is
Reviewed-by: Emil Velikov <emil.velikov at collabora.com>

Thanks
Emil