Re: [PATCH] ole32: fix composite moniker behaviour.

[Alexandre Julliard]

Reece Dunn  writes:
> @@ -365,8 +365,7 @@ CompositeMonikerImpl_BindToObject(IMoniker* iface, IBindCtx* pbc,
>              /* if the requested class was loaded before ! we don't need to reload it */
>              res = IRunningObjectTable_GetObject(prot,iface,(IUnknown**)ppvResult);
> -
>        if (res==S_OK)
> -
>            return res;
> -
>        return res==S_OK ? S_OK : E_INVALIDARG;

This doesn't look right. Just because the test wants E_INVALIDARG
doesn't mean it's the correct error for all possible failures.