Ian Lance Taylor writes:
> I'll preapprove a similar patch to libcpp/system.h if you want to take a
> look at that.
Sure. The following patch has been bootstrapped successfully on
i386-pc-solaris2.11, installed on mainline.
Rainer
2011-07-22 Rainer Orth
* system.h [__cplu
Rainer Orth writes:
> 2011-07-21 Rainer Orth
>
> gcc:
> * system.h [__cplusplus]: Wrap C function declarations in extern "C".
>
> include:
> * xregex.h (regoff_t): Define.
This is OK.
I'll preapprove a similar patch to libcpp/system.h if you want to take a
look at tha
Ian,
> I agree with Joseph that this is wrong. We must never wrap the #include
> of a system header file with extern "C". That will simply break on some
> systems. You should only wrap extern "C" around the various HAVE_DECL
> declarations.
ok, the following patch does this. Bootstrap on alph
Rainer Orth writes:
> diff --git a/gcc/system.h b/gcc/system.h
> --- a/gcc/system.h
> +++ b/gcc/system.h
> @@ -24,6 +24,10 @@ along with GCC; see the file COPYING3.
> #ifndef GCC_SYSTEM_H
> #define GCC_SYSTEM_H
>
> +#ifdef __cplusplus
> +extern "C" {
> +#endif
> +
> /* We must include stda
On Thu, 21 Jul 2011, Rainer Orth wrote:
> diff --git a/gcc/system.h b/gcc/system.h
> --- a/gcc/system.h
> +++ b/gcc/system.h
> @@ -24,6 +24,10 @@ along with GCC; see the file COPYING3.
> #ifndef GCC_SYSTEM_H
> #define GCC_SYSTEM_H
>
> +#ifdef __cplusplus
> +extern "C" {
> +#endif
> +
I don'
