[clang] #101784 part 1: introduce ctyped in an independent manner (PR #101941)
Jannick Kremer via cfe-commits
cfe-commits at lists.llvm.org
Tue Aug 6 05:00:32 PDT 2024
https://github.com/DeinAlptraum commented:
I agree with what Endill said: this is a rather big change and I'm not sure I see it adding enough value to justify this.
It also adds quite some complexity on top of the otherwise relatively light-weight Python bindings. I generally like seeing changes towards better typing annotations, but this is not something we need to push through no-matter-what and e.g. the `type: ignore`s are "good enough" for our case.
Another reason I'm hesitant to support this, is that the bindings currently depend only on the Python standard library, and adding e.g. `ctyped` would break this. Adding outside dependencies is not _strictly_ a problem, but again any change to this would imo need a good justification that I don't see at the moment.
https://github.com/llvm/llvm-project/pull/101941
More information about the cfe-commits
mailing list