December 2010 - Wine-commits

Michael Stefaniuc : msxml3: Use an iface instead of a vtbl pointer in parse_error_t.
by Alexandre Julliard 29 Dec '10

29 Dec '10

Module: wine Branch: master Commit: e53582fdc11226b74c96c796ee1b18ebf9f9ff34 URL: http://source.winehq.org/git/wine.git/?a=commit;h=e53582fdc11226b74c96c796e… Author: Michael Stefaniuc <mstefani(a)redhat.de> Date: Wed Dec 29 02:52:22 2010 +0100 msxml3: Use an iface instead of a vtbl pointer in parse_error_t. --- dlls/msxml3/parseerror.c | 12 ++++++------ 1 files changed, 6 insertions(+), 6 deletions(-) diff --git a/dlls/msxml3/parseerror.c b/dlls/msxml3/parseerror.c index d5e4cde..2869907 100644 --- a/dlls/msxml3/parseerror.c +++ b/dlls/msxml3/parseerror.c @@ -39,7 +39,7 @@ WINE_DEFAULT_DEBUG_CHANNEL(msxml); typedef struct { - const struct IXMLDOMParseErrorVtbl *lpVtbl; + IXMLDOMParseError IXMLDOMParseError_iface; LONG ref; LONG code, line, linepos, filepos; BSTR url, reason, srcText; @@ -47,7 +47,7 @@ typedef struct static inline parse_error_t *impl_from_IXMLDOMParseError( IXMLDOMParseError *iface ) { - return (parse_error_t *)((char*)iface - FIELD_OFFSET(parse_error_t, lpVtbl)); + return CONTAINING_RECORD(iface, parse_error_t, IXMLDOMParseError_iface); } static HRESULT WINAPI parseError_QueryInterface( @@ -180,8 +180,8 @@ static HRESULT WINAPI parseError_Invoke( hr = get_typeinfo(IXMLDOMParseError_tid, &typeinfo); if(SUCCEEDED(hr)) { - hr = ITypeInfo_Invoke(typeinfo, &(This->lpVtbl), dispIdMember, wFlags, pDispParams, - pVarResult, pExcepInfo, puArgErr); + hr = ITypeInfo_Invoke(typeinfo, &This->IXMLDOMParseError_iface, dispIdMember, wFlags, + pDispParams, pVarResult, pExcepInfo, puArgErr); ITypeInfo_Release(typeinfo); } @@ -291,7 +291,7 @@ IXMLDOMParseError *create_parseError( LONG code, BSTR url, BSTR reason, BSTR src if ( !This ) return NULL; - This->lpVtbl = &parseError_vtbl; + This->IXMLDOMParseError_iface.lpVtbl = &parseError_vtbl; This->ref = 1; This->code = code; @@ -302,5 +302,5 @@ IXMLDOMParseError *create_parseError( LONG code, BSTR url, BSTR reason, BSTR src This->linepos = linepos; This->filepos = filepos; - return (IXMLDOMParseError*) &This->lpVtbl; + return &This->IXMLDOMParseError_iface; }

1 0

Michael Stefaniuc : explorerframe/tests: COM cleanup for INameSpaceTreeControlEventsImpl.
by Alexandre Julliard 29 Dec '10

29 Dec '10

Module: wine Branch: master Commit: d15f5722cbca155b02dd35834d9343b3055cb87b URL: http://source.winehq.org/git/wine.git/?a=commit;h=d15f5722cbca155b02dd35834… Author: Michael Stefaniuc <mstefani(a)redhat.de> Date: Wed Dec 29 02:53:01 2010 +0100 explorerframe/tests: COM cleanup for INameSpaceTreeControlEventsImpl. --- dlls/explorerframe/tests/nstc.c | 103 +++++++++++++++++++++++++++----------- 1 files changed, 73 insertions(+), 30 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=d15f5722cbca155b02dd3…

1 0

Michael Stefaniuc : xmllite/tests: Use an iface instead of a vtbl pointer in testinput.
by Alexandre Julliard 29 Dec '10

29 Dec '10

Module: wine Branch: master Commit: a36f51f35718239a2f6ab4bc5d81e1573ef0b14b URL: http://source.winehq.org/git/wine.git/?a=commit;h=a36f51f35718239a2f6ab4bc5… Author: Michael Stefaniuc <mstefani(a)redhat.de> Date: Wed Dec 29 02:53:57 2010 +0100 xmllite/tests: Use an iface instead of a vtbl pointer in testinput. --- dlls/xmllite/tests/reader.c | 24 +++++++++++++----------- 1 files changed, 13 insertions(+), 11 deletions(-) diff --git a/dlls/xmllite/tests/reader.c b/dlls/xmllite/tests/reader.c index 316f8f7..bd6ac1a 100644 --- a/dlls/xmllite/tests/reader.c +++ b/dlls/xmllite/tests/reader.c @@ -19,6 +19,7 @@ */ #define COBJMACROS +#define CONST_VTABLE #include <stdarg.h> #include <stdio.h> @@ -258,13 +259,13 @@ static void test_read_state_(IXmlReader *reader, XmlReadState expected, typedef struct _testinput { - const IUnknownVtbl *lpVtbl; + IUnknown IUnknown_iface; LONG ref; } testinput; static inline testinput *impl_from_IUnknown(IUnknown *iface) { - return (testinput *)((char*)iface - FIELD_OFFSET(testinput, lpVtbl)); + return CONTAINING_RECORD(iface, testinput, IUnknown_iface); } static HRESULT WINAPI testinput_QueryInterface(IUnknown *iface, REFIID riid, void** ppvObj) @@ -317,10 +318,10 @@ static HRESULT testinput_createinstance(void **ppObj) input = HeapAlloc(GetProcessHeap(), 0, sizeof (*input)); if(!input) return E_OUTOFMEMORY; - input->lpVtbl = &testinput_vtbl; + input->IUnknown_iface.lpVtbl = &testinput_vtbl; input->ref = 1; - *ppObj = &input->lpVtbl; + *ppObj = &input->IUnknown_iface; return S_OK; } @@ -372,13 +373,14 @@ static void test_reader_create(void) hr = testinput_createinstance((void**)&input); ok(hr == S_OK, "Expected S_OK, got %08x\n", hr); - input_iids.count = 0; - hr = IXmlReader_SetInput(reader, input); - ok(hr == E_NOINTERFACE, "Expected E_NOINTERFACE, got %08x\n", hr); - ok_iids(&input_iids, setinput_full, setinput_full_old, FALSE); - - IUnknown_Release(input); - + if (hr == S_OK) + { + input_iids.count = 0; + hr = IXmlReader_SetInput(reader, input); + ok(hr == E_NOINTERFACE, "Expected E_NOINTERFACE, got %08x\n", hr); + ok_iids(&input_iids, setinput_full, setinput_full_old, FALSE); + IUnknown_Release(input); + } IXmlReader_Release(reader); }

1 0

Michael Stefaniuc : wmiutils: Use an iface instead of a vtbl pointer in wmiutils_cf.
by Alexandre Julliard 29 Dec '10

29 Dec '10

Module: wine Branch: master Commit: 957cfa68dabba3fb5ff91a9204425db55f402026 URL: http://source.winehq.org/git/wine.git/?a=commit;h=957cfa68dabba3fb5ff91a920… Author: Michael Stefaniuc <mstefani(a)redhat.de> Date: Wed Dec 29 02:53:49 2010 +0100 wmiutils: Use an iface instead of a vtbl pointer in wmiutils_cf. --- dlls/wmiutils/main.c | 8 ++++---- 1 files changed, 4 insertions(+), 4 deletions(-) diff --git a/dlls/wmiutils/main.c b/dlls/wmiutils/main.c index bd1d9f5..4f761bb 100644 --- a/dlls/wmiutils/main.c +++ b/dlls/wmiutils/main.c @@ -39,13 +39,13 @@ typedef HRESULT (*fnCreateInstance)( IUnknown *pUnkOuter, LPVOID *ppObj ); typedef struct { - const struct IClassFactoryVtbl *vtbl; + IClassFactory IClassFactory_iface; fnCreateInstance pfnCreateInstance; } wmiutils_cf; static inline wmiutils_cf *impl_from_IClassFactory( IClassFactory *iface ) { - return (wmiutils_cf *)((char *)iface - FIELD_OFFSET( wmiutils_cf, vtbl )); + return CONTAINING_RECORD(iface, wmiutils_cf, IClassFactory_iface); } static HRESULT WINAPI wmiutils_cf_QueryInterface( IClassFactory *iface, REFIID riid, LPVOID *ppobj ) @@ -112,7 +112,7 @@ static const struct IClassFactoryVtbl wmiutils_cf_vtbl = wmiutils_cf_LockServer }; -static wmiutils_cf status_code_cf = { &wmiutils_cf_vtbl, WbemStatusCodeText_create }; +static wmiutils_cf status_code_cf = { { &wmiutils_cf_vtbl }, WbemStatusCodeText_create }; BOOL WINAPI DllMain( HINSTANCE hinst, DWORD reason, LPVOID lpv ) { @@ -138,7 +138,7 @@ HRESULT WINAPI DllGetClassObject( REFCLSID rclsid, REFIID iid, LPVOID *ppv ) if (IsEqualGUID( rclsid, &CLSID_WbemStatusCode )) { - cf = (IClassFactory *)&status_code_cf.vtbl; + cf = &status_code_cf.IClassFactory_iface; } if (!cf) return CLASS_E_CLASSNOTAVAILABLE; return IClassFactory_QueryInterface( cf, iid, ppv );

1 0

Michael Stefaniuc : crypt32: Don't open code CONTAINING_RECORD().
by Alexandre Julliard 29 Dec '10

29 Dec '10

Module: wine Branch: master Commit: ef39e0ec1af20e1753f6833d418714dda9053728 URL: http://source.winehq.org/git/wine.git/?a=commit;h=ef39e0ec1af20e1753f6833d4… Author: Michael Stefaniuc <mstefani(a)redhat.de> Date: Wed Dec 29 02:39:42 2010 +0100 crypt32: Don't open code CONTAINING_RECORD(). --- dlls/crypt32/decode.c | 14 ++++++-------- 1 files changed, 6 insertions(+), 8 deletions(-) diff --git a/dlls/crypt32/decode.c b/dlls/crypt32/decode.c index 62231cd..a1431fb 100644 --- a/dlls/crypt32/decode.c +++ b/dlls/crypt32/decode.c @@ -3731,9 +3731,8 @@ static BOOL CRYPT_AsnDecodeRequireExplicit(const BYTE *pbEncoded, } else { - CERT_POLICY_CONSTRAINTS_INFO *info = - (CERT_POLICY_CONSTRAINTS_INFO *)((BYTE *)pvStructInfo - - offsetof(CERT_POLICY_CONSTRAINTS_INFO, fRequireExplicitPolicy)); + CERT_POLICY_CONSTRAINTS_INFO *info = CONTAINING_RECORD(pvStructInfo, + CERT_POLICY_CONSTRAINTS_INFO, fRequireExplicitPolicy); *pcbStructInfo = bytesNeeded; /* The BOOL is implicit: if the integer is present, then it's @@ -3779,9 +3778,8 @@ static BOOL CRYPT_AsnDecodeInhibitMapping(const BYTE *pbEncoded, } else { - CERT_POLICY_CONSTRAINTS_INFO *info = - (CERT_POLICY_CONSTRAINTS_INFO *)((BYTE *)pvStructInfo - - offsetof(CERT_POLICY_CONSTRAINTS_INFO, fInhibitPolicyMapping)); + CERT_POLICY_CONSTRAINTS_INFO *info = CONTAINING_RECORD(pvStructInfo, + CERT_POLICY_CONSTRAINTS_INFO, fInhibitPolicyMapping); *pcbStructInfo = bytesNeeded; /* The BOOL is implicit: if the integer is present, then it's @@ -5107,8 +5105,8 @@ static BOOL CRYPT_AsnDecodeMaximum(const BYTE *pbEncoded, } else { - CERT_GENERAL_SUBTREE *subtree = (CERT_GENERAL_SUBTREE *) - ((BYTE *)pvStructInfo - offsetof(CERT_GENERAL_SUBTREE, fMaximum)); + CERT_GENERAL_SUBTREE *subtree = CONTAINING_RECORD(pvStructInfo, + CERT_GENERAL_SUBTREE, fMaximum); *pcbStructInfo = bytesNeeded; /* The BOOL is implicit: if the integer is present, then it's

1 0

Michael Stefaniuc : mapi32: Fully merge the IMAPIProp ifaces into IPropData .
by Alexandre Julliard 29 Dec '10

29 Dec '10

Module: wine Branch: master Commit: e67e836ef9615c4afb75a5f9976887b65b9b0d45 URL: http://source.winehq.org/git/wine.git/?a=commit;h=e67e836ef9615c4afb75a5f99… Author: Michael Stefaniuc <mstefani(a)redhat.de> Date: Wed Dec 29 02:43:13 2010 +0100 mapi32: Fully merge the IMAPIProp ifaces into IPropData. --- dlls/mapi32/prop.c | 326 +++++++++------------------------------------------- 1 files changed, 54 insertions(+), 272 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=e67e836ef9615c4afb75a…

1 0

Michael Stefaniuc : mapi32: COM cleanup for the IPropData iface.
by Alexandre Julliard 29 Dec '10

29 Dec '10

Module: wine Branch: master Commit: abcd84aa0ed3130b3e0319477353d9cc55fb56c0 URL: http://source.winehq.org/git/wine.git/?a=commit;h=abcd84aa0ed3130b3e0319477… Author: Michael Stefaniuc <mstefani(a)redhat.de> Date: Wed Dec 29 02:44:19 2010 +0100 mapi32: COM cleanup for the IPropData iface. --- dlls/mapi32/prop.c | 32 ++++++++++++++++++-------------- 1 files changed, 18 insertions(+), 14 deletions(-) diff --git a/dlls/mapi32/prop.c b/dlls/mapi32/prop.c index 7123eca..716c528 100644 --- a/dlls/mapi32/prop.c +++ b/dlls/mapi32/prop.c @@ -1401,7 +1401,7 @@ typedef struct /* The main property data collection structure */ typedef struct { - const IPropDataVtbl *lpVtbl; + IPropData IPropData_iface; LONG lRef; /* Reference count */ ALLOCATEBUFFER *lpAlloc; /* Memory allocation routine */ ALLOCATEMORE *lpMore; /* Linked memory allocation routine */ @@ -1412,6 +1412,11 @@ typedef struct CRITICAL_SECTION cs; /* Lock for thread safety */ } IPropDataImpl; +static inline IPropDataImpl *impl_from_IPropData(IPropData *iface) +{ + return CONTAINING_RECORD(iface, IPropDataImpl, IPropData_iface); +} + /* Internal - Get a property value, assumes lock is held */ static IPropDataItem *IMAPIPROP_GetValue(IPropDataImpl *This, ULONG ulPropTag) { @@ -1487,7 +1492,7 @@ static inline void IMAPIPROP_Unlock(IPropDataImpl *This) */ static WINAPI HRESULT IPropData_fnQueryInterface(LPPROPDATA iface, REFIID riid, LPVOID *ppvObj) { - IPropDataImpl *This = (IPropDataImpl*)iface; + IPropDataImpl *This = impl_from_IPropData(iface); TRACE("(%p,%s,%p)\n", This, debugstr_guid(riid), ppvObj); @@ -1518,7 +1523,7 @@ static WINAPI HRESULT IPropData_fnQueryInterface(LPPROPDATA iface, REFIID riid, */ static ULONG WINAPI IPropData_fnAddRef(LPPROPDATA iface) { - IPropDataImpl *This = (IPropDataImpl*)iface; + IPropDataImpl *This = impl_from_IPropData(iface); TRACE("(%p)->(count before=%u)\n", This, This->lRef); @@ -1533,7 +1538,7 @@ static ULONG WINAPI IPropData_fnAddRef(LPPROPDATA iface) */ static ULONG WINAPI IPropData_fnRelease(LPPROPDATA iface) { - IPropDataImpl *This = (IPropDataImpl*)iface; + IPropDataImpl *This = impl_from_IPropData(iface); LONG lRef; TRACE("(%p)->(count before=%u)\n", This, This->lRef); @@ -1642,9 +1647,9 @@ static HRESULT WINAPI IPropData_fnSaveChanges(LPPROPDATA iface, ULONG ulFlags) static HRESULT WINAPI IPropData_fnGetProps(LPPROPDATA iface, LPSPropTagArray lpTags, ULONG ulFlags, ULONG *lpCount, LPSPropValue *lppProps) { + IPropDataImpl *This = impl_from_IPropData(iface); ULONG i; HRESULT hRet = S_OK; - IPropDataImpl *This = (IPropDataImpl*)iface; TRACE("(%p,%p,0x%08x,%p,%p) stub\n", iface, lpTags, ulFlags, lpCount, lppProps); @@ -1708,7 +1713,7 @@ static HRESULT WINAPI IPropData_fnGetProps(LPPROPDATA iface, LPSPropTagArray lpT static HRESULT WINAPI IPropData_fnGetPropList(LPPROPDATA iface, ULONG ulFlags, LPSPropTagArray *lppTags) { - IPropDataImpl *This = (IPropDataImpl*)iface; + IPropDataImpl *This = impl_from_IPropData(iface); ULONG i; HRESULT hRet; @@ -1779,7 +1784,7 @@ static HRESULT WINAPI IPropData_fnOpenProperty(LPPROPDATA iface, ULONG ulPropTag static HRESULT WINAPI IPropData_fnSetProps(LPPROPDATA iface, ULONG ulValues, LPSPropValue lpProps, LPSPropProblemArray *lppProbs) { - IPropDataImpl *This = (IPropDataImpl*)iface; + IPropDataImpl *This = impl_from_IPropData(iface); HRESULT hRet = S_OK; ULONG i; @@ -1865,7 +1870,7 @@ static HRESULT WINAPI IPropData_fnSetProps(LPPROPDATA iface, ULONG ulValues, LPS static HRESULT WINAPI IPropData_fnDeleteProps(LPPROPDATA iface, LPSPropTagArray lpTags, LPSPropProblemArray *lppProbs) { - IPropDataImpl *This = (IPropDataImpl*)iface; + IPropDataImpl *This = impl_from_IPropData(iface); ULONG i, numProbs = 0; HRESULT hRet = S_OK; @@ -2055,7 +2060,7 @@ static HRESULT WINAPI IPropData_fnGetIDsFromNames(LPPROPDATA iface, ULONG ulName static HRESULT WINAPI IPropData_fnHrSetObjAccess(LPPROPDATA iface, ULONG ulAccess) { - IPropDataImpl *This = (IPropDataImpl*)iface; + IPropDataImpl *This = impl_from_IPropData(iface); TRACE("(%p,%x)\n", iface, ulAccess); @@ -2109,8 +2114,7 @@ static HRESULT WINAPI IPropData_fnHrSetPropAccess(LPPROPDATA iface, LPSPropTagArray lpTags, ULONG *lpAccess) { - IPropDataImpl *This = (IPropDataImpl*)iface; - + IPropDataImpl *This = impl_from_IPropData(iface); ULONG i; TRACE("(%p,%p,%p)\n", iface, lpTags, lpAccess); @@ -2161,7 +2165,7 @@ static HRESULT WINAPI IPropData_fnHrGetPropAccess(LPPROPDATA iface, LPSPropTagArray *lppTags, ULONG **lppAccess) { - IPropDataImpl *This = (IPropDataImpl*)iface; + IPropDataImpl *This = impl_from_IPropData(iface); LPVOID lpMem; HRESULT hRet; ULONG i; @@ -2316,7 +2320,7 @@ SCODE WINAPI CreateIProp(LPCIID iid, ALLOCATEBUFFER *lpAlloc, if (SUCCEEDED(scode)) { - lpPropData->lpVtbl = &IPropDataImpl_vtbl; + lpPropData->IPropData_iface.lpVtbl = &IPropDataImpl_vtbl; lpPropData->lRef = 1; lpPropData->lpAlloc = lpAlloc; lpPropData->lpMore = lpMore; @@ -2326,7 +2330,7 @@ SCODE WINAPI CreateIProp(LPCIID iid, ALLOCATEBUFFER *lpAlloc, list_init(&lpPropData->values); InitializeCriticalSection(&lpPropData->cs); lpPropData->cs.DebugInfo->Spare[0] = (DWORD_PTR)(__FILE__ ": IPropDataImpl.cs"); - *lppPropData = (LPPROPDATA)lpPropData; + *lppPropData = &lpPropData->IPropData_iface; } return scode; }

1 0

Michael Stefaniuc : hlink: Use ifaces instead of vtbl pointers in HlinkImpl .
by Alexandre Julliard 29 Dec '10

29 Dec '10

Module: wine Branch: master Commit: 5101f249c8c39dbb18d467ddd6d6f2767fa0a9e5 URL: http://source.winehq.org/git/wine.git/?a=commit;h=5101f249c8c39dbb18d467ddd… Author: Michael Stefaniuc <mstefani(a)redhat.de> Date: Wed Dec 29 02:48:17 2010 +0100 hlink: Use ifaces instead of vtbl pointers in HlinkImpl. --- dlls/hlink/link.c | 89 ++++++++++++++++++++++++++++------------------------- 1 files changed, 47 insertions(+), 42 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=5101f249c8c39dbb18d46…

1 0

Michael Stefaniuc : oledb32: Use ifaces instead of vtbl pointers in convert .
by Alexandre Julliard 29 Dec '10

29 Dec '10

Module: wine Branch: master Commit: 7af2899aeb6b4f8b4bdd8317a73cda6a69f9318e URL: http://source.winehq.org/git/wine.git/?a=commit;h=7af2899aeb6b4f8b4bdd8317a… Author: Michael Stefaniuc <mstefani(a)redhat.de> Date: Wed Dec 29 02:49:43 2010 +0100 oledb32: Use ifaces instead of vtbl pointers in convert. --- dlls/oledb32/convert.c | 22 +++++++++++----------- 1 files changed, 11 insertions(+), 11 deletions(-) diff --git a/dlls/oledb32/convert.c b/dlls/oledb32/convert.c index c98d0b2..203061b 100644 --- a/dlls/oledb32/convert.c +++ b/dlls/oledb32/convert.c @@ -38,8 +38,8 @@ WINE_DEFAULT_DEBUG_CHANNEL(oledb); typedef struct { - const struct IDataConvertVtbl *lpVtbl; - const struct IDCInfoVtbl *lpDCInfoVtbl; + IDataConvert IDataConvert_iface; + IDCInfo IDCInfo_iface; LONG ref; @@ -48,12 +48,12 @@ typedef struct static inline convert *impl_from_IDataConvert(IDataConvert *iface) { - return (convert *)((char*)iface - FIELD_OFFSET(convert, lpVtbl)); + return CONTAINING_RECORD(iface, convert, IDataConvert_iface); } static inline convert *impl_from_IDCInfo(IDCInfo *iface) { - return (convert *)((char*)iface - FIELD_OFFSET(convert, lpDCInfoVtbl)); + return CONTAINING_RECORD(iface, convert, IDCInfo_iface); } static HRESULT WINAPI convert_QueryInterface(IDataConvert* iface, @@ -72,7 +72,7 @@ static HRESULT WINAPI convert_QueryInterface(IDataConvert* iface, } else if(IsEqualIID(riid, &IID_IDCInfo)) { - *obj = &This->lpDCInfoVtbl; + *obj = &This->IDCInfo_iface; } else { @@ -985,21 +985,21 @@ static HRESULT WINAPI dcinfo_QueryInterface(IDCInfo* iface, REFIID riid, void ** { convert *This = impl_from_IDCInfo(iface); - return IDataConvert_QueryInterface((IDataConvert *)This, riid, obj); + return IDataConvert_QueryInterface(&This->IDataConvert_iface, riid, obj); } static ULONG WINAPI dcinfo_AddRef(IDCInfo* iface) { convert *This = impl_from_IDCInfo(iface); - return IDataConvert_AddRef((IDataConvert *)This); + return IDataConvert_AddRef(&This->IDataConvert_iface); } static ULONG WINAPI dcinfo_Release(IDCInfo* iface) { convert *This = impl_from_IDCInfo(iface); - return IDataConvert_Release((IDataConvert *)This); + return IDataConvert_Release(&This->IDataConvert_iface); } static HRESULT WINAPI dcinfo_GetInfo(IDCInfo *iface, ULONG num, DCINFOTYPE types[], DCINFO **info_ptr) @@ -1081,12 +1081,12 @@ HRESULT create_oledb_convert(IUnknown *outer, void **obj) This = HeapAlloc(GetProcessHeap(), 0, sizeof(*This)); if(!This) return E_OUTOFMEMORY; - This->lpVtbl = &convert_vtbl; - This->lpDCInfoVtbl = &dcinfo_vtbl; + This->IDataConvert_iface.lpVtbl = &convert_vtbl; + This->IDCInfo_iface.lpVtbl = &dcinfo_vtbl; This->ref = 1; This->version = 0x110; - *obj = &This->lpVtbl; + *obj = &This->IDataConvert_iface; return S_OK; }

1 0

Michael Stefaniuc : msxml3: Use an iface instead of a vtbl pointer in httprequest.
by Alexandre Julliard 29 Dec '10

29 Dec '10

Module: wine Branch: master Commit: 2a2bc6a98d6d5c5faaa2a55e1405f8ca9123bd0b URL: http://source.winehq.org/git/wine.git/?a=commit;h=2a2bc6a98d6d5c5faaa2a55e1… Author: Michael Stefaniuc <mstefani(a)redhat.de> Date: Wed Dec 29 02:52:14 2010 +0100 msxml3: Use an iface instead of a vtbl pointer in httprequest. --- dlls/msxml3/httprequest.c | 12 ++++++------ 1 files changed, 6 insertions(+), 6 deletions(-) diff --git a/dlls/msxml3/httprequest.c b/dlls/msxml3/httprequest.c index 9e8cf69..23bc227 100644 --- a/dlls/msxml3/httprequest.c +++ b/dlls/msxml3/httprequest.c @@ -60,7 +60,7 @@ struct reqheader typedef struct { - const struct IXMLHTTPRequestVtbl *lpVtbl; + IXMLHTTPRequest IXMLHTTPRequest_iface; LONG ref; READYSTATE state; @@ -85,7 +85,7 @@ typedef struct static inline httprequest *impl_from_IXMLHTTPRequest( IXMLHTTPRequest *iface ) { - return (httprequest *)((char*)iface - FIELD_OFFSET(httprequest, lpVtbl)); + return CONTAINING_RECORD(iface, httprequest, IXMLHTTPRequest_iface); } static void httprequest_setreadystate(httprequest *This, READYSTATE state) @@ -617,8 +617,8 @@ static HRESULT WINAPI httprequest_Invoke(IXMLHTTPRequest *iface, DISPID dispIdMe hr = get_typeinfo(IXMLHTTPRequest_tid, &typeinfo); if(SUCCEEDED(hr)) { - hr = ITypeInfo_Invoke(typeinfo, &(This->lpVtbl), dispIdMember, wFlags, pDispParams, - pVarResult, pExcepInfo, puArgErr); + hr = ITypeInfo_Invoke(typeinfo, &This->IXMLHTTPRequest_iface, dispIdMember, wFlags, + pDispParams, pVarResult, pExcepInfo, puArgErr); ITypeInfo_Release(typeinfo); } @@ -1001,7 +1001,7 @@ HRESULT XMLHTTPRequest_create(IUnknown *pUnkOuter, void **ppObj) if( !req ) return E_OUTOFMEMORY; - req->lpVtbl = &dimimpl_vtbl; + req->IXMLHTTPRequest_iface.lpVtbl = &dimimpl_vtbl; req->ref = 1; req->async = FALSE; @@ -1016,7 +1016,7 @@ HRESULT XMLHTTPRequest_create(IUnknown *pUnkOuter, void **ppObj) req->reqheader_size = 0; list_init(&req->reqheaders); - *ppObj = &req->lpVtbl; + *ppObj = &req->IXMLHTTPRequest_iface; TRACE("returning iface %p\n", *ppObj);

1 0