On Sat, 21 May 2016 13:24:38 +0800 Sam Spilsbury <[email protected]> wrote:
> On Sat, May 21, 2016 at 12:18 PM, Yong Bakos <[email protected]> wrote: > > From: Yong Bakos <[email protected]> > > > > 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 <[email protected]> > > --- > > 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 > > [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
pgp3t4sijmeUO.pgp
Description: OpenPGP digital signature
_______________________________________________ wayland-devel mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/wayland-devel
