[cfe-commits] [Patch] [PythonBindings] Exposing cursor semantic and lexical parent to python (now with proper tests)

Manuel Klimek klimek at google.com
Wed May 2 10:23:39 PDT 2012


Looks good.

Very minor nit: Comments are easier to read (IMO) if you're adding a
'.' when you have a full sentence.

On Tue, May 1, 2012 at 7:37 AM, Evan P <hiwrac at gmail.com> wrote:
> When I was writing tests for the original version of this patch I discovered
> that clang_getCursorLexicalParent was actually broken for some edge cases.
> Now that my patch for that issue was committed in r155858, lets try this
> again.
>
> - Adding lexical_parent and semantic_parent properties to
> clang.cindex.Cursor
> - Two new tests (one for each property), require libclang built from r155858
> or later to pass
> - New test utility function (get_cursors) that gets all the nodes with a
> specific spelling.
>
>
> Thanks
> -Evan Pipho
>
>
>
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>




More information about the cfe-commits mailing list