Re: [PATCH weston 1.2] compositor.h: Don't include config.h

2013-08-26 Thread Kristian Høgsberg
eoff=intel@lists.freedesktop.org > > [mailto:wayland-devel- > > bounces+ullysses.a.eoff=intel@lists.freedesktop.org] On Behalf Of Pier > > Luigi Fiorini > > Sent: Friday, August 23, 2013 2:43 PM > > To: wayland-devel@lists.freedesktop.org > > Cc: Pier Luigi Fior

RE: [PATCH weston 1.2] compositor.h: Don't include config.h

2013-08-26 Thread Eoff, Ullysses A
Subject: [PATCH weston 1.2] compositor.h: Don't include config.h > > Public headers should not include config.h. > --- > src/compositor.h | 2 -- > 1 file changed, 2 deletions(-) > > diff --git a/src/compositor.h b/src/compositor.h > index 3eaac42..3437aea 100644 >

RE: [PATCH weston 1.2] compositor.h: Don't include config.h

2013-08-26 Thread Eoff, Ullysses A
ehalf Of Pier > Luigi Fiorini > Sent: Friday, August 23, 2013 2:43 PM > To: wayland-devel@lists.freedesktop.org > Cc: Pier Luigi Fiorini > Subject: [PATCH weston 1.2] compositor.h: Don't include config.h > > Public headers should not include config.h. > --- >

[PATCH weston 1.2] compositor.h: Don't include config.h

2013-08-23 Thread Pier Luigi Fiorini
Public headers should not include config.h. --- src/compositor.h | 2 -- 1 file changed, 2 deletions(-) diff --git a/src/compositor.h b/src/compositor.h index 3eaac42..3437aea 100644 --- a/src/compositor.h +++ b/src/compositor.h @@ -28,8 +28,6 @@ extern "C" { #endif -#include "config.h" - #i