public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug libquadmath/55473] New: quadmath.h should have extern "C" for C++ users
@ 2012-11-26 19:54 jb at gcc dot gnu.org
  2013-03-06  8:53 ` [Bug libquadmath/55473] " skannan at redhat dot com
                   ` (4 more replies)
  0 siblings, 5 replies; 6+ messages in thread
From: jb at gcc dot gnu.org @ 2012-11-26 19:54 UTC (permalink / raw)
  To: gcc-bugs


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55473

             Bug #: 55473
           Summary: quadmath.h should have extern "C" for C++ users
    Classification: Unclassified
           Product: gcc
           Version: unknown
            Status: UNCONFIRMED
          Severity: enhancement
          Priority: P3
         Component: libquadmath
        AssignedTo: unassigned@gcc.gnu.org
        ReportedBy: jb@gcc.gnu.org


quadmath.h is missing something like:


#ifdef __cplusplus
extern "C" {
#endif

/* Content of file */

#ifdef __cplusplus
} /* End extern "C".  */
#endif


As it is, C++ users who wish to use libquadmath have to do this manually before
including quadmath.h. E.g.

http://gcc.gnu.org/ml/gcc-help/2011-06/msg00148.html

or for a more recent example

http://stackoverflow.com/questions/13571621/quadruple-precision-in-g-4-6-3-linux-using-quadmath


^ permalink raw reply	[flat|nested] 6+ messages in thread
* [Bug libquadmath/55473] quadmath.h should have extern "C" for C++ users
  2012-11-26 19:54 [Bug libquadmath/55473] New: quadmath.h should have extern "C" for C++ users jb at gcc dot gnu.org
@ 2013-03-06  8:53 ` skannan at redhat dot com
  2013-03-06 21:43 ` burnus at gcc dot gnu.org
                   ` (3 subsequent siblings)
  4 siblings, 0 replies; 6+ messages in thread
From: skannan at redhat dot com @ 2013-03-06  8:53 UTC (permalink / raw)
  To: gcc-bugs


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55473

--- Comment #1 from Shakthi Kannan <skannan at redhat dot com> 2013-03-06 08:53:19 UTC ---
Created attachment 29593
  --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=29593
Added cplusplus macro usage


^ permalink raw reply	[flat|nested] 6+ messages in thread
* [Bug libquadmath/55473] quadmath.h should have extern "C" for C++ users
  2012-11-26 19:54 [Bug libquadmath/55473] New: quadmath.h should have extern "C" for C++ users jb at gcc dot gnu.org
  2013-03-06  8:53 ` [Bug libquadmath/55473] " skannan at redhat dot com
@ 2013-03-06 21:43 ` burnus at gcc dot gnu.org
  2013-03-07  6:14 ` skannan at redhat dot com
                   ` (2 subsequent siblings)
  4 siblings, 0 replies; 6+ messages in thread
From: burnus at gcc dot gnu.org @ 2013-03-06 21:43 UTC (permalink / raw)
  To: gcc-bugs


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55473

Tobias Burnus <burnus at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |burnus at gcc dot gnu.org

--- Comment #2 from Tobias Burnus <burnus at gcc dot gnu.org> 2013-03-06 21:43:15 UTC ---
Janne and Shakth: The proposal is fine with me. However, it would help if you
would simply submit a/the patch. For what it is worth, I think the patch is
fine together with a ChangeLog.


^ permalink raw reply	[flat|nested] 6+ messages in thread
* [Bug libquadmath/55473] quadmath.h should have extern "C" for C++ users
  2012-11-26 19:54 [Bug libquadmath/55473] New: quadmath.h should have extern "C" for C++ users jb at gcc dot gnu.org
  2013-03-06  8:53 ` [Bug libquadmath/55473] " skannan at redhat dot com
  2013-03-06 21:43 ` burnus at gcc dot gnu.org
@ 2013-03-07  6:14 ` skannan at redhat dot com
  2013-03-07  7:53 ` jakub at gcc dot gnu.org
  2013-03-07  7:56 ` jakub at gcc dot gnu.org
  4 siblings, 0 replies; 6+ messages in thread
From: skannan at redhat dot com @ 2013-03-07  6:14 UTC (permalink / raw)
  To: gcc-bugs


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55473

Shakthi Kannan <skannan at redhat dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |skannan at redhat dot com

--- Comment #3 from Shakthi Kannan <skannan at redhat dot com> 2013-03-07 06:13:45 UTC ---
@Janne:

I had submitted the patch and ChangeLog at:

  http://gcc.gnu.org/ml/gcc-patches/2013-03/msg00223.html


^ permalink raw reply	[flat|nested] 6+ messages in thread
* [Bug libquadmath/55473] quadmath.h should have extern "C" for C++ users
  2012-11-26 19:54 [Bug libquadmath/55473] New: quadmath.h should have extern "C" for C++ users jb at gcc dot gnu.org
                   ` (2 preceding siblings ...)
  2013-03-07  6:14 ` skannan at redhat dot com
@ 2013-03-07  7:53 ` jakub at gcc dot gnu.org
  2013-03-07  7:56 ` jakub at gcc dot gnu.org
  4 siblings, 0 replies; 6+ messages in thread
From: jakub at gcc dot gnu.org @ 2013-03-07  7:53 UTC (permalink / raw)
  To: gcc-bugs


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55473

--- Comment #4 from Jakub Jelinek <jakub at gcc dot gnu.org> 2013-03-07 07:53:17 UTC ---
Author: jakub
Date: Thu Mar  7 07:53:10 2013
New Revision: 196517

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=196517
Log:
    PR libquadmath/55473
    * quadmath.h: Add 'extern "C"' block for C++ use.

Modified:
    trunk/libquadmath/ChangeLog
    trunk/libquadmath/quadmath.h


^ permalink raw reply	[flat|nested] 6+ messages in thread
* [Bug libquadmath/55473] quadmath.h should have extern "C" for C++ users
  2012-11-26 19:54 [Bug libquadmath/55473] New: quadmath.h should have extern "C" for C++ users jb at gcc dot gnu.org
                   ` (3 preceding siblings ...)
  2013-03-07  7:53 ` jakub at gcc dot gnu.org
@ 2013-03-07  7:56 ` jakub at gcc dot gnu.org
  4 siblings, 0 replies; 6+ messages in thread
From: jakub at gcc dot gnu.org @ 2013-03-07  7:56 UTC (permalink / raw)
  To: gcc-bugs


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55473

Jakub Jelinek <jakub at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
                 CC|                            |jakub at gcc dot gnu.org
         Resolution|                            |FIXED

--- Comment #5 from Jakub Jelinek <jakub at gcc dot gnu.org> 2013-03-07 07:56:19 UTC ---
Fixed.


^ permalink raw reply	[flat|nested] 6+ messages in thread
end of thread, other threads:[~2013-03-07  7:56 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-11-26 19:54 [Bug libquadmath/55473] New: quadmath.h should have extern "C" for C++ users jb at gcc dot gnu.org
2013-03-06  8:53 ` [Bug libquadmath/55473] " skannan at redhat dot com
2013-03-06 21:43 ` burnus at gcc dot gnu.org
2013-03-07  6:14 ` skannan at redhat dot com
2013-03-07  7:53 ` jakub at gcc dot gnu.org
2013-03-07  7:56 ` jakub at gcc dot gnu.org

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).