Whole Tomato Software Forums
Whole Tomato Software Forums
Main Site | Profile | Register | Active Topics | Members | Search | FAQ
 All Forums
 Visual Assist
 Technical Support
 [BUG] 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
Alexis Pautrot Posted - Feb 20 2012 : 08:54:02 AM
Hi, here is a bug reported for version 10.6.1862.0 built 2011.12.13 :

On a method with a comment on the same line :

class Foo
{
public:

  void Method()   // I like putting comment on the same line
  {
    ...
  }
}


The Move Implementation to Source File will result in moving the method body, but will add ';' to end of the method declaration line, hence in the comment.

class Foo
{
public:

  void Method()   // I like putting comment on the same line;
}



Bye.
6   L A T E S T    R E P L I E S    (Newest First)
feline Posted - Feb 17 2015 : 8:50:20 PM
A fair point, I have put in a bug report for this, but since we are still left with valid code this is not a high priority bug:

case=87958
AndreasS Posted - Feb 17 2015 : 05:58:50 AM
In formal your are right.
I used to write a semi-colon after every method in header file. I do this with or without a implementation.
VA have to insert a semi-colon if there isn't one, but maybe you can omit inserting a semi-colon if you find one at the same line (and at the right position ignoring spaces)
feline Posted - Feb 11 2015 : 1:50:14 PM
I am seeing the same effect, but strictly speaking VA is preserving the existing code when doing the move. You have an extra semi-colon on the function in the header file before the move, and you still have it after the move.

Is there a reason for this extra semi-colon in the header file? I cannot think of a situation where you would need it, but I might just be missing something.
AndreasS Posted - Feb 11 2015 : 04:19:34 AM
A similar problem is, that sometime remains two ';'

class Foo
{
public:

CString GetName() {return m_strName;};

protected:
CString m_strName;
};

The Move Implementation to Source File will result in moving the method body, and will add ';' to end of method declaration line.

class Foo
{
public:

CString GetName();;

protected:
CString m_strName;
};
sean Posted - Nov 07 2014 : 09:55:11 AM
case=30651 is fixed in build 2052
accord Posted - Feb 20 2012 : 12:26:55 PM
I am seeing the same effect here. Thank you for the clear description.

case=30651

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