From: SourceForge.net <no...@so...> - 2007-11-22 09:28:21
|
Bugs item #1836331, was opened at 2007-11-22 09:28 Message generated for change (Tracker Item Submitted) made by Item Submitter 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. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=101645&aid=1836331&group_id=1645 |
From: SourceForge.net <no...@so...> - 2007-11-22 09:57:35
|
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-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 |
From: SourceForge.net <no...@so...> - 2007-11-22 14:14:51
|
Bugs item #1836331, was opened at 2007-11-22 09:28 Message generated for change (Comment added) made by wsfulton 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: 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 |
From: SourceForge.net <no...@so...> - 2007-12-02 03:19:23
|
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 |
From: SourceForge.net <no...@so...> - 2007-12-03 22:42:21
|
Bugs item #1836331, was opened at 2007-11-22 09:28 Message generated for change (Comment added) made by wsfulton 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: Closed >Resolution: Wont Fix 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: William Fulton (wsfulton) Date: 2007-12-03 22:42 Message: Logged In: YES user_id=242951 Originator: YES I'm also don't really care if this is fixed or not. It also proved too difficult to fix in the short time I spent on it. Unexpectedly for me, and unfortunately, it requires modification of the parser which is not my strong point. As the syntax is not ISO C++ and is non-trivial for me to fix, I'm closing the bug. Patches will be accepted if anyone else finds a fix. ---------------------------------------------------------------------- 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 |