February 2011 - Wine-Commits

Austin English : inetmib1/tests: Add missing broken().
by Alexandre Julliard Feb. 28, 2011

Feb. 28, 2011

Module: wine Branch: master Commit: ead0efbacdcff46b5a9ef220aa713f9a42e9e463 URL: http://source.winehq.org/git/wine.git/?a=commit;h=ead0efbacdcff46b5a9ef220a… Author: Austin English <austinenglish(a)gmail.com> Date: Fri Feb 25 14:34:28 2011 -0800 inetmib1/tests: Add missing broken(). --- dlls/inetmib1/tests/main.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/dlls/inetmib1/tests/main.c b/dlls/inetmib1/tests/main.c index 03c9b62..80b3618 100644 --- a/dlls/inetmib1/tests/main.c +++ b/dlls/inetmib1/tests/main.c @@ -109,7 +109,7 @@ static void testQuery(void) ret = pQuery(SNMP_PDU_GET, &list, &error, &index); ok(ret, "SnmpExtensionQuery failed: %d, %d\n", error, index); ok(error == SNMP_ERRORSTATUS_NOERROR || - error == ERROR_FILE_NOT_FOUND /* Win9x */, + broken(error == ERROR_FILE_NOT_FOUND) /* NT4 */, "expected SNMP_ERRORSTATUS_NOERROR or ERROR_FILE_NOT_FOUND, got %d\n", error); if (error == SNMP_ERRORSTATUS_NOERROR)

1 0

Austin English : hlink/tests: Remove win9x hacks.
by Alexandre Julliard Feb. 28, 2011

Feb. 28, 2011

Module: wine Branch: master Commit: 5cce62a8f4d06b44d645b1814b7a658075324bc1 URL: http://source.winehq.org/git/wine.git/?a=commit;h=5cce62a8f4d06b44d645b1814… Author: Austin English <austinenglish(a)gmail.com> Date: Fri Feb 25 14:33:41 2011 -0800 hlink/tests: Remove win9x hacks. --- dlls/hlink/tests/hlink.c | 7 +------ 1 files changed, 1 insertions(+), 6 deletions(-) diff --git a/dlls/hlink/tests/hlink.c b/dlls/hlink/tests/hlink.c index d151aa5..2b2cfaa 100644 --- a/dlls/hlink/tests/hlink.c +++ b/dlls/hlink/tests/hlink.c @@ -355,7 +355,6 @@ static void test_persist_save_data(const char *testname, IHlink *lnk, const unsigned char *data; DWORD i; BOOL same; - unsigned int expected_data_win9x_size = 0; hr = IHlink_QueryInterface(lnk, &IID_IPersistStream, (void **)&ps); ok(hr == S_OK, "IHlink_QueryInterface failed with error 0x%08x\n", hr); @@ -373,13 +372,9 @@ static void test_persist_save_data(const char *testname, IHlink *lnk, data = GlobalLock(hglobal); - if (expected_data_size % 4) - expected_data_win9x_size = 4 * ((expected_data_size / 4) + 1); - /* first check we have the right amount of data */ ok((data_size == expected_data_size) || - (data_size == expected_data_alt_size) || - broken(data_size == expected_data_win9x_size), /* Win9x and WinMe */ + (data_size == expected_data_alt_size), "%s: Size of saved data differs (expected %d or %d, actual %d)\n", testname, expected_data_size, expected_data_alt_size, data_size);

1 0

Austin English : fusion/tests: Remove win9x hacks.
by Alexandre Julliard Feb. 28, 2011

Feb. 28, 2011

Module: wine Branch: master Commit: cbc81c97d03b08ef2e32073e0bf9efad52ec3a3d URL: http://source.winehq.org/git/wine.git/?a=commit;h=cbc81c97d03b08ef2e32073e0… Author: Austin English <austinenglish(a)gmail.com> Date: Fri Feb 25 14:32:51 2011 -0800 fusion/tests: Remove win9x hacks. --- dlls/fusion/tests/asmcache.c | 40 ++++++++++------------------------------ 1 files changed, 10 insertions(+), 30 deletions(-) diff --git a/dlls/fusion/tests/asmcache.c b/dlls/fusion/tests/asmcache.c index 8af1553..900d23a 100644 --- a/dlls/fusion/tests/asmcache.c +++ b/dlls/fusion/tests/asmcache.c @@ -1187,9 +1187,7 @@ static void test_QueryAssemblyInfo(void) "Expected 0, got %d\n", info.uliAssemblySizeInKB.u.HighPart); todo_wine { - /* win9x: 32 */ - ok((info.uliAssemblySizeInKB.u.LowPart == 4) || - broken(info.uliAssemblySizeInKB.u.LowPart == 32), + ok((info.uliAssemblySizeInKB.u.LowPart == 4), "Expected 4, got %d\n", info.uliAssemblySizeInKB.u.LowPart); } ok(!lstrcmpW(info.pszCurrentAssemblyPathBuf, asmpath), @@ -1210,9 +1208,7 @@ static void test_QueryAssemblyInfo(void) "Expected 0, got %d\n", info.uliAssemblySizeInKB.u.HighPart); todo_wine { - /* win9x: 32 */ - ok((info.uliAssemblySizeInKB.u.LowPart == 4) || - broken(info.uliAssemblySizeInKB.u.LowPart == 32), + ok((info.uliAssemblySizeInKB.u.LowPart == 4), "Expected 4, got %d\n", info.uliAssemblySizeInKB.u.LowPart); } ok(!lstrcmpW(info.pszCurrentAssemblyPathBuf, asmpath), @@ -1235,9 +1231,7 @@ static void test_QueryAssemblyInfo(void) "Expected HRESULT_FROM_WIN32(ERROR_INSUFFICIENT_BUFFER), got %08x\n", hr); todo_wine { - /* win9x: 32 */ - ok((info.uliAssemblySizeInKB.u.LowPart == 4) || - broken(info.uliAssemblySizeInKB.u.LowPart == 32), + ok((info.uliAssemblySizeInKB.u.LowPart == 4), "Expected 4, got %d\n", info.uliAssemblySizeInKB.u.LowPart); } ok(info.cchBuf == lstrlenW(asmpath) + 1, @@ -1260,9 +1254,7 @@ static void test_QueryAssemblyInfo(void) "Expected HRESULT_FROM_WIN32(ERROR_INSUFFICIENT_BUFFER), got %08x\n", hr); todo_wine { - /* win9x: 32 */ - ok((info.uliAssemblySizeInKB.u.LowPart == 4) || - broken(info.uliAssemblySizeInKB.u.LowPart == 32), + ok((info.uliAssemblySizeInKB.u.LowPart == 4), "Expected 4, got %d\n", info.uliAssemblySizeInKB.u.LowPart); } ok(info.cchBuf == lstrlenW(asmpath) + 1, @@ -1284,9 +1276,7 @@ static void test_QueryAssemblyInfo(void) "Expected %d, got %d\n", lstrlenW(asmpath) + 1, info.cchBuf); todo_wine { - /* win9x: 32 */ - ok((info.uliAssemblySizeInKB.u.LowPart == 4) || - broken(info.uliAssemblySizeInKB.u.LowPart == 32), + ok((info.uliAssemblySizeInKB.u.LowPart == 4), "Expected 4, got %d\n", info.uliAssemblySizeInKB.u.LowPart); } ok(!lstrcmpW(info.pszCurrentAssemblyPathBuf, asmpath), @@ -1308,9 +1298,7 @@ static void test_QueryAssemblyInfo(void) "Expected 0, got %d\n", info.uliAssemblySizeInKB.u.HighPart); todo_wine { - /* win9x: 32 */ - ok((info.uliAssemblySizeInKB.u.LowPart == 4) || - broken(info.uliAssemblySizeInKB.u.LowPart == 32), + ok((info.uliAssemblySizeInKB.u.LowPart == 4), "Expected 4, got %d\n", info.uliAssemblySizeInKB.u.LowPart); } ok(!lstrcmpW(info.pszCurrentAssemblyPathBuf, asmpath), @@ -1341,9 +1329,7 @@ static void test_QueryAssemblyInfo(void) "Expected 0, got %d\n", info.uliAssemblySizeInKB.u.HighPart); todo_wine { - /* win9x: 32 */ - ok((info.uliAssemblySizeInKB.u.LowPart == 4) || - broken(info.uliAssemblySizeInKB.u.LowPart == 32), + ok((info.uliAssemblySizeInKB.u.LowPart == 4), "Expected 4, got %d\n", info.uliAssemblySizeInKB.u.LowPart); } ok(!lstrcmpW(info.pszCurrentAssemblyPathBuf, asmpath), @@ -1387,9 +1373,7 @@ static void test_QueryAssemblyInfo(void) "Expected 0, got %d\n", info.uliAssemblySizeInKB.u.HighPart); todo_wine { - /* win9x: 32 */ - ok((info.uliAssemblySizeInKB.u.LowPart == 4) || - broken(info.uliAssemblySizeInKB.u.LowPart == 32), + ok((info.uliAssemblySizeInKB.u.LowPart == 4), "Expected 4, got %d\n", info.uliAssemblySizeInKB.u.LowPart); } ok(!lstrcmpW(info.pszCurrentAssemblyPathBuf, asmpath), @@ -1439,9 +1423,7 @@ static void test_QueryAssemblyInfo(void) "Expected 0, got %d\n", info.uliAssemblySizeInKB.u.HighPart); todo_wine { - /* win9x: 32 */ - ok((info.uliAssemblySizeInKB.u.LowPart == 4) || - broken(info.uliAssemblySizeInKB.u.LowPart == 32), + ok((info.uliAssemblySizeInKB.u.LowPart == 4), "Expected 4, got %d\n", info.uliAssemblySizeInKB.u.LowPart); } ok(!lstrcmpW(info.pszCurrentAssemblyPathBuf, asmpath), @@ -1485,9 +1467,7 @@ static void test_QueryAssemblyInfo(void) "Expected 0, got %d\n", info.uliAssemblySizeInKB.u.HighPart); todo_wine { - /* win9x: 32 */ - ok((info.uliAssemblySizeInKB.u.LowPart == 4) || - broken(info.uliAssemblySizeInKB.u.LowPart == 32), + ok((info.uliAssemblySizeInKB.u.LowPart == 4), "Expected 4, got %d\n", info.uliAssemblySizeInKB.u.LowPart); } ok(!lstrcmpW(info.pszCurrentAssemblyPathBuf, asmpath),

1 0

Austin English : kernel32/tests: Remove win9x hacks.
by Alexandre Julliard Feb. 28, 2011

Feb. 28, 2011

Module: wine Branch: master Commit: d385435876febbf48c1e0c70e329aa23a1958c54 URL: http://source.winehq.org/git/wine.git/?a=commit;h=d385435876febbf48c1e0c70e… Author: Austin English <austinenglish(a)gmail.com> Date: Fri Feb 25 14:34:56 2011 -0800 kernel32/tests: Remove win9x hacks. --- dlls/kernel32/tests/volume.c | 36 ++++++++++++------------------------ 1 files changed, 12 insertions(+), 24 deletions(-) diff --git a/dlls/kernel32/tests/volume.c b/dlls/kernel32/tests/volume.c index 4484a80..15f7e3c 100644 --- a/dlls/kernel32/tests/volume.c +++ b/dlls/kernel32/tests/volume.c @@ -40,14 +40,12 @@ static void test_query_dos_deviceA(void) char drivestr[] = "a:"; char *p, *buffer, buffer2[2000]; DWORD ret, ret2, buflen=32768; - BOOL iswin9x, found = FALSE; + BOOL found = FALSE; buffer = HeapAlloc( GetProcessHeap(), 0, buflen ); SetLastError(0xdeadbeef); ret = QueryDosDeviceA( NULL, buffer, buflen ); - iswin9x = !ret && (GetLastError() == ERROR_INVALID_PARAMETER /* win98 */ - || GetLastError() == ERROR_CALL_NOT_IMPLEMENTED /* win95*/); - ok((ret && GetLastError() != ERROR_INSUFFICIENT_BUFFER) || broken(iswin9x), + ok((ret && GetLastError() != ERROR_INSUFFICIENT_BUFFER), "QueryDosDeviceA failed to return list, last error %u\n", GetLastError()); if (ret && GetLastError() != ERROR_INSUFFICIENT_BUFFER) { @@ -64,15 +62,14 @@ static void test_query_dos_deviceA(void) for (;drivestr[0] <= 'z'; drivestr[0]++) { /* Older W2K fails with ERROR_INSUFFICIENT_BUFFER when buflen is > 32767 */ ret = QueryDosDeviceA( drivestr, buffer, buflen - 1); - /* fails for all drives in win9x */ - ok(ret || GetLastError() == ERROR_FILE_NOT_FOUND || broken(!ret && iswin9x), + ok(ret || GetLastError() == ERROR_FILE_NOT_FOUND, "QueryDosDeviceA failed to return current mapping for %s, last error %u\n", drivestr, GetLastError()); if(ret) { for (p = buffer; *p; p++) *p = toupper(*p); if (strstr(buffer, "HARDDISK") || strstr(buffer, "RAMDISK")) found = TRUE; } } - ok(found ^ iswin9x, "expected at least one devicename to contain HARDDISK or RAMDISK except on win9x\n"); + ok(found, "expected at least one devicename to contain HARDDISK or RAMDISK\n"); HeapFree( GetProcessHeap(), 0, buffer ); } @@ -239,9 +236,7 @@ static void test_GetLogicalDriveStringsA(void) ok(size2 == size-1, "size2 = %d\n", size2); for(ptr = buf; ptr < buf+size2; ptr += 4) { - ok(('A' <= *ptr && *ptr <= 'Z') || - (broken('a' <= *ptr && *ptr <= 'z')), /* Win9x and WinMe */ - "device name '%c' is not uppercase\n", *ptr); + ok(('A' <= *ptr && *ptr <= 'Z'), "device name '%c' is not uppercase\n", *ptr); ok(ptr[1] == ':', "ptr[1] = %c, expected ':'\n", ptr[1]); ok(ptr[2] == '\\', "ptr[2] = %c expected '\\'\n", ptr[2]); ok(!ptr[3], "ptr[3] = %c expected nullbyte\n", ptr[3]); @@ -333,8 +328,7 @@ static void test_GetVolumeInformationA(void) SetLastError(0xdeadbeef); ret = pGetVolumeInformationA(Root_Colon, vol_name_buf, vol_name_size, NULL, NULL, NULL, fs_name_buf, fs_name_len); - ok(!ret && (GetLastError() == ERROR_INVALID_NAME || - broken(GetLastError() == ERROR_BAD_PATHNAME/* win9x */)), + ok(!ret && (GetLastError() == ERROR_INVALID_NAME), "GetVolumeInformationA did%s fail, last error %u\n", ret ? " not":"", GetLastError()); /* reset current directory */ @@ -362,8 +356,7 @@ static void test_GetVolumeInformationA(void) SetLastError(0xdeadbeef); ret = pGetVolumeInformationA(Root_Colon, vol_name_buf, vol_name_size, NULL, NULL, NULL, fs_name_buf, fs_name_len); - ok(!ret && (GetLastError() == ERROR_INVALID_NAME || - broken(GetLastError() == ERROR_BAD_PATHNAME/* Win9x */)), + ok(!ret && (GetLastError() == ERROR_INVALID_NAME), "GetVolumeInformationA did%s fail, last error %u\n", ret ? " not":"", GetLastError()); /* Try normal drive letter with trailing \ */ @@ -381,8 +374,7 @@ static void test_GetVolumeInformationA(void) SetLastError(0xdeadbeef); ret = pGetVolumeInformationA(Root_Colon, vol_name_buf, vol_name_size, NULL, NULL, NULL, fs_name_buf, fs_name_len); - ok(!ret && (GetLastError() == ERROR_INVALID_NAME || - broken(GetLastError() == ERROR_BAD_PATHNAME/* Win9x */)), + ok(!ret && (GetLastError() == ERROR_INVALID_NAME), "GetVolumeInformationA did%s fail, last error %u\n", ret ? " not":"", GetLastError()); /* Now C:\ becomes the current directory on drive C: */ @@ -420,32 +412,28 @@ static void test_GetVolumeInformationA(void) SetLastError(0xdeadbeef); ret = pGetVolumeInformationA(Root_UNC, vol_name_buf, vol_name_size, &vol_serial_num, &max_comp_len, &fs_flags, fs_name_buf, fs_name_len); - ok(ret || broken(!ret /* win9x */ && GetLastError()==ERROR_BAD_NETPATH), - "GetVolumeInformationA did%s fail, root=%s, last error=%u\n", ret ? " not":"", Root_UNC, GetLastError()); + ok(ret, "GetVolumeInformationA did%s fail, root=%s, last error=%u\n", ret ? " not":"", Root_UNC, GetLastError()); /* try again with device name space */ Root_UNC[2] = '.'; SetLastError(0xdeadbeef); ret = pGetVolumeInformationA(Root_UNC, vol_name_buf, vol_name_size, &vol_serial_num, &max_comp_len, &fs_flags, fs_name_buf, fs_name_len); - ok(ret || broken(!ret /* win9x */ && GetLastError()==ERROR_BAD_NETPATH), - "GetVolumeInformationA did%s fail, root=%s, last error=%u\n", ret ? " not":"", Root_UNC, GetLastError()); + ok(ret, "GetVolumeInformationA did%s fail, root=%s, last error=%u\n", ret ? " not":"", Root_UNC, GetLastError()); /* try again with a directory off the root - should generate error */ if (windowsdir[strlen(windowsdir)-1] != '\\') strcat(windowsdir, "\\"); SetLastError(0xdeadbeef); ret = pGetVolumeInformationA(windowsdir, vol_name_buf, vol_name_size, &vol_serial_num, &max_comp_len, &fs_flags, fs_name_buf, fs_name_len); - ok(!ret && (GetLastError()==ERROR_DIR_NOT_ROOT || - broken(GetLastError()==ERROR_BAD_PATHNAME/* win9x */)), + ok(!ret && (GetLastError()==ERROR_DIR_NOT_ROOT), "GetVolumeInformationA did%s fail, root=%s, last error=%u\n", ret ? " not":"", windowsdir, GetLastError()); /* A subdir with trailing \ yields DIR_NOT_ROOT instead of INVALID_NAME */ if (windowsdir[strlen(windowsdir)-1] == '\\') windowsdir[strlen(windowsdir)-1] = 0; SetLastError(0xdeadbeef); ret = pGetVolumeInformationA(windowsdir, vol_name_buf, vol_name_size, &vol_serial_num, &max_comp_len, &fs_flags, fs_name_buf, fs_name_len); - ok(!ret && (GetLastError()==ERROR_INVALID_NAME || - broken(GetLastError()==ERROR_BAD_PATHNAME/* win9x */)), + ok(!ret && (GetLastError()==ERROR_INVALID_NAME), "GetVolumeInformationA did%s fail, root=%s, last error=%u\n", ret ? " not":"", windowsdir, GetLastError()); if (!pGetVolumeNameForVolumeMountPointA) {

1 0

Austin English : mlang/tests: Remove win9x hacks.
by Alexandre Julliard Feb. 28, 2011

Feb. 28, 2011

Module: wine Branch: master Commit: 5ed5f21d0277deddfe8f0a5fb3b12646df69b15e URL: http://source.winehq.org/git/wine.git/?a=commit;h=5ed5f21d0277deddfe8f0a5fb… Author: Austin English <austinenglish(a)gmail.com> Date: Fri Feb 25 14:35:46 2011 -0800 mlang/tests: Remove win9x hacks. --- dlls/mlang/tests/mlang.c | 52 ++++++--------------------------------------- 1 files changed, 7 insertions(+), 45 deletions(-) diff --git a/dlls/mlang/tests/mlang.c b/dlls/mlang/tests/mlang.c index 96a19a6..65e0266 100644 --- a/dlls/mlang/tests/mlang.c +++ b/dlls/mlang/tests/mlang.c @@ -254,28 +254,6 @@ static BOOL init_function_ptrs(void) ok(0, (format), string1, string2); \ } -/* lstrcmpW is not supported on Win9x! */ -static int mylstrcmpW(const WCHAR* str1, const WCHAR* str2) -{ - if (!str2) return 1; - while (*str1 && *str1==*str2) { - str1++; - str2++; - } - return *str1-*str2; -} - -/* lstrcpyW is not supported on Win95 */ -static void mylstrcpyW(WCHAR* str1, const WCHAR* str2) -{ - while (str2 && *str2) { - *str1 = *str2; - str1++; - str2++; - } - *str1 = '\0'; -} - static void test_multibyte_to_unicode_translations(IMultiLanguage2 *iML2) { /* these APIs are broken regarding constness of the input buffer */ @@ -586,8 +564,6 @@ static void test_EnumCodePages(IMultiLanguage2 *iML2, DWORD flags) #ifdef DUMP_CP_INFO trace("%u: codepage %u family %u\n", i, cpinfo[i].uiCodePage, cpinfo[i].uiFamilyCodePage); #endif - /* Win95 does not support UTF-7 */ - if (cpinfo[i].uiCodePage == CP_UTF7) continue; /* support files for some codepages might be not installed, or * the codepage is just an alias. @@ -1227,21 +1203,21 @@ static void test_GetRfc1766Info(IMultiLanguage2 *iML2) "#%02d: got '%s' (expected '%s')\n", i, rfc1766A, info_table[i].rfc1766); /* Some IE versions truncate an oversized name one character to short */ - mylstrcpyW(short_broken_name, info_table[i].broken_name); + lstrcpyW(short_broken_name, info_table[i].broken_name); short_broken_name[MAX_LOCALE_NAME - 2] = '\0'; if (info_table[i].todo & TODO_NAME) { todo_wine - ok( (!mylstrcmpW(prfc->wszLocaleName, info_table[i].localename)) || - broken(!mylstrcmpW(prfc->wszLocaleName, info_table[i].broken_name)) || /* IE < 6.0 */ - broken(!mylstrcmpW(prfc->wszLocaleName, short_broken_name)), + ok( (!lstrcmpW(prfc->wszLocaleName, info_table[i].localename)) || + broken(!lstrcmpW(prfc->wszLocaleName, info_table[i].broken_name)) || /* IE < 6.0 */ + broken(!lstrcmpW(prfc->wszLocaleName, short_broken_name)), "#%02d: got %s (expected %s)\n", i, wine_dbgstr_w(prfc->wszLocaleName), wine_dbgstr_w(info_table[i].localename)); } else - ok( (!mylstrcmpW(prfc->wszLocaleName, info_table[i].localename)) || - broken(!mylstrcmpW(prfc->wszLocaleName, info_table[i].broken_name)) || /* IE < 6.0 */ - broken(!mylstrcmpW(prfc->wszLocaleName, short_broken_name)), + ok( (!lstrcmpW(prfc->wszLocaleName, info_table[i].localename)) || + broken(!lstrcmpW(prfc->wszLocaleName, info_table[i].broken_name)) || /* IE < 6.0 */ + broken(!lstrcmpW(prfc->wszLocaleName, short_broken_name)), "#%02d: got %s (expected %s)\n", i, wine_dbgstr_w(prfc->wszLocaleName), wine_dbgstr_w(info_table[i].localename)); @@ -1902,20 +1878,6 @@ static void test_IsCodePageInstallable(IMultiLanguage2 *ml2) UINT i; HRESULT hr; - SetLastError(0xdeadbeef); - lstrcmpW(NULL, NULL); - if (GetLastError() == ERROR_CALL_NOT_IMPLEMENTED) - { - /* This corruption leads (sometimes) to test failures in oleaut32 but also - * to the inability to use the Regional Settings. - * This only seems to be an issue with Win98 and IE6 (mlang version 6.0.2800.1106). - * - * A reboot restores the codepages again. - */ - win_skip("IsCodePageInstallable could mess up the codepages on Win98\n"); - return; - } - for (i = 0; i < 0xffff; i++) { hr = IMultiLanguage2_IsCodePageInstallable(ml2, i);

1 0

Austin English : gdi32/tests: Remove win9x hacks.
by Alexandre Julliard Feb. 28, 2011

Feb. 28, 2011

Module: wine Branch: master Commit: 197f2f8cada0b6b78e9c7c77b4b60ccf618d23aa URL: http://source.winehq.org/git/wine.git/?a=commit;h=197f2f8cada0b6b78e9c7c77b… Author: Austin English <austinenglish(a)gmail.com> Date: Fri Feb 25 14:33:17 2011 -0800 gdi32/tests: Remove win9x hacks. --- dlls/gdi32/tests/icm.c | 34 ++++++++++++---------------------- 1 files changed, 12 insertions(+), 22 deletions(-) diff --git a/dlls/gdi32/tests/icm.c b/dlls/gdi32/tests/icm.c index a1c4f97..86f173a 100644 --- a/dlls/gdi32/tests/icm.c +++ b/dlls/gdi32/tests/icm.c @@ -63,16 +63,10 @@ static void test_GetICMProfileA( HDC dc ) ok( !ret, "GetICMProfileA succeeded\n" ); ok( size, "expected size > 0\n" ); ok( filename[0] == 0, "Expected filename to be empty\n" ); - ok( error == ERROR_INSUFFICIENT_BUFFER || - error == ERROR_SUCCESS, /* Win95 */ - "got %d, expected ERROR_INSUFFICIENT_BUFFER or ERROR_SUCCESS(Win95)\n", error ); + ok( error == ERROR_INSUFFICIENT_BUFFER, "got %d, expected ERROR_INSUFFICIENT_BUFFER\n", error ); - /* Next test will crash on Win95 */ - if ( error == ERROR_INSUFFICIENT_BUFFER ) - { - ret = GetICMProfileA( dc, NULL, filename ); - ok( !ret, "GetICMProfileA succeeded\n" ); - } + ret = GetICMProfileA( dc, NULL, filename ); + ok( !ret, "GetICMProfileA succeeded\n" ); size = MAX_PATH; ret = GetICMProfileA( dc, &size, filename ); @@ -190,15 +184,11 @@ static void test_EnumICMProfilesA( HDC dc ) ok(ret == -1 || broken(ret == 0) /* nt4 */, "expected -1, got %d\n", ret); ret = EnumICMProfilesA( dc, enum_profiles_callbackA, 0 ); - ok(ret == -1 || broken(ret == 0) /* nt4 */ || broken(ret == 1) /* win95 */, + ok(ret == -1 || broken(ret == 0) /* nt4 */, "expected -1, got %d\n", ret); - if (ret != 1) - { - /* Crashes on Win95 */ - ret = EnumICMProfilesA( dc, NULL, 0 ); - ok(ret == -1 || broken(ret == 0) /* nt4 */, "expected -1, got %d\n", ret); - } + ret = EnumICMProfilesA( dc, NULL, 0 ); + ok(ret == -1 || broken(ret == 0) /* nt4 */, "expected -1, got %d\n", ret); } static CALLBACK INT enum_profiles_callbackW( LPWSTR filename, LPARAM lparam ) @@ -211,13 +201,13 @@ static void test_EnumICMProfilesW( HDC dc ) INT ret; ret = EnumICMProfilesW( NULL, NULL, 0 ); - ok(ret == -1 || broken(ret == 0) /* win9x, nt4 */, "expected -1, got %d\n", ret); + ok(ret == -1 || broken(ret == 0) /* NT4 */, "expected -1, got %d\n", ret); ret = EnumICMProfilesW( dc, NULL, 0 ); - ok(ret == -1 || broken(ret == 0) /* win9x, nt4 */, "expected -1, got %d\n", ret); + ok(ret == -1 || broken(ret == 0) /* NT4 */, "expected -1, got %d\n", ret); ret = EnumICMProfilesW( dc, enum_profiles_callbackW, 0 ); - ok(ret == -1 || ret == 1 || broken(ret == 0) /* win9x, nt4 */, "expected -1, got %d\n", ret); + ok(ret == -1 || ret == 1 || broken(ret == 0) /* NT4 */, "expected -1, got %d\n", ret); } static void test_SetICMProfileA( HDC dc ) @@ -242,21 +232,21 @@ static void test_SetICMProfileA( HDC dc ) ret = SetICMProfileA( NULL, NULL ); error = GetLastError(); ok(!ret, "SetICMProfileA succeeded\n"); - ok(error == ERROR_INVALID_PARAMETER || broken(error == 0xdeadbeef) /* win9x */, + ok(error == ERROR_INVALID_PARAMETER, "expected ERROR_INVALID_PARAMETER, got %u\n", error); SetLastError( 0xdeadbeef ); ret = SetICMProfileA( NULL, profile ); error = GetLastError(); ok(!ret, "SetICMProfileA succeeded\n"); - ok(error == ERROR_INVALID_HANDLE || broken(error == 0xdeadbeef) /* win9x */, + ok(error == ERROR_INVALID_HANDLE, "expected ERROR_INVALID_HANDLE, got %u\n", error); SetLastError( 0xdeadbeef ); ret = SetICMProfileA( dc, NULL ); error = GetLastError(); ok(!ret, "SetICMProfileA succeeded\n"); - ok(error == ERROR_INVALID_PARAMETER || broken(error == 0xdeadbeef) /* win95 */, + ok(error == ERROR_INVALID_PARAMETER, "expected ERROR_INVALID_PARAMETER, got %u\n", error); ret = SetICMProfileA( dc, profile );

1 0

Austin English : dsound/tests: Remove win9x hack.
by Alexandre Julliard Feb. 28, 2011

Feb. 28, 2011

Module: wine Branch: master Commit: 62b83bf036dcd7ca491ef685934d3dd4509d35ec URL: http://source.winehq.org/git/wine.git/?a=commit;h=62b83bf036dcd7ca491ef6859… Author: Austin English <austinenglish(a)gmail.com> Date: Fri Feb 25 14:32:26 2011 -0800 dsound/tests: Remove win9x hack. --- dlls/dsound/tests/dsound.c | 7 +++---- 1 files changed, 3 insertions(+), 4 deletions(-) diff --git a/dlls/dsound/tests/dsound.c b/dlls/dsound/tests/dsound.c index 0bf04b7..f8b7aeb 100644 --- a/dlls/dsound/tests/dsound.c +++ b/dlls/dsound/tests/dsound.c @@ -750,7 +750,7 @@ static HRESULT test_secondary(LPGUID lpGuid) if (!gotdx8) { - skip("Not doing the WAVE_FORMAT_EXTENSIBLE tests\n"); + win_skip("Not doing the WAVE_FORMAT_EXTENSIBLE tests\n"); /* Apparently they succeed with bogus values, * which means that older dsound doesn't look at them */ @@ -1095,8 +1095,7 @@ START_TEST(dsound) SetLastError(0xdeadbeef); ret = FreeLibrary(hDsound); ok( ret || - broken(!ret && GetLastError() == ERROR_MOD_NOT_FOUND) || /* NT4 */ - broken(!ret && GetLastError() == ERROR_INVALID_HANDLE), /* Win9x */ + broken(!ret && GetLastError() == ERROR_MOD_NOT_FOUND), /* NT4 */ "FreeLibrary(2) returned %d\n", GetLastError()); ok(!FreeLibrary(hDsound), "DirectSound DLL still loaded\n"); } @@ -1118,7 +1117,7 @@ START_TEST(dsound) FreeLibrary(hDsound); } else - skip("dsound.dll not found!\n"); + win_skip("dsound.dll not found!\n"); CoUninitialize(); }

1 0

Austin English : comctl32/tests: Remove win9x hack.
by Alexandre Julliard Feb. 28, 2011

Feb. 28, 2011

Module: wine Branch: master Commit: 7c25705e3cd4c76f9f850fc94acb407125f84bd7 URL: http://source.winehq.org/git/wine.git/?a=commit;h=7c25705e3cd4c76f9f850fc94… Author: Austin English <austinenglish(a)gmail.com> Date: Fri Feb 25 14:29:08 2011 -0800 comctl32/tests: Remove win9x hack. --- dlls/comctl32/tests/header.c | 4 ---- 1 files changed, 0 insertions(+), 4 deletions(-) diff --git a/dlls/comctl32/tests/header.c b/dlls/comctl32/tests/header.c index d9b4a73..0670743 100644 --- a/dlls/comctl32/tests/header.c +++ b/dlls/comctl32/tests/header.c @@ -1024,10 +1024,6 @@ static void test_hdm_filterMessages(HWND hParent) else ok_sequence(sequences, HEADER_SEQ_INDEX, filterMessages_seq_noninteractive, "filterMessages sequence testing", FALSE); - /* Some Win9x versions don't send a WM_KILLFOCUS. - * Set the focus explicitly to the parent to avoid a crash. - */ - SetFocus(hParent); DestroyWindow(hChild); }

1 0

Austin English : d3d8/tests: Remove win9x hack.
by Alexandre Julliard Feb. 28, 2011

Feb. 28, 2011

Module: wine Branch: master Commit: 98678879af2b3324736795f89ed60bda0cf708af URL: http://source.winehq.org/git/wine.git/?a=commit;h=98678879af2b3324736795f89… Author: Austin English <austinenglish(a)gmail.com> Date: Fri Feb 25 14:30:46 2011 -0800 d3d8/tests: Remove win9x hack. --- dlls/d3d8/tests/device.c | 8 ++------ 1 files changed, 2 insertions(+), 6 deletions(-) diff --git a/dlls/d3d8/tests/device.c b/dlls/d3d8/tests/device.c index 4ff9803..6885dd4 100644 --- a/dlls/d3d8/tests/device.c +++ b/dlls/d3d8/tests/device.c @@ -1306,13 +1306,9 @@ static void test_lights(void) /* TODO: Test the rendering results in this situation */ hr = IDirect3DDevice8_LightEnable(device, i + 1, TRUE); - ok(hr == D3D_OK || - broken(hr == D3DERR_INVALIDCALL), /* Some Win9x and WinME */ - "Enabling one light more than supported returned %08x\n", hr); + ok(hr == D3D_OK, "Enabling one light more than supported returned %08x\n", hr); hr = IDirect3DDevice8_GetLightEnable(device, i + 1, &enabled); - ok(hr == D3D_OK || - broken(hr == D3DERR_INVALIDCALL), /* Some Win9x and WinME */ - "GetLightEnable on light %u failed with %08x\n", i + 1, hr); + ok(hr == D3D_OK, "GetLightEnable on light %u failed with %08x\n", i + 1, hr); ok(enabled, "Light %d is %s\n", i + 1, enabled ? "enabled" : "disabled"); hr = IDirect3DDevice8_LightEnable(device, i + 1, FALSE); ok(hr == D3D_OK, "Disabling the additional returned %08x\n", hr);

1 0

Rico Schüller : d3dcompiler/tests: Improve test_reflection_bound_resources().
by Alexandre Julliard Feb. 28, 2011

Feb. 28, 2011

Module: wine Branch: master Commit: f9d4a45fbb667f8db9ee34ab0d4dcf019e3a10e6 URL: http://source.winehq.org/git/wine.git/?a=commit;h=f9d4a45fbb667f8db9ee34ab0… Author: Rico Schüller <kgbricola(a)web.de> Date: Sun Feb 27 22:51:01 2011 +0100 d3dcompiler/tests: Improve test_reflection_bound_resources(). --- dlls/d3dcompiler_43/tests/reflection.c | 12 ++++++------ 1 files changed, 6 insertions(+), 6 deletions(-) diff --git a/dlls/d3dcompiler_43/tests/reflection.c b/dlls/d3dcompiler_43/tests/reflection.c index 76249f7..7d53f7a 100644 --- a/dlls/d3dcompiler_43/tests/reflection.c +++ b/dlls/d3dcompiler_43/tests/reflection.c @@ -1018,22 +1018,22 @@ static void test_reflection_bound_resources(void) /* check invalid cases */ hr = ref11->lpVtbl->GetResourceBindingDesc(ref11, 0, NULL); - ok(hr == E_INVALIDARG, "GetResourceBindingDesc failed %x\n", hr); + ok(hr == E_INVALIDARG, "GetResourceBindingDesc failed, got %x, expected %x\n", hr, E_INVALIDARG); hr = ref11->lpVtbl->GetResourceBindingDesc(ref11, 0xffffffff, &desc); - ok(hr == E_INVALIDARG, "GetResourceBindingDesc failed %x\n", hr); + ok(hr == E_INVALIDARG, "GetResourceBindingDesc failed, got %x, expected %x\n", hr, E_INVALIDARG); hr = ref11->lpVtbl->GetResourceBindingDescByName(ref11, NULL, &desc); - ok(hr == E_INVALIDARG, "GetResourceBindingDescByName failed %x\n", hr); + ok(hr == E_INVALIDARG, "GetResourceBindingDescByName failed, got %x, expected %x\n", hr, E_INVALIDARG); hr = ref11->lpVtbl->GetResourceBindingDescByName(ref11, "sam", NULL); - ok(hr == E_INVALIDARG, "GetResourceBindingDescByName failed %x\n", hr); + ok(hr == E_INVALIDARG, "GetResourceBindingDescByName failed, got %x, expected %x\n", hr, E_INVALIDARG); hr = ref11->lpVtbl->GetResourceBindingDescByName(ref11, "invalid", NULL); - ok(hr == E_INVALIDARG, "GetResourceBindingDescByName failed %x\n", hr); + ok(hr == E_INVALIDARG, "GetResourceBindingDescByName failed, got %x, expected %x\n", hr, E_INVALIDARG); hr = ref11->lpVtbl->GetResourceBindingDescByName(ref11, "invalid", &desc); - ok(hr == E_INVALIDARG, "GetResourceBindingDescByName failed %x\n", hr); + ok(hr == E_INVALIDARG, "GetResourceBindingDescByName failed, got %x, expected %x\n", hr, E_INVALIDARG); /* GetResourceBindingDesc */ for (i = 0; i < sizeof(test_reflection_bound_resources_result)/sizeof(*test_reflection_bound_resources_result); ++i)

1 0