Re: [PATCH v3 0/7] MR9136: mshtml: Use a common object implementation for collection enumerators.

[Jacek Caban (＠jacek)]

Jacek Caban (@jacek) commented about dlls/mshtml/htmlnode.c:
> +{
> +    HTMLDOMChildrenCollection *This = impl_from_DispatchEx(dispex);
> +    nsIDOMNode *nsnode;
> +    HTMLDOMNode *node;
> +    nsresult nsres;
> +    HRESULT hres;
> +
> +    nsres = nsIDOMNodeList_Item(This->nslist, index, &nsnode);
> +    if(NS_FAILED(nsres))
> +        return map_nsresult(nsres);
> +
> +    hres = get_node(nsnode, TRUE, &node);
> +    nsIDOMNode_Release(nsnode);
> +    if(SUCCEEDED(hres))
> +        *p = (IDispatch*)&node->IHTMLDOMNode_iface;
> +    return hres;
> This duplicates `HTMLDOMChildrenCollection_item`. There is similar pattern in a few other patches, please try to avoid that.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/9136#note_119107