[PATCH 1/3] amstream: Add CLSID_AMAudioData implementation. (try 2)

[Christian Costa]

2012/4/3 Michael Stefaniuc <mstefani at redhat.com>

> 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.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.winehq.org/pipermail/wine-devel/attachments/20120403/c4e1f92d/attachment-0001.html>