[PATCH] D26118: [clang-tidy] Change readability-redundant-member-init to get base type from constructor

Aaron Ballman via cfe-commits cfe-commits at lists.llvm.org
Tue Nov 1 11:04:43 PDT 2016


aaron.ballman added a comment.

This change is missing a test case.



================
Comment at: clang-tidy/readability/RedundantMemberInitCheck.cpp:57
            "initializer for base class %0 is redundant")
-          << Init->getTypeSourceInfo()->getType()
+          << Construct->getType()
           << FixItHint::CreateRemoval(Init->getSourceRange());
----------------
Why is it more correct to use the CXXConstructExpr type information rather than the CXXCtorInitializer?


Repository:
  rL LLVM

https://reviews.llvm.org/D26118





More information about the cfe-commits mailing list