Whole Tomato Software Support Forum
Whole Tomato Software Support Forum
Home | Profile | Register | Active Topics | Members | Search | FAQ
 All Forums
 Visual Assist X
 General Release
 _T("%SYSTEMROOT%") expanded during "move to impl"

Note: You must be registered in order to post a reply.
To register, click here. Registration is FREE!

Screensize:
UserName:
Password:
Format Mode:
Format: BoldItalicizedUnderlineStrikethrough Align LeftCenteredAlign Right Horizontal Rule Insert HyperlinkInsert EmailInsert Image Insert CodeInsert QuoteInsert List
   
Message:

* HTML is OFF
* Forum Code is ON
Smilies
Smile [:)] Big Smile [:D] Cool [8D] Blush [:I]
Tongue [:P] Evil [):] Wink [;)] Clown [:o)]
Black Eye [B)] Eight Ball [8] Frown [:(] Shy [8)]
Shocked [:0] Angry [:(!] Dead [xx(] Sleepy [|)]
Kisses [:X] Approve [^] Disapprove [V] Question [?]

   Insert an Image

Check here 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

Whole Tomato Software Support Forum © 2010 Whole Tomato Software, Inc Go To Top Of Page
Snitz Forums 2000