wafsamba/CHECK_LIB: replace dots in library names
Andrew Bartlett
abartlet at samba.org
Sun Apr 6 18:18:15 MDT 2014
On Thu, 2014-03-27 at 18:30 +0100, Ralph Böhme wrote:
> Hi
>
> just ran across an issue where in CHECK_LIB a library containing a
> version number including a dot in the library name resulted in a
> macro defintion including the dot, eg
>
> libtracker-sparql-0.16
>
> resulting macro:
>
> HAVE_LIBTRACKER_SPARQL_0.16
>
> Compiler diagnostic:
>
> default/include/config.h:945:33: warning: missing whitespace after
> the macro name [enabled by default]
>
> <http://gcc.gnu.org/onlinedocs/cpp/Tokenization.html#Tokenization>:
>
> "An identifier is the same as an identifier in C: any sequence of
> letters, digits, or underscores, which begins with a letter or
> underscore."
>
> I've crafted a small patch which fixes the issue by substituting the
> dot with an underscore. Review appreciated!
This looks reasonable.
Reviewed-by: Andrew Bartlett <abartlet at samba.org>
Jeremy,
What do you think?
--
Andrew Bartlett
http://samba.org/~abartlet/
Authentication Developer, Samba Team http://samba.org
Samba Developer, Catalyst IT http://catalyst.net.nz/services/samba
More information about the samba-technical
mailing list