Whole Tomato Software Forums
Whole Tomato Software Forums
Main Site | Profile | Register | Active Topics | Members | Search | FAQ
 All Forums
 Visual Assist
 Technical Support
 Can't move function implementation to header

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
legalize Posted - Jul 30 2014 : 11:50:46 PM
leap.h:
#if !defined(LEAP_H)
#define LEAP_H

namespace leap
{

extern bool is_leap_year(int year);

}

#endif


leap.cpp:
#include "leap.h"

namespace leap
{

extern bool is_leap_year(int year)
{
    return (year % 100) ? (year % 4 == 0) : (year % 400 == 0);
}

}


VAX won't let me move the implementation from the .cpp file to the .h file.
4   L A T E S T    R E P L I E S    (Newest First)
sean Posted - Jul 18 2016 : 10:53:31 PM
case=10239 is implemented in build 2107.
feline Posted - Jul 31 2014 : 9:28:09 PM
Which is why we are looking to add this ability, but we have not gotten there yet. One reason for this is that there is a lot of interest in moving the implementation into the cpp file, but much less interest in moving it back into the header file. We have to prioritise our work somehow, and interest level is a consideration.

If you want to place the implementation into the header file, but outside of the class declaration, then you can cut and paste the implementation, and VA Outline can help you here.
legalize Posted - Jul 31 2014 : 4:18:52 PM
So the existing one only moves implementation from header to source?

Yep, looks like it.

Refactorings should be reversible, without relying on editor undo.
feline Posted - Jul 31 2014 : 3:16:04 PM
We don't currently have a refactoring command to move the implementation to the header file, but we are looking to add this ability:

case=10239

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