[PATCH 1/3] dwrite/tests: Fix fallback builder test skip

Nikolay Sivov nsivov at codeweavers.com
Fri Sep 22 07:34:50 CDT 2017


Signed-off-by: Nikolay Sivov <nsivov at codeweavers.com>
---
 dlls/dwrite/tests/layout.c | 12 +++++-------
 1 file changed, 5 insertions(+), 7 deletions(-)

diff --git a/dlls/dwrite/tests/layout.c b/dlls/dwrite/tests/layout.c
index ba3e852664..493eaf317f 100644
--- a/dlls/dwrite/tests/layout.c
+++ b/dlls/dwrite/tests/layout.c
@@ -4672,17 +4672,15 @@ static void test_FontFallbackBuilder(void)
     hr = IDWriteFactory_QueryInterface(factory, &IID_IDWriteFactory2, (void**)&factory2);
     IDWriteFactory_Release(factory);
 
-    if (factory2) {
-        EXPECT_REF(factory2, 1);
-        hr = IDWriteFactory2_CreateFontFallbackBuilder(factory2, &builder);
-        EXPECT_REF(factory2, 2);
-    }
-
     if (hr != S_OK) {
-        skip("IDWriteFontFallbackBuilder is not supported\n");
+        win_skip("IDWriteFontFallbackBuilder is not supported\n");
         return;
     }
 
+    EXPECT_REF(factory2, 1);
+    hr = IDWriteFactory2_CreateFontFallbackBuilder(factory2, &builder);
+    EXPECT_REF(factory2, 2);
+
     fallback = NULL;
     EXPECT_REF(factory2, 2);
     EXPECT_REF(builder, 1);
-- 
2.14.1




More information about the wine-patches mailing list