Hi Emil, > On Oct 12, 2017, at 5:39 AM, Emil Velikov <[email protected]> wrote: > > From: Emil Velikov <[email protected]> > > This makes the header self-contained, since the struct is considered > opaque from waylad-cursor POV. > > As we're here move the wl_shm fwd. declaration alongside the others. > Making it easier to read and track. > > Signed-off-by: Emil Velikov <[email protected]>
This is the only other lingering type dependency in wayland-cursor.c coming from wayland-client.h et al. So, similar to the wl_shm forward declaration, this new declaration indeed makes the header self-contained. Reviewed-by: Yong Bakos <[email protected]> Cheers, yong > --- > cursor/wayland-cursor.h | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/cursor/wayland-cursor.h b/cursor/wayland-cursor.h > index 09a13a7..40d3fc5 100644 > --- a/cursor/wayland-cursor.h > +++ b/cursor/wayland-cursor.h > @@ -33,6 +33,8 @@ extern "C" { > #endif > > struct wl_cursor_theme; > +struct wl_buffer; > +struct wl_shm; > > struct wl_cursor_image { > uint32_t width; /* actual width */ > @@ -48,8 +50,6 @@ struct wl_cursor { > char *name; > }; > > -struct wl_shm; > - > struct wl_cursor_theme * > wl_cursor_theme_load(const char *name, int size, struct wl_shm *shm); > > -- > 2.14.1 > > _______________________________________________ > wayland-devel mailing list > [email protected] > https://lists.freedesktop.org/mailman/listinfo/wayland-devel _______________________________________________ wayland-devel mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/wayland-devel
