Whole Tomato Software Forums
Whole Tomato Software Forums
Main Site | Profile | Register | Active Topics | Members | Search | FAQ
 All Forums
 Visual Assist
 Technical Support
 _T("%SYSTEMROOT%") expanded during "move to impl"

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
peterchen Posted - Jan 07 2010 : 10:07:16 AM
foo.h:
class Foo
{
   void Bar()
   {
      CPath p(_T("%SYSTEMROOT%\\\\some.file"));
      p.ExpandEnvStrings();
   }
}


Select Bar(), run "Move to implementation".

----- Observed:

void Foo::Bar()
{
   CPath p(_T("C:\\Windows\\\\some.file"));
   p.ExpandEnvStrings();
}


----- Expected:
My precious platform independent string constants remain platform independent (and precious).


VS 2008, VAX 1738 (also, VAX 1734), XP

Severity: this was part of a larger method, and I caught this only because the compiler was warning about '\\W' being an invalid escape sequence. It would likely slip through all tests. From this POV, I'd rate it "dangerous".
2   L A T E S T    R E P L I E S    (Newest First)
support Posted - Jan 31 2010 : 1:38:28 PM
case=37970 is fixed in build 1810
accord Posted - Jan 07 2010 : 1:47:33 PM
I am seeing the same effect here. Thank you for the clear description:

case=37970

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