[PATCH] kernel32/tests: Don't skip the CompareStringA() tests on Windows.

Francois Gouget fgouget at codeweavers.com
Sun Aug 22 11:15:42 CDT 2021


Signed-off-by: Francois Gouget <fgouget at codeweavers.com>
---
Found another if (0)...
---
 dlls/kernel32/tests/locale.c | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/dlls/kernel32/tests/locale.c b/dlls/kernel32/tests/locale.c
index c65c2cce3f2..4625283035c 100644
--- a/dlls/kernel32/tests/locale.c
+++ b/dlls/kernel32/tests/locale.c
@@ -1903,9 +1903,13 @@ static void test_CompareStringA(void)
 
     ret = lstrcmpA(NULL, "");
     ok (ret == -1 || broken(ret == -2) /* win9x */, "lstrcmpA(NULL, \"\") should return -1, got %d\n", ret);
- 
 
-    if (0) { /* this requires collation table patch to make it MS compatible */
+
+    /* this requires collation table patch to make it MS compatible */
+    if (strcmp(winetest_platform, "wine") == 0)
+        skip("in Wine due to the lack of a compatible collation table\n");
+    else
+    {
     ret = CompareStringA(LOCALE_SYSTEM_DEFAULT, 0, "'o", -1, "-o", -1 );
     ok(ret == CSTR_LESS_THAN, "'o vs -o expected CSTR_LESS_THAN, got %d\n", ret);
 
-- 
2.20.1



More information about the wine-devel mailing list