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