Hi Gregory, yay for the simplification!
One minor thing: in test_cursor.py I'd probably get rid of the intermediate vars for source... Overall looks good. Cheers, /Manuel On Mon, Mar 5, 2012 at 1:57 AM, Gregory Szorc <[email protected]> wrote: > I don't believe anything functional changed with this patch. Just some > housekeeping. It was big enough that I didn't think it was trivial and > qualified for automatic commit. > > --- > bindings/python/tests/cindex/test_cursor.py | 32 ++++------ > bindings/python/tests/cindex/test_diagnostics.py | 33 +++------- > bindings/python/tests/cindex/test_location.py | 78 > ++++++++++++---------- > bindings/python/tests/cindex/test_type.py | 32 +-------- > bindings/python/tests/cindex/util.py | 65 ++++++++++++++++++ > 5 files changed, 130 insertions(+), 110 deletions(-) > create mode 100644 bindings/python/tests/cindex/util.py > > _______________________________________________ > 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
