qcap: Use the right FAILED/SUCCEEDED macro instead of negating the opposite.

Michael Stefaniuc mstefani at redhat.de
Tue Oct 7 18:36:01 CDT 2008


---
 dlls/qcap/qcap_main.c  |    2 +-
 dlls/qcap/vfwcapture.c |    2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/dlls/qcap/qcap_main.c b/dlls/qcap/qcap_main.c
index e5c9e3b..8549f42 100644
--- a/dlls/qcap/qcap_main.c
+++ b/dlls/qcap/qcap_main.c
@@ -265,7 +265,7 @@ static HRESULT WINAPI DSCF_CreateInstance(LPCLASSFACTORY iface, LPUNKNOWN pOuter
     if (!punk)
     {
         /* No object created, update error if it isn't done already and return */
-        if (!FAILED(hres))
+        if (SUCCEEDED(hres))
             hres = E_OUTOFMEMORY;
     return hres;
     }
diff --git a/dlls/qcap/vfwcapture.c b/dlls/qcap/vfwcapture.c
index 738e1d2..cfb9487 100644
--- a/dlls/qcap/vfwcapture.c
+++ b/dlls/qcap/vfwcapture.c
@@ -115,7 +115,7 @@ IUnknown * WINAPI QCAP_createVFWCaptureFilter(IUnknown *pUnkOuter, HRESULT *phr)
     pVfwCapture->csFilter.DebugInfo->Spare[0] = (DWORD_PTR)(__FILE__ ": VfwCapture.csFilter");
     hr = VfwPin_Construct((IBaseFilter *)&pVfwCapture->lpVtbl,
                    &pVfwCapture->csFilter, &pVfwCapture->pOutputPin);
-    if (!SUCCEEDED(hr))
+    if (FAILED(hr))
     {
         CoTaskMemFree(pVfwCapture);
         return NULL;
-- 
1.6.0.2
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
Url : http://www.winehq.org/pipermail/wine-patches/attachments/20081008/d888feb3/attachment.pgp 


More information about the wine-patches mailing list