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.