Whole Tomato Software Forums
Whole Tomato Software Forums
Main Site | Profile | Register | Active Topics | Members | Search | FAQ
User name:
Password:
Save Password
Forgot your password?

 All Forums
 Visual Assist
 Technical Support
 Change signature may break the code
 New Topic  Reply to Topic
 Printer Friendly
Author Previous Topic Topic Next Topic  

nodet
New Member

8 Posts

Posted - Oct 11 2007 :  09:51:57 AM  Show Profile  Reply with Quote
Hi all,

Using 10.3.1561.0.
Consider the following piece of code (note that the parameter does not have the same name in the declaration and the definition):

class A {
int f(int i);
};

int A::f(int j) {
return j;
}

Using 'Change signature' on A::f to add make the method 'const' will break the code because the parameter will use the name of the declaration ('i') instead of the one of the definition ('j').

Note that if the method was declared without a parameter name, Visual Assist will also break the code by removing the name of the parameter in the definition.

In the 'Change signature' dialog, I'd rather use the way the method is defined, instead of the way it's declared. If possible, I would also try to avoid changing parts of the declaration that do not have to be changed (here, changing the name is not needed).

Xavier Nodet
"They that can give up essential liberty to obtain a little temporary
safety deserve neither liberty nor safety." - Benjamin Franklin, 1759.

feline
Whole Tomato Software

United Kingdom
17256 Posts

Posted - Oct 11 2007 :  2:40:34 PM  Show Profile  Reply with Quote
Oops. I am seeing the same effect here. Thank you for the clear description.

case=9327

zen is the art of being at one with the two'ness
Go to Top of Page

sean
Whole Tomato Software

USA
2817 Posts

Posted - Nov 18 2013 :  3:17:09 PM  Show Profile  Reply with Quote
Change Signature was overhauled in build 2007.
case=9327 is fixed in build 2007.
Go to Top of Page
  Previous Topic Topic Next Topic  
 New Topic  Reply to Topic
 Printer Friendly
Jump To:
© 2019 Whole Tomato Software, LLC Go To Top Of Page
Snitz Forums 2000