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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to