Thanks -- I like to see it when more tests are passing!

Do you think that any non-Rhino code would be depending on the signature of "getIndexIds"? I've never seen it but I thought I'd ask.

Finally, can you squash these commits? Otherwise I can do it.

Thanks -- I like to see it when more tests are passing!

Same here, but there is still a lot of things to do.

Do you think that any non-Rhino code would be depending on the signature of "getIndexIds"? I've never seen it but I thought I'd ask.

Good question and i was sure you will ask this while changing this code. As always i have no idea. This is definitive a breaking change and we have to document it. But all the places using this method internally are fine with this change because they are using foreach. So i have pressed the submit button....
Maybe you can have a look at the history, might be the method is relatively new and added for a special need.
Or simply leave this commit out - i do not need this change.

Finally, can you squash these commits? Otherwise I can do it.

Last time i have tried squashing was a disaster. So please do it. But i learned in between how i can squash before making the push request - there is hope for the future :-)