Re: [PATCH wayland] cursor: add forward declaration for struct wl_buffer

2017-12-05 Thread Daniel Stone
Hi Emil, On 12 October 2017 at 13:39, Emil Velikov wrote: > 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. Thanks for this! Review

Re: [PATCH wayland] cursor: add forward declaration for struct wl_buffer

2017-10-20 Thread Emil Velikov
On 12 October 2017 at 18:19, Yong Bakos wrote: > Hi Emil, > >> On Oct 12, 2017, at 5:39 AM, Emil Velikov wrote: >> >> From: Emil Velikov >> >> 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. declarati

Re: [PATCH wayland] cursor: add forward declaration for struct wl_buffer

2017-10-12 Thread Yong Bakos
Hi Emil, > On Oct 12, 2017, at 5:39 AM, Emil Velikov wrote: > > From: Emil Velikov > > 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

[PATCH wayland] cursor: add forward declaration for struct wl_buffer

2017-10-12 Thread Emil Velikov
From: Emil Velikov 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 --- cursor/wayland-cursor.h | 4 ++-- 1 f