[PATCH] D17841: [libclang/python] Add bindings for children of diagnostics

[Hanson Wang via cfe-commits]

hansonw added a comment.

Thanks! I don't have commit permissions; could you check this in for me?


================
Comment at: bindings/python/clang/cindex.py:369
@@ +368,3 @@
+            def __len__(self):
+                return int(conf.lib.clang_getNumDiagnosticsInSet(self.diag_set))
+
----------------
compnerd wrote:
> Why does this need the explicit int construction?
This returns a long on 64-bit systems - however `__len__` must return a plain int.


http://reviews.llvm.org/D17841