On Sat, May 21, 2016 at 12:23 PM, Yong Bakos <[email protected]> wrote:
> From: Yong Bakos <[email protected]>
>
> 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 <[email protected]>
> ---
>  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
> [email protected]
> 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 <[email protected]>
_______________________________________________
wayland-devel mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/wayland-devel

Reply via email to