[PATCH] D37905: [libclang, bindings]: add spelling location
Masud Rahman via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Sep 28 10:40:28 PDT 2017
frutiger added inline comments.
Comment at: bindings/python/clang/cindex.py:320
+ return self._get_spelling()['offset']
def __eq__(self, other):
> Does it make sense to introduce two new properties `expansion` and `spelling` and have the four fields be properties on those properties? It seems like it would be more pythonic.
I agree, but I was concerned about breaking existing users that might be using the expansion properties directly on this object. Would marking them deprecated in the documentation suffice?
More information about the cfe-commits