Thanks for the review! I've sent a new patch
På Onsdag 20 august 2008 , 12:46:38 skrev du:
> Alexander Nicolaysen Sørnes <[EMAIL PROTECTED]> writes:
> > +static HRESULT WINAPI IDirectPlay8PeerImpl_Initialize(PDIRECTPLAY8PEER
> > iface, PVOID CONST pvUserContext, CONST PFNDPNMESSAGEHANDLER pfn, CONST
> > DWORD dwFlags) +{
> > + FIXME("(%p)->(%p,%p,%x): stub\n", iface, pvUserContext, pfn,
> > dwFlags); +
> > + return DPN_OK;
> > +}
> > +
> > +static HRESULT WINAPI
> > IDirectPlay8PeerImpl_EnumServiceProviders(PDIRECTPLAY8PEER iface, CONST
> > GUID *CONST pguidServiceProvider, CONST GUID *CONST pguidApplication,
> > DPN_SERVICE_PROVIDER_INFO *CONST pSPInfoBuffer, DWORD *CONST pcbEnumData,
> > DWORD *CONST pcReturned, CONST DWORD dwFlags) +{
> > + FIXME("(%p)->(%p,%p,%p,%p,%p,%x): stub\n", iface,
> > pguidServiceProvider, pguidApplication, pSPInfoBuffer, pcbEnumData,
> > pcReturned, dwFlags); + return DPN_OK;
> > +}
>
> Returning OK from stubs is usually not a good idea.
>
ok
> > +static const IDirectPlay8PeerVtbl DirectPlay8Peer_Vtbl =
> > +{
> > + IDirectPlay8PeerImpl_QueryInterface,
> > + IDirectPlay8PeerImpl_AddRef,
> > + IDirectPlay8PeerImpl_Release,
> > + IDirectPlay8PeerImpl_Initialize,
> > + IDirectPlay8PeerImpl_EnumServiceProviders
> > +};
>
> You can't initialize a vtbl partially, you need to stub all the functions.
Ok. I thought it was ok since the rest was added in the patchset.
