[clang-tidy] Clarify message for the google-explicit-constructor check

Use "constructors that are callable with a single argument" instead of
"single-argument constructors" when referring to constructors using default
arguments or parameter packs.

llvm-svn: 233702

1 files changed, 5 insertions, 1 deletions

diff --git a/clang-tools-extra/clang-tidy/google/ExplicitConstructorCheck.cpp b/clang-tools-extra/clang-tidy/google/ExplicitConstructorCheck.cpp
index d262f9fa89c..9552e1afa34 100644
--- a/clang-tools-extra/clang-tidy/google/ExplicitConstructorCheck.cpp
+++ b/clang-tools-extra/clang-tidy/google/ExplicitConstructorCheck.cpp
@@ -113,8 +113,12 @@ void ExplicitConstructorCheck::check(const MatchFinder::MatchResult &Result) {
       takesInitializerList)
     return;
 
+  bool SingleArgument =
+      Ctor->getNumParams() == 1 && !Ctor->getParamDecl(0)->isParameterPack();
   SourceLocation Loc = Ctor->getLocation();
-  diag(Loc, "single-argument constructors must be explicit")
+  diag(Loc, SingleArgument ? "single-argument constructors must be explicit"
+                           : "constructors that are callable with a single "
+                             "argument must be marked explicit")
       << FixItHint::CreateInsertion(Loc, "explicit ");
 }