diff options
| author | Daniel Jasper <djasper@google.com> | 2014-07-21 06:06:38 +0000 |
|---|---|---|
| committer | Daniel Jasper <djasper@google.com> | 2014-07-21 06:06:38 +0000 |
| commit | 58ed9c91679dc81406de8ff4bcd53f4b40ebbbee (patch) | |
| tree | 75682089f816547c4b5d7825efba80b2925bd066 /clang-tools-extra/clang-tidy | |
| parent | ab4ebb42f971f476b220751ba17f75f937537f17 (diff) | |
| download | bcm5719-llvm-58ed9c91679dc81406de8ff4bcd53f4b40ebbbee.tar.gz bcm5719-llvm-58ed9c91679dc81406de8ff4bcd53f4b40ebbbee.zip | |
clang-tidy: [misc-use-override] Slightly tweak the wording of warning.
'final' should really be used with care.
llvm-svn: 213501
Diffstat (limited to 'clang-tools-extra/clang-tidy')
| -rw-r--r-- | clang-tools-extra/clang-tidy/misc/UseOverride.cpp | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/clang-tools-extra/clang-tidy/misc/UseOverride.cpp b/clang-tools-extra/clang-tidy/misc/UseOverride.cpp index d1a9e2f1c77..685750eec35 100644 --- a/clang-tools-extra/clang-tidy/misc/UseOverride.cpp +++ b/clang-tools-extra/clang-tidy/misc/UseOverride.cpp @@ -71,11 +71,11 @@ void UseOverride::check(const MatchFinder::MatchResult &Result) { if (!OnlyVirtualSpecified && KeywordCount == 1) return; // Nothing to do. - DiagnosticBuilder Diag = - diag(Method->getLocation(), - OnlyVirtualSpecified - ? "Prefer using 'override' or 'final' instead of 'virtual'" - : "Use exactly one of 'virtual', 'override' and 'final'"); + DiagnosticBuilder Diag = diag( + Method->getLocation(), + OnlyVirtualSpecified + ? "Prefer using 'override' or 'final' instead of 'virtual'" + : "Use exactly one of 'virtual', 'override' or (rarely) 'final'"); CharSourceRange FileRange = Lexer::makeFileCharRange( CharSourceRange::getTokenRange(Method->getSourceRange()), Sources, |
