Feedback incorporated. Also, the comment in my previous email about changing the get_cursor() implementation really applies to this patch.<br><br><div class="gmail_quote">On Fri, Feb 17, 2012 at 4:43 AM, Manuel Klimek <span dir="ltr"><<a href="mailto:klimek@google.com">klimek@google.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">+    i, j = None, None<br>
+    for cursor in tu.cursor.get_children():<br>
+        if cursor.spelling == 'i':<br>
+            i = cursor<br>
+        elif cursor.spelling == 'j':<br>
+            j = cursor<br>
<br>
I'd reuse get_cursor from patch 2 here.<br>
<br>
Cheers,<br>
/Manuel<br>
<div><div class="h5"><br>
On Fri, Feb 17, 2012 at 8:58 AM, Gregory Szorc <<a href="mailto:gregory.szorc@gmail.com">gregory.szorc@gmail.com</a>> wrote:<br>
> ---<br>
>  bindings/python/tests/cindex/test_type.py |   39 +++++++++++++++++++++++++++++<br>
>  1 files changed, 39 insertions(+), 0 deletions(-)<br>
><br>
</div></div>> _______________________________________________<br>
> cfe-commits mailing list<br>
> <a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
><br>
</blockquote></div><br>