----- Original Message ----- > On Thu, Nov 19, 2015 at 02:18:25PM -0200, Eduardo Lima (Etrunko) wrote: > > On 11/19/2015 02:00 PM, Marc-André Lureau wrote: > > > > > > > > > ----- Original Message ----- > > >> On 11/19/2015 10:30 AM, Marc-André Lureau wrote: > > >>> If the compiler in use is a C++ compiler, adds extern "C" around the > > >>> header. > > >>> > > >>> Signed-off-by: Marc-André Lureau <marcandre.lureau@xxxxxxxxx> > > >>> --- > > >>> src/libcacard.h | 6 ++++++ > > >>> 1 file changed, 6 insertions(+) > > >>> > > >>> diff --git a/src/libcacard.h b/src/libcacard.h > > >>> index 0770d13..2b2082e 100644 > > >>> --- a/src/libcacard.h > > >>> +++ b/src/libcacard.h > > >>> @@ -5,6 +5,10 @@ > > >>> #ifndef LIBCACARD_H > > >>> #define LIBCACARD_H > > >>> > > >>> +#include <glib.h> > > >>> + > > >> > > >> I guess this glib.h include shouldn't be here. > > > > > > Why? (need more coffee?) > > > > Well, it might as well be a convention on the project, but in my opinion > > it should only include files from the project itself. For instance, you > > don't have a glib.h include in gtk.h > > gtk.h does not include glib.h because the G_BEGIN_DECLS/G_END_DECLS > macros are not used in gtk.h, but in each individual file included from > gtk.h (ie gtkwidget.h and friends). > Here a different approach is taken, and G_BEGIN_DECLS/G_END_DECLS are > only present in the main include file, which in turn includes the > various headers. > Given that libcacard.h uses macros from glib headers, it makes sense to > me that it directly includes <glib.h> > An alternative would be to move the G_BEGIN_DECLS/G_END_DECLS to each > individual files, in which case the #include <glib.h> could be moved to > each includes as well. It's fine by me. Imho, it's enough to have in top-level header because the other files shouldn't be used directly anymore. > Christophe > > _______________________________________________ > Spice-devel mailing list > Spice-devel@xxxxxxxxxxxxxxxxxxxxx > http://lists.freedesktop.org/mailman/listinfo/spice-devel > _______________________________________________ Spice-devel mailing list Spice-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/spice-devel