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
 Feature Requests
 Change Signature should handle const-changes
 New Topic  Reply to Topic
 Printer Friendly
Author Previous Topic Topic Next Topic  

StefanEgo
Ketchup Master

56 Posts

Posted - Nov 19 2012 :  06:09:12 AM  Show Profile  Reply with Quote
Steps to reproduce:
1. use copy-paste the following code into a new project file:

class Foo
{
	virtual void bar() {}
};

class Bar : public Foo
{
	void bar() {}
};

2. Right-click on Foo::bar() and choose "Refactor -> Change Signature..."
3. In the box change "virtual void bar()" to "virtual void bar() const" and click on OK.

Actual result:
Window closes and the bar()-method in Foo is changed.

Expected result:
Either of the following would be acceptable/expected:
a) Window closes and bar()-methods in Foo AND Bar are changed.
b) Window closes and a Warning-popup is displayed "Change in constness of method in new signature. You should now update references if they need to accomodate the change."
VA Find References then shows Bar::bar() as an entry to investigate.

feline
Whole Tomato Software

United Kingdom
18750 Posts

Posted - Nov 20 2012 :  2:38:45 PM  Show Profile  Reply with Quote
We are looking into making Change Signature aware of derived classes, so that all of the functions will be updated:

case=3373

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:29:38 PM  Show Profile  Reply with Quote
Change Signature was overhauled in build 2007 and now supports changes to a class hierarchy.
Go to Top of Page
  Previous Topic Topic Next Topic  
 New Topic  Reply to Topic
 Printer Friendly
Jump To:
© 2023 Whole Tomato Software, LLC Go To Top Of Page
Snitz Forums 2000