[cfe-commits] [PATCH 5/5] [clang.py] Format and add documention for Type

Manuel Klimek klimek at google.com
Fri Feb 17 04:47:39 PST 2012


Looks good.

On Fri, Feb 17, 2012 at 8:58 AM, Gregory Szorc <gregory.szorc at gmail.com> wrote:
> Just some misc formatting. Do I even need a review on style nits?

I'd say "no", citing the http://llvm.org/docs/DeveloperPolicy.html
"""You are allowed to commit patches without approval which you think
are obvious. This is clearly a subjective decision — we simply expect
you to use good judgement. Examples include: fixing build breakage,
reverting obviously broken patches, documentation/comment changes, any
other minor changes. """

Cheers,
/Manuel

> ---
>  bindings/python/clang/cindex.py           |   22 +++++++++++-----------
>  bindings/python/tests/cindex/test_type.py |    3 +++
>  2 files changed, 14 insertions(+), 11 deletions(-)
>
> _______________________________________________
> 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