urlmon: Use an iface instead of a vtbl pointer in AsyncBindCtx.

This commit is contained in:
Michael Stefaniuc 2011-01-11 00:41:11 +01:00 committed by Alexandre Julliard
parent 2f7da964ac
commit 346e766d53
1 changed files with 25 additions and 26 deletions

View File

@ -616,32 +616,33 @@ HRESULT WINAPI RevokeBindStatusCallback(IBindCtx *pbc, IBindStatusCallback *pbsc
} }
typedef struct { typedef struct {
const IBindCtxVtbl *lpBindCtxVtbl; IBindCtx IBindCtx_iface;
LONG ref; LONG ref;
IBindCtx *bindctx; IBindCtx *bindctx;
} AsyncBindCtx; } AsyncBindCtx;
#define BINDCTX(x) ((IBindCtx*) &(x)->lpBindCtxVtbl) static inline AsyncBindCtx *impl_from_IBindCtx(IBindCtx *iface)
{
#define BINDCTX_THIS(iface) DEFINE_THIS(AsyncBindCtx, BindCtx, iface) return CONTAINING_RECORD(iface, AsyncBindCtx, IBindCtx_iface);
}
static HRESULT WINAPI AsyncBindCtx_QueryInterface(IBindCtx *iface, REFIID riid, void **ppv) static HRESULT WINAPI AsyncBindCtx_QueryInterface(IBindCtx *iface, REFIID riid, void **ppv)
{ {
AsyncBindCtx *This = BINDCTX_THIS(iface); AsyncBindCtx *This = impl_from_IBindCtx(iface);
*ppv = NULL; *ppv = NULL;
if(IsEqualGUID(riid, &IID_IUnknown)) { if(IsEqualGUID(riid, &IID_IUnknown)) {
TRACE("(%p)->(IID_IUnknown %p)\n", This, ppv); TRACE("(%p)->(IID_IUnknown %p)\n", This, ppv);
*ppv = BINDCTX(This); *ppv = &This->IBindCtx_iface;
}else if(IsEqualGUID(riid, &IID_IBindCtx)) { }else if(IsEqualGUID(riid, &IID_IBindCtx)) {
TRACE("(%p)->(IID_IBindCtx %p)\n", This, ppv); TRACE("(%p)->(IID_IBindCtx %p)\n", This, ppv);
*ppv = BINDCTX(This); *ppv = &This->IBindCtx_iface;
}else if(IsEqualGUID(riid, &IID_IAsyncBindCtx)) { }else if(IsEqualGUID(riid, &IID_IAsyncBindCtx)) {
TRACE("(%p)->(IID_IAsyncBindCtx %p)\n", This, ppv); TRACE("(%p)->(IID_IAsyncBindCtx %p)\n", This, ppv);
*ppv = BINDCTX(This); *ppv = &This->IBindCtx_iface;
} }
if(*ppv) { if(*ppv) {
@ -655,7 +656,7 @@ static HRESULT WINAPI AsyncBindCtx_QueryInterface(IBindCtx *iface, REFIID riid,
static ULONG WINAPI AsyncBindCtx_AddRef(IBindCtx *iface) static ULONG WINAPI AsyncBindCtx_AddRef(IBindCtx *iface)
{ {
AsyncBindCtx *This = BINDCTX_THIS(iface); AsyncBindCtx *This = impl_from_IBindCtx(iface);
LONG ref = InterlockedIncrement(&This->ref); LONG ref = InterlockedIncrement(&This->ref);
TRACE("(%p) ref=%d\n", This, ref); TRACE("(%p) ref=%d\n", This, ref);
@ -665,7 +666,7 @@ static ULONG WINAPI AsyncBindCtx_AddRef(IBindCtx *iface)
static ULONG WINAPI AsyncBindCtx_Release(IBindCtx *iface) static ULONG WINAPI AsyncBindCtx_Release(IBindCtx *iface)
{ {
AsyncBindCtx *This = BINDCTX_THIS(iface); AsyncBindCtx *This = impl_from_IBindCtx(iface);
LONG ref = InterlockedDecrement(&This->ref); LONG ref = InterlockedDecrement(&This->ref);
TRACE("(%p) ref=%d\n", This, ref); TRACE("(%p) ref=%d\n", This, ref);
@ -680,7 +681,7 @@ static ULONG WINAPI AsyncBindCtx_Release(IBindCtx *iface)
static HRESULT WINAPI AsyncBindCtx_RegisterObjectBound(IBindCtx *iface, IUnknown *punk) static HRESULT WINAPI AsyncBindCtx_RegisterObjectBound(IBindCtx *iface, IUnknown *punk)
{ {
AsyncBindCtx *This = BINDCTX_THIS(iface); AsyncBindCtx *This = impl_from_IBindCtx(iface);
TRACE("(%p)->(%p)\n", This, punk); TRACE("(%p)->(%p)\n", This, punk);
@ -689,7 +690,7 @@ static HRESULT WINAPI AsyncBindCtx_RegisterObjectBound(IBindCtx *iface, IUnknown
static HRESULT WINAPI AsyncBindCtx_RevokeObjectBound(IBindCtx *iface, IUnknown *punk) static HRESULT WINAPI AsyncBindCtx_RevokeObjectBound(IBindCtx *iface, IUnknown *punk)
{ {
AsyncBindCtx *This = BINDCTX_THIS(iface); AsyncBindCtx *This = impl_from_IBindCtx(iface);
TRACE("(%p %p)\n", This, punk); TRACE("(%p %p)\n", This, punk);
@ -698,7 +699,7 @@ static HRESULT WINAPI AsyncBindCtx_RevokeObjectBound(IBindCtx *iface, IUnknown *
static HRESULT WINAPI AsyncBindCtx_ReleaseBoundObjects(IBindCtx *iface) static HRESULT WINAPI AsyncBindCtx_ReleaseBoundObjects(IBindCtx *iface)
{ {
AsyncBindCtx *This = BINDCTX_THIS(iface); AsyncBindCtx *This = impl_from_IBindCtx(iface);
TRACE("(%p)\n", This); TRACE("(%p)\n", This);
@ -707,7 +708,7 @@ static HRESULT WINAPI AsyncBindCtx_ReleaseBoundObjects(IBindCtx *iface)
static HRESULT WINAPI AsyncBindCtx_SetBindOptions(IBindCtx *iface, BIND_OPTS *pbindopts) static HRESULT WINAPI AsyncBindCtx_SetBindOptions(IBindCtx *iface, BIND_OPTS *pbindopts)
{ {
AsyncBindCtx *This = BINDCTX_THIS(iface); AsyncBindCtx *This = impl_from_IBindCtx(iface);
TRACE("(%p)->(%p)\n", This, pbindopts); TRACE("(%p)->(%p)\n", This, pbindopts);
@ -716,7 +717,7 @@ static HRESULT WINAPI AsyncBindCtx_SetBindOptions(IBindCtx *iface, BIND_OPTS *pb
static HRESULT WINAPI AsyncBindCtx_GetBindOptions(IBindCtx *iface, BIND_OPTS *pbindopts) static HRESULT WINAPI AsyncBindCtx_GetBindOptions(IBindCtx *iface, BIND_OPTS *pbindopts)
{ {
AsyncBindCtx *This = BINDCTX_THIS(iface); AsyncBindCtx *This = impl_from_IBindCtx(iface);
TRACE("(%p)->(%p)\n", This, pbindopts); TRACE("(%p)->(%p)\n", This, pbindopts);
@ -725,7 +726,7 @@ static HRESULT WINAPI AsyncBindCtx_GetBindOptions(IBindCtx *iface, BIND_OPTS *pb
static HRESULT WINAPI AsyncBindCtx_GetRunningObjectTable(IBindCtx *iface, IRunningObjectTable **pprot) static HRESULT WINAPI AsyncBindCtx_GetRunningObjectTable(IBindCtx *iface, IRunningObjectTable **pprot)
{ {
AsyncBindCtx *This = BINDCTX_THIS(iface); AsyncBindCtx *This = impl_from_IBindCtx(iface);
TRACE("(%p)->(%p)\n", This, pprot); TRACE("(%p)->(%p)\n", This, pprot);
@ -734,7 +735,7 @@ static HRESULT WINAPI AsyncBindCtx_GetRunningObjectTable(IBindCtx *iface, IRunni
static HRESULT WINAPI AsyncBindCtx_RegisterObjectParam(IBindCtx *iface, LPOLESTR pszkey, IUnknown *punk) static HRESULT WINAPI AsyncBindCtx_RegisterObjectParam(IBindCtx *iface, LPOLESTR pszkey, IUnknown *punk)
{ {
AsyncBindCtx *This = BINDCTX_THIS(iface); AsyncBindCtx *This = impl_from_IBindCtx(iface);
TRACE("(%p)->(%s %p)\n", This, debugstr_w(pszkey), punk); TRACE("(%p)->(%s %p)\n", This, debugstr_w(pszkey), punk);
@ -743,7 +744,7 @@ static HRESULT WINAPI AsyncBindCtx_RegisterObjectParam(IBindCtx *iface, LPOLESTR
static HRESULT WINAPI AsyncBindCtx_GetObjectParam(IBindCtx* iface, LPOLESTR pszkey, IUnknown **punk) static HRESULT WINAPI AsyncBindCtx_GetObjectParam(IBindCtx* iface, LPOLESTR pszkey, IUnknown **punk)
{ {
AsyncBindCtx *This = BINDCTX_THIS(iface); AsyncBindCtx *This = impl_from_IBindCtx(iface);
TRACE("(%p)->(%s %p)\n", This, debugstr_w(pszkey), punk); TRACE("(%p)->(%s %p)\n", This, debugstr_w(pszkey), punk);
@ -752,7 +753,7 @@ static HRESULT WINAPI AsyncBindCtx_GetObjectParam(IBindCtx* iface, LPOLESTR pszk
static HRESULT WINAPI AsyncBindCtx_RevokeObjectParam(IBindCtx *iface, LPOLESTR pszkey) static HRESULT WINAPI AsyncBindCtx_RevokeObjectParam(IBindCtx *iface, LPOLESTR pszkey)
{ {
AsyncBindCtx *This = BINDCTX_THIS(iface); AsyncBindCtx *This = impl_from_IBindCtx(iface);
TRACE("(%p)->(%s)\n", This, debugstr_w(pszkey)); TRACE("(%p)->(%s)\n", This, debugstr_w(pszkey));
@ -761,15 +762,13 @@ static HRESULT WINAPI AsyncBindCtx_RevokeObjectParam(IBindCtx *iface, LPOLESTR p
static HRESULT WINAPI AsyncBindCtx_EnumObjectParam(IBindCtx *iface, IEnumString **pszkey) static HRESULT WINAPI AsyncBindCtx_EnumObjectParam(IBindCtx *iface, IEnumString **pszkey)
{ {
AsyncBindCtx *This = BINDCTX_THIS(iface); AsyncBindCtx *This = impl_from_IBindCtx(iface);
TRACE("(%p)->(%p)\n", This, pszkey); TRACE("(%p)->(%p)\n", This, pszkey);
return IBindCtx_EnumObjectParam(This->bindctx, pszkey); return IBindCtx_EnumObjectParam(This->bindctx, pszkey);
} }
#undef BINDCTX_THIS
static const IBindCtxVtbl AsyncBindCtxVtbl = static const IBindCtxVtbl AsyncBindCtxVtbl =
{ {
AsyncBindCtx_QueryInterface, AsyncBindCtx_QueryInterface,
@ -876,16 +875,16 @@ HRESULT WINAPI CreateAsyncBindCtxEx(IBindCtx *ibind, DWORD options,
ret = heap_alloc(sizeof(AsyncBindCtx)); ret = heap_alloc(sizeof(AsyncBindCtx));
ret->lpBindCtxVtbl = &AsyncBindCtxVtbl; ret->IBindCtx_iface.lpVtbl = &AsyncBindCtxVtbl;
ret->ref = 1; ret->ref = 1;
ret->bindctx = bindctx; ret->bindctx = bindctx;
hres = init_bindctx(BINDCTX(ret), options, callback, format); hres = init_bindctx(&ret->IBindCtx_iface, options, callback, format);
if(FAILED(hres)) { if(FAILED(hres)) {
IBindCtx_Release(BINDCTX(ret)); IBindCtx_Release(&ret->IBindCtx_iface);
return hres; return hres;
} }
*pbind = BINDCTX(ret); *pbind = &ret->IBindCtx_iface;
return S_OK; return S_OK;
} }