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. |