[03/18] windowscodecs: Do not assume that vtable is the first element of the object, avoid not necessary casts.
Dmitry Timoshkov
dmitry at baikal.ru
Tue Jul 24 01:03:20 CDT 2012
---
dlls/windowscodecs/clsfactory.c | 9 +++++----
1 file changed, 5 insertions(+), 4 deletions(-)
diff --git a/dlls/windowscodecs/clsfactory.c b/dlls/windowscodecs/clsfactory.c
index 40c8df3..6a81a9d 100644
--- a/dlls/windowscodecs/clsfactory.c
+++ b/dlls/windowscodecs/clsfactory.c
@@ -82,9 +82,10 @@ static HRESULT WINAPI ClassFactoryImpl_QueryInterface(IClassFactory *iface,
if (!ppv) return E_INVALIDARG;
- if (IsEqualIID(&IID_IUnknown, iid) || IsEqualIID(&IID_IClassFactory, iid))
+ if (IsEqualIID(&IID_IUnknown, iid) ||
+ IsEqualIID(&IID_IClassFactory, iid))
{
- *ppv = This;
+ *ppv = &This->IClassFactory_iface;
}
else
{
@@ -155,8 +156,8 @@ static HRESULT ClassFactoryImpl_Constructor(classinfo *info, REFIID riid, LPVOID
This->ref = 1;
This->info = info;
- ret = IClassFactory_QueryInterface((IClassFactory*)This, riid, ppv);
- IClassFactory_Release((IClassFactory*)This);
+ ret = IClassFactory_QueryInterface(&This->IClassFactory_iface, riid, ppv);
+ IClassFactory_Release(&This->IClassFactory_iface);
return ret;
}
--
1.7.11.2
More information about the wine-patches
mailing list