Endilll wrote:

> Good point about the restype for clang_Cursor_getTemplateArgumentKind though, 
> I didn't notice and we should probably change that

Yes, I'd like us to stop relying on that for the same reason I don't like 
`errcheck`. The fact that it's deprecated further adds to it.

> I don't quite get what 17 meant. As far as I know about ctypes, 
> conf.lib.clang_getRangeStart should be guaranteed to return an instance of 
> SourceLocation. If it is to extinguish type errors, 
> typing.cast(SourceLocation, conf.lib.clang_getRangeStart(self)) can do this.

Yes, 17 is about type errors when a structural type is returned. Good point 
about `typing.cast()`. It's been long enough I forgot that it exists. It might 
be the best way to address type errors in this case.

https://github.com/llvm/llvm-project/pull/101941
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to