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

Pekka Paalanen ppaalanen at gmail.com
Tue Jun 7 13:09:40 UTC 2016


On Sat, 21 May 2016 13:24:38 +0800
Sam Spilsbury <smspillaz at gmail.com> wrote:

> 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>

Pushed:
   36aeaf0..fe31d2f  master -> master


Thanks,
pq
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 811 bytes
Desc: OpenPGP digital signature
URL: <https://lists.freedesktop.org/archives/wayland-devel/attachments/20160607/39884c98/attachment.sig>


More information about the wayland-devel mailing list