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