RESEND: Do not hardcode C: for current drive letter in the 'path' kernel32 test

Dmitry Timoshkov dmitry at baikal.ru
Tue Feb 24 08:59:13 CST 2004


Hello,

Sorry, my previous patch wasn't correct, curDrive already contains
drive letter, not a numerical value.

--
Finally I've found why the kernel path test was succeeding inside
the todo_wine block: I have "Temp" = "c:\\windows\\temp" in my
config file, while the default config has "Temp" = "e:\\" and
the path test has a hardcoded C: for current drive letter.

Now path tests has no unexpected successes for me.

Changelog:
    Dmitry Timoshkov <dmitry at codeweavers.com>
    Do not hardcode C: for current drive letter in the 'path' kernel32 test.

--- cvs/hq/wine/dlls/kernel/tests/path.c	Sat Feb  7 19:17:44 2004
+++ wine/dlls/kernel/tests/path.c	Tue Feb 24 20:27:00 2004
@@ -751,9 +751,9 @@ static void test_PathNameA(CHAR *curdir,
   sprintf(tmpstr,"/%s/%s",SHORTDIR,SHORTFILE);
   ok(GetFullPathNameA(tmpstr,MAX_PATH,tmpstr1,&strptr),"GetFullPathNameA failed\n");
   todo_wine {
     if( curDrive != NOT_A_VALID_DRIVE) {
-      sprintf(tmpstr,"C:\\%s\\%s",SHORTDIR,SHORTFILE);
+      sprintf(tmpstr, "%c:\\%s\\%s", curDrive, SHORTDIR, SHORTFILE);
       ok(lstrcmpiA(tmpstr,tmpstr1)==0,
          "GetFullPathNameA returned '%s' instead of '%s'\n",tmpstr1,tmpstr);
     }
   }






More information about the wine-patches mailing list