On Wed, 29 Apr 2015 13:18:22 +0300 Pekka Paalanen <ppaala...@gmail.com> wrote:
> On Tue, 28 Apr 2015 22:57:18 +0300 > Giulio Camuffo <giuliocamu...@gmail.com> wrote: > > > The new core header doesn't include any other header, since it really > > is not needed. > > --- > > Makefile.am | 1 + > > src/wayland-egl-core.h | 56 > > ++++++++++++++++++++++++++++++++++++++++++++++++++ > > src/wayland-egl.h | 29 +------------------------- > > 3 files changed, 58 insertions(+), 28 deletions(-) > > create mode 100644 src/wayland-egl-core.h > > -#ifdef __cplusplus > > -} > > -#endif > > +#include "wayland-egl-core.h" > > Hmm, that looks funny: > > #include <wayland-client.h> > #include "wayland-egl-core.h" > > I think the wayland-client.h should be in "" to match the style of > every other include in the wayland headers, but fixing that should be a > separate patch. Reading cpp.info, "" is correct. > > Reviewed-by: Pekka Paalanen <pekka.paala...@collabora.co.uk> With Jason' R-b added, pushed. d74a9c0..ea73cb0 master -> master Thanks, pq _______________________________________________ wayland-devel mailing list wayland-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/wayland-devel
