Whole Tomato Software Support Forum
Whole Tomato Software Support Forum
Home | Profile | Register | Active Topics | Members | Search | FAQ
Username:
Password:
Save Password
Forgot your Password?

 All Forums
 Visual Assist X
 General Release
 _T("%SYSTEMROOT%") expanded during "move to impl"
 New Topic  Reply to Topic
 Printer Friendly
Author Previous Topic Topic Next Topic  

peterchen
Ketchup Master

88 Posts

Posted - Jan 07 2010 :  10:07:16 AM  Show Profile  Visit peterchen's Homepage  Reply with Quote
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".

Edited by - peterchen on Jan 07 2010 11:12:55 AM

accord
Moderator

Hungary
1215 Posts

Posted - Jan 07 2010 :  1:47:33 PM  Show Profile  Reply with Quote
I am seeing the same effect here. Thank you for the clear description:

case=37970
Go to Top of Page

support
Moderator

5029 Posts

Posted - Jan 31 2010 :  1:38:28 PM  Show Profile  Visit support's Homepage  Reply with Quote
case=37970 is fixed in build 1810

Whole Tomato Software, Inc.
Go to Top of Page
  Previous Topic Topic Next Topic  
 New Topic  Reply to Topic
 Printer Friendly
Jump To:
Whole Tomato Software Support Forum © 2010 Whole Tomato Software, Inc Go To Top Of Page
Snitz Forums 2000