[PATCH] D33722: [clang-tidy] Add checker for undelegated copy of base classes
Gábor Horváth via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Nov 3 07:54:40 PDT 2017
xazax.hun marked 5 inline comments as done.
xazax.hun added a comment.
Two problems are not resolved. One is Aaron prefers to query some infor from the AST instead of relexing. The second is providing base initializers in the wrong order.
I think there are other checks that do relexing in some cases, this should not be a blocker. I am not sure that we should complicate the fixit logic with the order. If -Wreorder has fixit hints, user should be able to get rid of the warning by applying that.
https://reviews.llvm.org/D33722
More information about the cfe-commits
mailing list