Whole Tomato Software Forums
Whole Tomato Software Forums
Main Site | Profile | Register | Active Topics | Members | Search | FAQ
 All Forums
 Visual Assist
 Technical Support
 Move Implementation to Source File

You must be registered to post a reply.
Click here to register.

Screensize:
UserName:
Password:
Format: BoldItalicizeUnderlineStrikethrough Align leftCenterAlign right Insert horizontal ruleUpload and insert imageInsert hyperlinkInsert email addressInsert codeInsert quoted textInsert listInsert Emoji
   
Message:

Forum code is on.
Html is off.

 
Check to subscribe to this topic.
   

T O P I C    R E V I E W
blq Posted - Nov 08 2006 : 06:54:51 AM
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
2   L A T E S T    R E P L I E S    (Newest First)
sean Posted - Nov 18 2013 : 3:12:56 PM
case=3519 is fixed in build 2007
feline Posted - Nov 08 2006 : 1:07:26 PM
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.

© 2023 Whole Tomato Software, LLC Go To Top Of Page
Snitz Forums 2000