2012/4/3 Michael Stefaniuc
> Hello Christian,
>
> On 04/03/2012 07:53 AM, Christian Costa wrote:
> > +/*** IUnknown methods ***/
> > +static HRESULT WINAPI IAudioDataImpl_QueryInterface(IAudioData *iface,
> REFIID riid, void **ppv)
> > +{
> > +AMAudioDataImpl *This = impl_from_IAudioData(iface);
> > +
> > +TRACE("(%p/%p)->(%s,%p)\n", iface, This, debugstr_guid(riid), ppv);
> > +
> > +if (IsEqualGUID(riid, &IID_IUnknown) ||
> > +IsEqualGUID(riid, &IID_IMemoryData) ||
> > +IsEqualGUID(riid, &IID_IAudioData))
> > +{
> > +IUnknown_AddRef(iface);
> > +*ppv = This;
> even though MS loves to call ppv ppvObject it isn't an object but a
> pointer to an interface. As this object has just one interface you can
> get rid of This and just assign iface to *ppv.
>
> > +return S_OK;
> > +}
> > +
> > +ERR("(%p)->(%s,%p),not found\n", This, debugstr_guid(riid), ppv);
> > +return E_NOINTERFACE;
> > +}
> > +
>
> bye
> michael
>
Ok I will resend a patch and.
Next two patches of the serie are independant and can be applied without
this patch.
