qmgr: Use FAILED instead of !SUCCEEDED.

dlls/qmgr/file.c

@@ -425,7 +425,7 @@ BOOL processFile(BackgroundCopyFileImpl *file, BackgroundCopyJobImpl *job)
             return FALSE;
         }
     }
-    else if (!SUCCEEDED(hr))
+    else if (FAILED(hr))
     {
         ERR("URLDownload failed: eh 0x%08x\n", hr);
         transitionJobState(job, BG_JOB_STATE_TRANSFERRING, BG_JOB_STATE_ERROR);

dlls/qmgr/job.c

@@ -83,7 +83,7 @@ static HRESULT WINAPI BITS_IBackgroundCopyJob_AddFileSet(
     {
         HRESULT hr = IBackgroundCopyJob_AddFile(iface, pFileSet[i].RemoteName,
                                                 pFileSet[i].LocalName);
-        if (!SUCCEEDED(hr))
+        if (FAILED(hr))
             return hr;
     }
     return S_OK;

dlls/qmgr/service.c

@@ -85,20 +85,20 @@ StartCount(void)
     TRACE("\n");
 
     hr = CoInitializeEx(NULL, COINIT_MULTITHREADED);
-    if (!SUCCEEDED(hr))
+    if (FAILED(hr))
         return FALSE;
 
     hr = CoInitializeSecurity(NULL, -1, NULL, NULL, RPC_C_AUTHN_LEVEL_NONE,
                               RPC_C_IMP_LEVEL_IMPERSONATE, NULL, EOAC_NONE,
                               NULL);
-    if (!SUCCEEDED(hr))
+    if (FAILED(hr))
         return FALSE;
 
     hr = CoRegisterClassObject(&CLSID_BackgroundCopyManager,
                                (IUnknown *) &BITS_ClassFactory,
                                CLSCTX_LOCAL_SERVER, REGCLS_MULTIPLEUSE,
                                &dwReg);
-    if (!SUCCEEDED(hr))
+    if (FAILED(hr))
         return FALSE;
 
     return TRUE;