Re: [DotGNU]System.Xml

[minddog]

> if (attributeIndex < attributes.Count)
> {
>       value = attributes[attributeIndex].Value;
>       nodeType = XmlNodeType.Text;
>       return true;
> }
> else
> {
>       return false;
> }

You should also make sure to check for attributeIndex placement... != -1

if(attributeIndex < attributes.Count && attributeIndex > -1) { ... }

Make sure to remember that <?xml parsing is in a different area and
currently isn't handled like an element.  I've been hacking out this exact
problem in the newer parser, but again its not ready for primetime so
these bugs must be fixed.  I got stumped on this one because I was
wondering where we should increment, inside ReadAttributeValue()?  because
it looks like gopal's test case will run a while loop on the function
waiting for it to turn false, but if its stationary(attributeIndex is not
inc/dec) then it will always return true.  Something to think about...