thyecust updated this revision to Diff 501407. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D145031/new/
https://reviews.llvm.org/D145031 Files: clang-tools-extra/clang-tidy/google/ExplicitConstructorCheck.cpp Index: clang-tools-extra/clang-tidy/google/ExplicitConstructorCheck.cpp =================================================================== --- clang-tools-extra/clang-tidy/google/ExplicitConstructorCheck.cpp +++ clang-tools-extra/clang-tidy/google/ExplicitConstructorCheck.cpp @@ -98,7 +98,7 @@ const auto *Ctor = Result.Nodes.getNodeAs<CXXConstructorDecl>("ctor"); if (Ctor->isOutOfLine() || Ctor->getNumParams() == 0 || - Ctor->getMinRequiredArguments() > 1) + Ctor->isConvertingConstructor(true)) return; bool TakesInitializerList = isStdInitializerList(
Index: clang-tools-extra/clang-tidy/google/ExplicitConstructorCheck.cpp =================================================================== --- clang-tools-extra/clang-tidy/google/ExplicitConstructorCheck.cpp +++ clang-tools-extra/clang-tidy/google/ExplicitConstructorCheck.cpp @@ -98,7 +98,7 @@ const auto *Ctor = Result.Nodes.getNodeAs<CXXConstructorDecl>("ctor"); if (Ctor->isOutOfLine() || Ctor->getNumParams() == 0 || - Ctor->getMinRequiredArguments() > 1) + Ctor->isConvertingConstructor(true)) return; bool TakesInitializerList = isStdInitializerList(
_______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits