Define __CORRECT_ISO_CPP_STRING_H_PROTO correctly for Clang.
authorBrooks Moses <bmoses@google.com>
Thu, 12 Dec 2013 01:46:46 +0000 (17:46 -0800)
committerBrooks Moses <bmoses@google.com>
Mon, 23 Dec 2013 23:50:54 +0000 (15:50 -0800)
commit3f637079f549956fe71a363fd82e1b66295deda2
treefb4a09326c7092e81e9dfe937abe18e0e73aa403
parentabc26e998f74750850cc02f9c249ee794cbdd8e8
Define __CORRECT_ISO_CPP_STRING_H_PROTO correctly for Clang.

In the string/string.h and string/strings.h headers, we have a couple
of macros that "tell the caller that we provide correct C++
prototypes" according to the comment; they are used to determine
whether to wrap some prototypes in "extern "C++"" (and provide
multiple overloads of them, and some other magic) when __cplusplus is
defined.

The macros are set to check for sufficiently-recent GCC versions (4.4
and later), but this is not the right check for non-GCC compilers.  In
particular, these macros should also be set when using Clang -- if
they are not set, then Clang will be unable to correctly diagnose a
number of subtle bugs that will be errors in GCC compilations.

As per discussion on earlier versions of this patch, rather than
restrict the fix to Clang per se, we assume that all C++ compilers that
claim to fully support C++98 are using a standard-conforming C++
standard library, which seems pretty reasonable.  Clang has been
providing an appropriate value of __cplusplus since May 2012.
ChangeLog
string/string.h
string/strings.h