[cfe-commits] Request for approval: Pythonstuff including Cursor.enum_value

Manuel Klimek klimek at google.com
Wed May 2 10:33:50 PDT 2012


Regarding patch 0001:
There are utility methods in util.py which make writing those tests
somewhat more convenient:
tu = get_tu(annotationInput)
annotated_foo = get_cursor(tu, 'foo')

Also, the surrounding style seems to use kSomething for file level
string constants, so I'd use that for the test, too. In general, I'd
make it look like the tests in test_cursor.py already do.

Cheers,
/Manuel

On Wed, May 2, 2012 at 10:49 AM, Anders Waldenborg <anders at 0x63.nu> wrote:
> 5 patches attached, 4 first is mostly trivial stuff.
>
>
> * [python] Add testcase for annotation cursor kind
>
>  Old testcase I had laying around.
>
> * [python] Fix warning in testcase
>
>  Trivial fix for a warning in the c-code in testcase
>
> * [python] Run tests for c++ with std=c++11
>
>  Makes c++ python testcases compile with -std=c++11,
>  which is needed by...
>
> * [python] Add testcase for enum with specified underlaying type
>
>  Adds a more interesting enum_type testcase which
>  has a different type.
>
> * [python] Add Cursor.enum_value wrapping clang_getEnumConstantDeclValue
>
>  This is the main patch. It adds a new "enum_value" property to the
>  Cursor type, allowing to get the value of enum constants
>
> _______________________________________________
> 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