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 <[email protected]> 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 > [email protected] > http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits > _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
