Whole Tomato Software Forums
Whole Tomato Software Forums
Main Site | Profile | Register | Active Topics | Members | Search | FAQ
 All Forums
 Visual Assist
 Technical Support
 Move impl to source on inlined constructor

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
andre.horst Posted - Nov 05 2008 : 05:17:47 AM
Latest VAX, VS2008 SP1, C++, WinXP SP2

"Moving implementation to source" on a inlined constructor does not remove the "inline" keyword.

write down in your header:


class foo
{
public:
  inline foo()
  {
     // some code
  }
}

Now move cursor to constructor "foo" and "Move implementation to source", that results in


class foo
{
public:
  inline foo();
}

3   L A T E S T    R E P L I E S    (Newest First)
accord Posted - Nov 12 2008 : 3:32:17 PM
Good work!
I am seeing the same effect here. Thank you for the clear description!

case=20884

The problem was caused by the initializer list.
andre.horst Posted - Nov 10 2008 : 03:15:31 AM


I tried the example too and it worked as expected. Than i tried following that was more "real"

class CTestInl
{
public:
	inline CTestInl(void)
	: m_i(0)
	{
		int i = 0 ;
	}
	~CTestInl(void);

private:
	int m_i ;
};

Now "Move implementation" of constructor to source file and get wondered what happend
accord Posted - Nov 08 2008 : 09:13:39 AM
Interesting: I pasted your example into a header, but everything working as expected (inline keyword removed both from header and cpp file) using VS2008 SP1 and VAX 1649.

Can you try to create a new win32 test project and then create a new header which have a cpp-pair and paste your example into the header file? I am curious if your problem will appear here or not. Your project may have something that confuse our parser.

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