Re: [PATCH v2 0/1] MR6993: msxml3: Correct looping of Document Element node map.

[Nikolay Sivov (＠nsivov)]

Nikolay Sivov (@nsivov) commented about dlls/msxml3/tests/domdoc.c:
> - hr = IXMLDOMNamedNodeMap_get_length(map, &len);
> - ok(hr == S_OK, "Unexpected hr %#lx.\n", hr);
> - ok(len == 4, "len %#lx.\n", len);
> -
> - for(i=0; i < len; i++)
> - {
> -
>    IXMLDOMNode *pr_node = NULL;
> -
> -
>    hr = IXMLDOMNamedNodeMap_get_item(map, i, &pr_node);
> -
>    ok( hr == S_OK, "Unexpected hr %#lx (%ld).\n", hr, i);
> -
> -
>    hr = IXMLDOMNode_get_nodeName(pr_node, &nodename);
> -
>    ok(hr == S_OK, "Unexpected hr %#lx.\n", hr);
> -
>    ok(!lstrcmpW(nodename, _bstr_(namespaces[i])), "got %s\n", wine_dbgstr_w(nodename));
> -
>    SysFreeString(nodename);
> - }

You don't need `_bstr_` here, just use WCHAR constants. It's also missing release for the pr_node.