[Swig-devel] [ swig-Bugs-1836331 ] Extra scoping on destructor

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

Bugs item #1836331, was opened at 2007-11-22 09:28
Message generated for change (Comment added) made by olly
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=101645&aid=1836331&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: code generation (general)
Group: None
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: William Fulton (wsfulton)
Assigned to: David M. Beazley (beazley)
Summary: Extra scoping on destructor

Initial Comment:

%inline %{
class MCConvergenceList
{
public:
    MCConvergenceList::~MCConvergenceList();
};
%}

example.i:8: Warning(504): Function ~MCConvergenceList must have a return type.

Constructors and methods in SWIG allow this extra unnecessary scoping and the same fix needs applying to destructors.

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

>Comment By: Olly Betts (olly)
Date: 2007-12-02 03:19

Message:
Logged In: YES 
user_id=14972
Originator: NO

I don't particularly care if this is accepted or rejected - I just
remembered that this had been reported before, so dug up the previous
report in case it was relevant.

If this is currently accepted for ctors and normal methods, but not
destructors, it would be better to be consistent about it.


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

Comment By: William Fulton (wsfulton)
Date: 2007-11-22 14:14

Message:
Logged In: YES 
user_id=242951
Originator: YES

I know it is not ISO C++, but this extra scoping is a common programming
error which unfortunately many compilers accept. I also think I know I can
fix it so unless anyone strongly objects to me fixing it, I will probably
implement a fix soon.

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

Comment By: Olly Betts (olly)
Date: 2007-11-22 09:57

Message:
Logged In: YES 
user_id=14972
Originator: NO

FWIW, Dave rejected "fixing" this when previously reported:

http://sourceforge.net/tracker/index.php?func=detail&aid=1660778&group_id=1645&atid=101645

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

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