[PATCH] D117129: [clang-tidy] Extract Class IncluderClangTidyCheck
Richard via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Jan 12 13:21:36 PST 2022
LegalizeAdulthood added a comment.
In D117129#3238433 <https://reviews.llvm.org/D117129#3238433>, @ymandel wrote:
> Overall, this change looks good and makes sense. But, it doesn't scale past 1 feature
> [...] Might there be some other way to get the benefits of saving implementers the
> cut and paste (and bug-prone code that comes with it) without changing the class
> hierarchy?
Maybe some sort of mix-in approach or "implementation inheritance" as they say?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D117129/new/
https://reviews.llvm.org/D117129
More information about the cfe-commits
mailing list