Michael Stefaniuc : qmgr: Use FAILED instead of !SUCCEEDED.

Alexandre Julliard julliard at winehq.org
Wed Oct 8 08:32:55 CDT 2008


Module: wine
Branch: master
Commit: dd843b87b61b606010a0b382dba844368faf5880
URL:    http://source.winehq.org/git/wine.git/?a=commit;h=dd843b87b61b606010a0b382dba844368faf5880

Author: Michael Stefaniuc <mstefani at redhat.de>
Date:   Wed Oct  8 01:33:25 2008 +0200

qmgr: Use FAILED instead of !SUCCEEDED.

---

 dlls/qmgr/file.c    |    2 +-
 dlls/qmgr/job.c     |    2 +-
 dlls/qmgr/service.c |    6 +++---
 3 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/dlls/qmgr/file.c b/dlls/qmgr/file.c
index 622f7a2..4129887 100644
--- a/dlls/qmgr/file.c
+++ b/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);
diff --git a/dlls/qmgr/job.c b/dlls/qmgr/job.c
index e344027..871f14c 100644
--- a/dlls/qmgr/job.c
+++ b/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;
diff --git a/dlls/qmgr/service.c b/dlls/qmgr/service.c
index b00a922..1e0a545 100644
--- a/dlls/qmgr/service.c
+++ b/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;




More information about the wine-cvs mailing list