[PATCH 2/2 resend] devenum: Return S_OK in IMoniker::IsEqual() stub.
Andrew Eikum
aeikum at codeweavers.com
Fri Mar 10 08:15:11 CST 2017
I'm not sure how application use these monikers, but lying about
moniker equality makes me a little nervous. This doesn't look that
hard to implement correctly.
Andrew
On Fri, Mar 10, 2017 at 06:26:55PM +0800, Jactry Zeng wrote:
> Superseded patch 131295.
>
> Signed-off-by: Jactry Zeng <jzeng at codeweavers.com>
> ---
> dlls/devenum/mediacatenum.c | 5 ++++-
> 1 file changed, 4 insertions(+), 1 deletion(-)
>
>
> diff --git a/dlls/devenum/mediacatenum.c b/dlls/devenum/mediacatenum.c
> index 66343342c9..3543182cf4 100644
> --- a/dlls/devenum/mediacatenum.c
> +++ b/dlls/devenum/mediacatenum.c
> @@ -530,7 +530,10 @@ static HRESULT WINAPI DEVENUM_IMediaCatMoniker_IsEqual(IMoniker *iface, IMoniker
> {
> FIXME("(%p)->(%p): stub\n", iface, pmkOtherMoniker);
>
> - return E_NOTIMPL;
> + if (!pmkOtherMoniker)
> + return E_INVALIDARG;
> +
> + return S_OK;
> }
>
> static HRESULT WINAPI DEVENUM_IMediaCatMoniker_Hash(IMoniker *iface, DWORD *pdwHash)
>
>
More information about the wine-patches
mailing list