[PATCH wayland] wayland-shm: Include wayland-util.h

Sam Spilsbury smspillaz at gmail.com
Sat May 21 05:24:38 UTC 2016


On Sat, May 21, 2016 at 12:18 PM, Yong Bakos <junk at humanoriented.com> wrote:
> From: Yong Bakos <ybakos at humanoriented.com>
>
> wayland-shm.c uses WL_EXPORT and wl_array, which are defined in
> wayland-util.h.
>
> Include wayland-util.h explicitly, rather than transitively through
> wayland-server.h.
>
> Signed-off-by: Yong Bakos <ybakos at humanoriented.com>
> ---
>  src/wayland-shm.c | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/src/wayland-shm.c b/src/wayland-shm.c
> index 5efbd70..20bb8c8 100644
> --- a/src/wayland-shm.c
> +++ b/src/wayland-shm.c
> @@ -40,6 +40,7 @@
>  #include <signal.h>
>  #include <pthread.h>
>
> +#include "wayland-util.h"
>  #include "wayland-private.h"
>  #include "wayland-server.h"
>
> --
> 2.7.2
>
> _______________________________________________
> wayland-devel mailing list
> wayland-devel at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/wayland-devel

Technically makes sense to me, always better to have a flat include
tree than one where the dependencies are hidden.

Reviewed-by: Sam Spilsbury <smspillaz at gmail.com>


More information about the wayland-devel mailing list