msctf/tests: Return if the constructor fails. This avoids several clang warnings.

Signed-off-by: Huw Davies <huw@codeweavers.com> Signed-off-by: Alexandre Julliard <julliard@winehq.org>

msctf/tests: Return if the constructor fails. This avoids several clang warnings.
aac4cc8f · Huw Davies · Alexandre Julliard · 5fb2267a · aac4cc8f
Commit aac4cc8f authored Jan 17, 2017 by Huw Davies Committed by Alexandre Julliard Jan 17, 2017
Hide whitespace changes
Inline Side-by-side

Showing with 5 additions and 5 deletions

inputprocessor.c dlls/msctf/tests/inputprocessor.c +5 -5

No files found.
--- a/dlls/msctf/tests/inputprocessor.c
+++ b/dlls/msctf/tests/inputprocessor.c
@@ -1550,11 +1550,11 @@ static void test_startSession(void)
    ITfDocumentMgr_Release(dmtest);

    hr = TextStoreACP_Constructor((IUnknown**)&ts);
-    if (SUCCEEDED(hr))
-    {
-        hr = ITfDocumentMgr_CreateContext(g_dm, cid, 0, (IUnknown*)ts, &cxt, &editCookie);
-        ok(SUCCEEDED(hr),"CreateContext Failed\n");
-    }
+    ok(SUCCEEDED(hr),"Constructor Failed\n");
+    if (FAILED(hr)) return;
+
+    hr = ITfDocumentMgr_CreateContext(g_dm, cid, 0, (IUnknown*)ts, &cxt, &editCookie);
+    ok(SUCCEEDED(hr),"CreateContext Failed\n");

    hr = ITfDocumentMgr_CreateContext(g_dm, cid, 0, NULL, &cxt2, &editCookie);
    ok(SUCCEEDED(hr),"CreateContext Failed\n");