[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