Does anyone else think that "arguments" may not be the best name? Perhaps "argument_types" would be more suitable...<br><br><div class="gmail_quote">On Sat, Feb 18, 2012 at 8:28 PM, Gregory Szorc <span dir="ltr"><<a href="mailto:gregory.szorc@gmail.com">gregory.szorc@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I've refactored existing tests to use the new get_cursor(). I also changed the implementation of get_cursor to support recursion. In a forthcoming patch, I have added an argument to get_children() to recurse. Until then, this suboptimal implementation can persist.<div class="HOEnZb">
<div class="h5"><br>
<br><div class="gmail_quote">On Fri, Feb 17, 2012 at 4:39 AM, Manuel Klimek <span dir="ltr"><<a href="mailto:klimek@google.com" target="_blank">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">

Regarding the introduction of get_cursor(...): same as in the other<br>
patch: looks like a good idea, but then use that in the other tests,<br>
too. Also, like in the other patch, I'm still confused whether there's<br>
a scheme behind when you use ok_ and when you use assert...<br>
<br>
In general: very nice tests :)<br>
<br>
Cheers,<br>
/Manuel<br>
<div><div><br>
On Fri, Feb 17, 2012 at 8:57 AM, Gregory Szorc <<a href="mailto:gregory.szorc@gmail.com" target="_blank">gregory.szorc@gmail.com</a>> wrote:<br>
> I incorporated feedback from Tobias and exposed arguments as an<br>
> iterable and indexable container object.<br>
><br>
> ---<br>
>  bindings/python/clang/cindex.py           |   47 +++++++++++++++++++<br>
>  bindings/python/tests/cindex/test_type.py |   69 +++++++++++++++++++++++++++++<br>
>  2 files changed, 116 insertions(+), 0 deletions(-)<br>
><br>
</div></div>> _______________________________________________<br>
> cfe-commits mailing list<br>
> <a href="mailto:cfe-commits@cs.uiuc.edu" target="_blank">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>
</div></div></blockquote></div><br>