<br><br><div class="gmail_quote">2012/4/3 Michael Stefaniuc <span dir="ltr"><<a href="mailto:mstefani@redhat.com">mstefani@redhat.com</a>></span><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hello Christian,<br>
<div class="im"><br>
On 04/03/2012 07:53 AM, Christian Costa wrote:<br>
> +/*** IUnknown methods ***/<br>
> +static HRESULT WINAPI IAudioDataImpl_QueryInterface(IAudioData *iface, REFIID riid, void **ppv)<br>
> +{<br>
> +    AMAudioDataImpl *This = impl_from_IAudioData(iface);<br>
> +<br>
> +    TRACE("(%p/%p)->(%s,%p)\n", iface, This, debugstr_guid(riid), ppv);<br>
> +<br>
> +    if (IsEqualGUID(riid, &IID_IUnknown) ||<br>
> +        IsEqualGUID(riid, &IID_IMemoryData) ||<br>
> +        IsEqualGUID(riid, &IID_IAudioData))<br>
> +    {<br>
> +        IUnknown_AddRef(iface);<br>
> +        *ppv = This;<br>
</div>even though MS loves to call ppv ppvObject it isn't an object but a<br>
pointer to an interface. As this object has just one interface you can<br>
get rid of This and just assign iface to *ppv.<br>
<div class="im"><br>
> +        return S_OK;<br>
> +    }<br>
> +<br>
> +    ERR("(%p)->(%s,%p),not found\n", This, debugstr_guid(riid), ppv);<br>
> +    return E_NOINTERFACE;<br>
> +}<br>
> +<br>
<br>
</div>bye<br>
<span class="HOEnZb"><font color="#888888">        michael<br>
</font></span></blockquote></div><br>Ok I will resend a patch and.<br>Next two patches of the serie are independant and can be applied without this patch.<br>