quartz: Use the right FAILED/SUCCEEDED macro instead of negating the opposite.
Michael Stefaniuc
mstefani at redhat.de
Tue Oct 7 18:33:43 CDT 2008
---
dlls/quartz/filtergraph.c | 4 ++--
dlls/quartz/filtermapper.c | 2 +-
2 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/dlls/quartz/filtergraph.c b/dlls/quartz/filtergraph.c
index c9e4e56..5837a47 100644
--- a/dlls/quartz/filtergraph.c
+++ b/dlls/quartz/filtergraph.c
@@ -1367,7 +1367,7 @@ error:
IFilterGraph2_RemoveFilter(iface, pfilter);
IBaseFilter_Release(pfilter);
}
- if (!FAILED(hr)) DebugBreak();
+ if (SUCCEEDED(hr)) DebugBreak();
}
IEnumMoniker_Release(pEnumMoniker);
@@ -2140,7 +2140,7 @@ static HRESULT all_renderers_seek(IFilterGraphImpl *This, fnFoundSeek FoundSeek,
IMediaSeeking_Release(seek);
if (hr_return != E_NOTIMPL)
allnotimpl = FALSE;
- if (hr_return == S_OK || (FAILED(hr) && hr != E_NOTIMPL && !FAILED(hr_return)))
+ if (hr_return == S_OK || (FAILED(hr) && hr != E_NOTIMPL && SUCCEEDED(hr_return)))
hr_return = hr;
}
}
diff --git a/dlls/quartz/filtermapper.c b/dlls/quartz/filtermapper.c
index a085b0a..5732755 100644
--- a/dlls/quartz/filtermapper.c
+++ b/dlls/quartz/filtermapper.c
@@ -1297,7 +1297,7 @@ static HRESULT WINAPI FilterMapper_EnumMatchingFilters(
NULL,
&GUID_NULL);
- if (!SUCCEEDED(hr))
+ if (FAILED(hr))
return hr;
while(IEnumMoniker_Next(ppEnumMoniker, 1, &IMon, &nb) == S_OK)
--
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/293cfc7b/attachment.pgp
More information about the wine-patches
mailing list