Re: [PATCH libcacard 3/3] libcacard: add G_BEGIN/END_DECLS

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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

Cheers, Eduardo.

-- 
Eduardo de Barros Lima (Etrunko)
Software Engineer - RedHat
etrunko@xxxxxxxxxx
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/spice-devel




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]