diff options
author | Alexander Kornienko <alexfh@google.com> | 2014-04-29 15:50:14 +0000 |
---|---|---|
committer | Alexander Kornienko <alexfh@google.com> | 2014-04-29 15:50:14 +0000 |
commit | 3b26849f38f807ef4bf94bb4ade3c58da1dfa48e (patch) | |
tree | e74b9b67de2f4a8815f87b8ff2d307bbef71e800 /clang-tools-extra/clang-tidy/google/GoogleTidyModule.cpp | |
parent | c282e54eb68e5313ca4b60c6fbbfa58369f2f66c (diff) | |
download | bcm5719-llvm-3b26849f38f807ef4bf94bb4ade3c58da1dfa48e.tar.gz bcm5719-llvm-3b26849f38f807ef4bf94bb4ade3c58da1dfa48e.zip |
Fix the message for "Move constructor declared explicit".
Make the tests for removal of 'explicit' more diverse.
llvm-svn: 207534
Diffstat (limited to 'clang-tools-extra/clang-tidy/google/GoogleTidyModule.cpp')
-rw-r--r-- | clang-tools-extra/clang-tidy/google/GoogleTidyModule.cpp | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/clang-tools-extra/clang-tidy/google/GoogleTidyModule.cpp b/clang-tools-extra/clang-tidy/google/GoogleTidyModule.cpp index a150a8c43ec..a732242f778 100644 --- a/clang-tools-extra/clang-tidy/google/GoogleTidyModule.cpp +++ b/clang-tools-extra/clang-tidy/google/GoogleTidyModule.cpp @@ -71,7 +71,8 @@ void ExplicitConstructorCheck::check(const MatchFinder::MatchResult &Result) { Ctor->getOuterLocStart(), Ctor->getLocEnd(), isKWExplicit); if (ExplicitTokenRange.isValid()) { DiagnosticBuilder Diag = diag(ExplicitTokenRange.getBegin(), - "Copy constructor declared explicit."); + "%0 constructor declared explicit.") + << (Ctor->isMoveConstructor() ? "Move" : "Copy"); Diag << FixItHint::CreateRemoval( CharSourceRange::getCharRange(ExplicitTokenRange)); } |