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
 Move Implementation to Source File		  New Topic  Reply to Topic
 Printer Friendly
Author Previous Topic Topic Next Topic  

blq
Junior Member

Sweden
18 Posts
Posted - Nov 08 2006 :  06:54:51 AM  Show Profile  Reply with Quote
Hi,

I've noticed an error in the "Move Implementation to Source File" refactoring for inlined constructors (VAX 1540, both VC++6 and VS2005).

Example case:
---- initial Foo.hpp ----

class Foo
{
public:
Foo() :
m_bar(new char[256])
{}

private:
char* m_bar;
};

---- Foo.cpp, after "Move Implementation to Source File" ----

Foo::Foo() :
m_bar( char[256]) // <= "new" keyword has been dropped!
{

}

------

Minor second point regarding formatting - Why can't the refactoring(s) simply copy the code verbatim?

Regards
// Fredrik

feline
Whole Tomato Software

United Kingdom
19022 Posts
Posted - Nov 08 2006 :  1:07:26 PM  Show Profile  Reply with Quote
I am seeing the same effect here. Thank you for the clear description.

case=3519

The formatting change, this is because Move Implementation to Source File uses the same autotext entry as Create Implementation. This is to allow people to specify the formatting they want, so that it is applied consistently.
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:12:56 PM  Show Profile  Reply with Quote
case=3519 is fixed in build 2007
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