[PATCH v2 1/7] wmp: Add IWMPPlayer interface

Jacek Caban jacek at codeweavers.com
Tue Feb 6 10:34:21 CST 2018


Hi Anton,

On 02/02/2018 07:00 AM, Anton Romanov wrote:
>          TRACE("(%p)->(IID_IWMPPlayer4 %p)\n", This, ppv);
>          *ppv = &This->IWMPPlayer4_iface;
> +    }else if(IsEqualGUID(riid, &IID_IWMPPlayer)) {
> +        TRACE("(%p)->(IID_IWMPPlayer %p)\n", This, ppv);
> +        *ppv = &This->IWMPPlayer4_iface;


You can't do that. IWMPPlayer4 doesn't inherit from IWMPPlayer, so it's
not the same interface. Also a short test (just calling QueryInterface()
and checking the result) would be nice.


Thanks,

Jacek




More information about the wine-devel mailing list