On Sat, 21 May 2016 13:23:44 +0800 Sam Spilsbury <[email protected]> wrote:
> 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]> Pushed: 36aeaf0..fe31d2f master -> master Thanks, pq
pgpFUF0S9nOCz.pgp
Description: OpenPGP digital signature
_______________________________________________ wayland-devel mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/wayland-devel
