[PATCH wayland] event-loop: Include wayland-util.h

Sam Spilsbury smspillaz at gmail.com
Sat May 21 05:23:44 UTC 2016


On Sat, May 21, 2016 at 12:23 PM, Yong Bakos <junk at humanoriented.com> wrote:
> From: Yong Bakos <ybakos at humanoriented.com>
>
> event-loop.c uses WL_EXPORT and wl_list, which are defined in
> wayland-util.h.
>
> Include wayland-util.h explicitly, rather than transitively through
> wayland-server-core.h.
>
> Signed-off-by: Yong Bakos <ybakos at humanoriented.com>
> ---
>  src/event-loop.c | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/src/event-loop.c b/src/event-loop.c
> index 11a9bf2..7f1c7e5 100644
> --- a/src/event-loop.c
> +++ b/src/event-loop.c
> @@ -36,6 +36,7 @@
>  #include <sys/signalfd.h>
>  #include <sys/timerfd.h>
>  #include <unistd.h>
> +#include "wayland-util.h"
>  #include "wayland-private.h"
>  #include "wayland-server-core.h"
>  #include "wayland-os.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