[3/4] msxml3: Add IDispatchEx support for IXMLDOMNamedNodeMap

Jacek Caban jacek at codeweavers.com
Fri Nov 4 12:00:12 CDT 2011


Hi Nikolay,

On 11/04/11 16:46, Nikolay Sivov wrote:
> +    if(This->data->vtbl&&  This->data->vtbl->invoke) {
> +        hres = This->data->vtbl->invoke(This->outer, id, lcid, wFlags, pdp, pvarRes, pei);
> +        if (hres != E_NOTIMPL) return hres;
> +    }

Using E_NOTIMPL for special meaning here seems wrong to me. 
DISP_E_UNKNOWNNAME could be a better choice. However, if you'd move 
custom invoke implementation to be the last choice, after dynamic and 
typelib-based values, then you could simply return whatever invoke returns.

Jacek



More information about the wine-devel mailing list