[SSSD] [PATCHES] Use pattern #elif defined(identifier)

Pavel Reichl preichl at redhat.com
Fri Mar 14 12:52:55 UTC 2014


On Fri, 2014-03-14 at 12:58 +0100, Lukas Slebodnik wrote:
> On (12/03/14 17:52), Pavel Reichl wrote:
> >On Mon, 2014-03-10 at 09:20 +0100, Lukas Slebodnik wrote:
> >> ehlo,
> >> 
> >> We had in source code following pattern #elif HAVE_<name>
> >> It worked because undefined identifier(in some cases) was evaluated to 0.
> >> But we do not care about value of HAVE_SOMETHING. We just need to know
> >> whether identifier was defined.
> >> 
> >> There is not equivalent to #ifdef (short for of #if definded)
> >> We need to use long form: #elif defined HAVE_<name>
> >> 
> >> It causes also compiler warning with enabled compiler flag -Wundef.
> >> 
> >> LS
> >> _______________________________________________
> >> sssd-devel mailing list
> >> sssd-devel at lists.fedorahosted.org
> >> https://lists.fedorahosted.org/mailman/listinfo/sssd-devel
> >
> >Hello Lukas,
> >
> >Patch 1:
> >While testing I applied 3rd patch first and it (as expected) raised some
> >errors which were fixed by 2nd patch.
> >I can't see any problems which are addressed by this patch. There aren't
> >any #if directives or any other usage of (#define) constants from
> >config.h.  Could you possibly describe in more detail the benefit of
> >this patch, please?
> In header file sss_utf8.h there are lines:
> 
> #ifdef HAVE_LIBUNISTRING
> #include <unistr.h>
> #include <unicase.h>
> #elif defined(HAVE_GLIB2)
> #include <glib.h>
> #endif
> 
> But header file config.h was not included before header file sss_utf8.h in some
> places. So neither unistring header files nor glib header files were included.
> 
> I realised these header files are necessary only in implementation module
> sss_utf8.c. I move part with ifdef into sss_utf8.c
> 
> New patch is attached.
> 
> >
> >Patch 2:
> >ACK
> >
> >Patch 3:
> >ACK - I think it's really useful - like it:-).
> >
> >Thanks.
> >
> >Pavel Reichl
> >
> 
> 
> _______________________________________________
> sssd-devel mailing list
> sssd-devel at lists.fedorahosted.org
> https://lists.fedorahosted.org/mailman/listinfo/sssd-devel

Thank you Lukas, 

new patches apply and I can't see any warnings.

ACK to all patches.

Pavel Reichl







More information about the sssd-devel mailing list