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

[Nikolay Sivov (＠nsivov)]

Nikolay Sivov (@nsivov) commented about dlls/msxml3/element.c:
>  if (!node->nsDef)
>      return S_FALSE;
>
> - attrIndex++;
> ns = node->nsDef;
> - if (!ns->next)
> -
>    attrIndex++;
> for (; attrIndex < index && ns->next != NULL; attrIndex++)
>        ns = ns->next;

This is a bit hard to read. Can we maybe incorporate that in loop conditions somehow instead of doing sort of unroll of the first iteration?