r211482 - cindex.py: remove obsolete workaround and FIXME

Alp Toker alp at nuanti.com
Sun Jun 22 16:28:54 PDT 2014


Author: alp
Date: Sun Jun 22 18:28:54 2014
New Revision: 211482

URL: http://llvm.org/viewvc/llvm-project?rev=211482&view=rev
Log:
cindex.py: remove obsolete workaround and FIXME

clang_getCursorSpelling() doesn't assert on non-declarations any more and the
behaviour is covered by c-index tests.

Passes nosetests.

Modified:
    cfe/trunk/bindings/python/clang/cindex.py

Modified: cfe/trunk/bindings/python/clang/cindex.py
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/bindings/python/clang/cindex.py?rev=211482&r1=211481&r2=211482&view=diff
==============================================================================
--- cfe/trunk/bindings/python/clang/cindex.py (original)
+++ cfe/trunk/bindings/python/clang/cindex.py Sun Jun 22 18:28:54 2014
@@ -1166,10 +1166,6 @@ class Cursor(Structure):
     @property
     def spelling(self):
         """Return the spelling of the entity pointed at by the cursor."""
-        if not self.kind.is_declaration():
-            # FIXME: clang_getCursorSpelling should be fixed to not assert on
-            # this, for consistency with clang_getCursorUSR.
-            return None
         if not hasattr(self, '_spelling'):
             self._spelling = conf.lib.clang_getCursorSpelling(self)
 





More information about the cfe-commits mailing list