[Swig-devel] [ swig-Bugs-3571361 ] C++ style comments in extern C

[SourceForge.net <no...@so...>]

Bugs item #3571361, was opened at 2012-09-24 16:49
Message generated for change (Comment added) made by rdonch
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=101645&aid=3571361&group_id=1645

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: perl
Group: None
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Ben Reser (breser)
Assigned to: Nobody/Anonymous (nobody)
Summary: C++ style comments in extern C

Initial Comment:
Over at the Subversion project we've noticed a failure on one of our build bots that recently was upgraded to SWIG 2.0.7.  We end up with the following error:
core.c: In function 'boot_SVN___Core':
core.c:42819:5: error: expected expression before '/' token
core.c:42819:37: error: too many decimal points in number

Looking at like 42819 we find:
// Casts only needed for Perl < 5.10.

which is in the SWIG_init function which is preceded with:
#ifdef __cplusplus
extern "C"
#endif

The error can easily be fixed by applying the attached patch to Lib/perl5/perlinit.swg which just changes the comment to a C style comment /* */

We're likely going to work around this by just removing the -ansi flag from our build options for the Perl bindings, but we shouldn't have to, SWIG really should be producing valid code.

----------------------------------------------------------------------

Comment By: Roman Donchenko (rdonch)
Date: 2012-09-25 12:25

Message:
extern "C" doesn't really have anything to do with it - the bug is simply
that this line doesn't conform to C90, even though the rest of the file
does.


----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=101645&aid=3571361&group_id=1645