aaron.ballman accepted this revision.
aaron.ballman added a comment.
This revision is now accepted and ready to land.
LGTM with a small nit, but you may want to wait for @djasper to weigh in on the
removed comment question.
================
Comment at: clang-tidy/modernize/UseDefaultCheck.cpp:266
// If there are constructor initializers, they must be removed.
- if (Ctor->getNumCtorInitializers() != 0) {
- StartLoc = getColonLoc(Result.Context, Ctor);
- if (!StartLoc.isValid())
- return;
+ for (const CXXCtorInitializer *Init : Ctor->inits()) {
+ RemoveInitializers.emplace_back(
----------------
You can use `const auto *` here.
https://reviews.llvm.org/D25769
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
