This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH 09/36] floatformat.h: Wrap in extern "C".
- From: Jakub Jelinek <jakub at redhat dot com>
- To: Pedro Alves <palves at redhat dot com>
- Cc: Andrew Pinski <pinskia at gmail dot com>, "gdb-patches at sourceware dot org" <gdb-patches at sourceware dot org>, GCC Patches <gcc-patches at gcc dot gnu dot org>
- Date: Wed, 18 Feb 2015 20:54:51 +0100
- Subject: Re: [PATCH 09/36] floatformat.h: Wrap in extern "C".
- Authentication-results: sourceware.org; auth=none
- References: <1423524046-20605-1-git-send-email-palves at redhat dot com> <1423524046-20605-10-git-send-email-palves at redhat dot com> <CA+=Sn1npOYOeTU6ShGk50Vcy=ZpuYMjUfDac0gXWU20grzyoyw at mail dot gmail dot com> <54D94780 dot 9050606 at redhat dot com> <54DC932D dot 6000904 at redhat dot com>
- Reply-to: Jakub Jelinek <jakub at redhat dot com>
On Thu, Feb 12, 2015 at 11:49:01AM +0000, Pedro Alves wrote:
> On 02/09/2015 11:49 PM, Pedro Alves wrote:
> > On 02/09/2015 11:35 PM, Andrew Pinski wrote:
> >> On Mon, Feb 9, 2015 at 3:20 PM, Pedro Alves <palves@redhat.com> wrote:
> >>> Just like libiberty.h. So that C++ programs, such as GDB when built
> >>> as a C++ program, can use it.
> >>
> >> Why is not needed for GCC building with C++ compiler?
> >
> > Because it doesn't include it.
> >
> > The header of the file claims it is part of GDB, though MAINTAINERS
> > nowadays says that everything under include/ is owned by GCC.
>
> Here's an update that moves the extern "C" below the #include.
>
> OK to push to the GCC repo?
>
> From: Pedro Alves <palves@redhat.com>
> Subject: [PATCH] floatformat.h: Wrap in extern "C".
>
> Just like libiberty.h. So that C++ programs, such as GDB when built
> as a C++ program, can use it.
>
> include/ChangeLog:
> 2015-02-12 Pedro Alves <palves@redhat.com>
>
> * floatformat.h [__cplusplus]: Wrap in extern "C".
Ok, thanks.
Jakub