Christian Costa : strmbase: Simplify IEnumMediaTypesImpl_QueryInterface and turn FIXME into WARN.
Alexandre Julliard
julliard at winehq.org
Fri Nov 23 12:53:23 CST 2012
Module: wine
Branch: master
Commit: 3070dcee3b05e3124eae522fa210ec728b6bac21
URL: http://source.winehq.org/git/wine.git/?a=commit;h=3070dcee3b05e3124eae522fa210ec728b6bac21
Author: Christian Costa <titan.costa at gmail.com>
Date: Fri Nov 23 09:47:05 2012 +0100
strmbase: Simplify IEnumMediaTypesImpl_QueryInterface and turn FIXME into WARN.
---
dlls/strmbase/mediatype.c | 21 +++++++++------------
1 files changed, 9 insertions(+), 12 deletions(-)
diff --git a/dlls/strmbase/mediatype.c b/dlls/strmbase/mediatype.c
index a168755..d100cc9 100644
--- a/dlls/strmbase/mediatype.c
+++ b/dlls/strmbase/mediatype.c
@@ -145,24 +145,21 @@ HRESULT WINAPI EnumMediaTypes_Construct(BasePin *basePin, BasePin_GetMediaType e
return S_OK;
}
-static HRESULT WINAPI IEnumMediaTypesImpl_QueryInterface(IEnumMediaTypes * iface, REFIID riid, LPVOID * ppv)
+static HRESULT WINAPI IEnumMediaTypesImpl_QueryInterface(IEnumMediaTypes * iface, REFIID riid, void ** ret_iface)
{
- TRACE("(%p)->(%s, %p)\n", iface, debugstr_guid(riid), ppv);
+ TRACE("(%p)->(%s, %p)\n", iface, debugstr_guid(riid), ret_iface);
- *ppv = NULL;
-
- if (IsEqualIID(riid, &IID_IUnknown))
- *ppv = iface;
- else if (IsEqualIID(riid, &IID_IEnumMediaTypes))
- *ppv = iface;
-
- if (*ppv)
+ if (IsEqualIID(riid, &IID_IUnknown) ||
+ IsEqualIID(riid, &IID_IEnumMediaTypes))
{
- IUnknown_AddRef((IUnknown *)(*ppv));
+ IEnumMediaTypes_AddRef(iface);
+ *ret_iface = iface;
return S_OK;
}
- FIXME("No interface for %s!\n", debugstr_guid(riid));
+ *ret_iface = NULL;
+
+ WARN("No interface for %s\n", debugstr_guid(riid));
return E_NOINTERFACE;
}
More information about the wine-cvs
mailing list