blob: 897c60a9203d12e5e90f27350c0c7b2fcdccafbe [file] [log] [blame]
Alexander Kornienkobef51cd2014-05-19 16:39:08 +00001//===--- NamespaceCommentCheck.cpp - clang-tidy ---------------------------===//
2//
3// The LLVM Compiler Infrastructure
4//
5// This file is distributed under the University of Illinois Open Source
6// License. See LICENSE.TXT for details.
7//
8//===----------------------------------------------------------------------===//
9
10#include "NamespaceCommentCheck.h"
11#include "clang/AST/ASTContext.h"
12#include "clang/ASTMatchers/ASTMatchers.h"
13#include "clang/Lex/Lexer.h"
Alexander Kornienko6e0cbc82014-09-12 08:53:36 +000014#include "llvm/ADT/StringExtras.h"
Alexander Kornienkobef51cd2014-05-19 16:39:08 +000015
16using namespace clang::ast_matchers;
17
18namespace clang {
19namespace tidy {
20
Alexander Kornienko6e0cbc82014-09-12 08:53:36 +000021NamespaceCommentCheck::NamespaceCommentCheck(StringRef Name,
22 ClangTidyContext *Context)
23 : ClangTidyCheck(Name, Context),
24 NamespaceCommentPattern("^/[/*] *(end (of )?)? *(anonymous|unnamed)? *"
Alexander Kornienkobef51cd2014-05-19 16:39:08 +000025 "namespace( +([a-zA-Z0-9_]+))? *(\\*/)?$",
26 llvm::Regex::IgnoreCase),
Alexander Kornienko6e0cbc82014-09-12 08:53:36 +000027 ShortNamespaceLines(Options.get("ShortNamespaceLines", 1u)),
28 SpacesBeforeComments(Options.get("SpacesBeforeComments", 1u)) {}
29
30void NamespaceCommentCheck::storeOptions(ClangTidyOptions::OptionMap &Opts) {
31 Options.store(Opts, "ShortNamespaceLines", ShortNamespaceLines);
32 Options.store(Opts, "SpacesBeforeComments", SpacesBeforeComments);
33}
Alexander Kornienkobef51cd2014-05-19 16:39:08 +000034
35void NamespaceCommentCheck::registerMatchers(MatchFinder *Finder) {
36 Finder->addMatcher(namespaceDecl().bind("namespace"), this);
37}
38
39bool locationsInSameFile(const SourceManager &Sources, SourceLocation Loc1,
40 SourceLocation Loc2) {
41 return Loc1.isFileID() && Loc2.isFileID() &&
42 Sources.getFileID(Loc1) == Sources.getFileID(Loc2);
43}
44
Alexander Kornienko6e0cbc82014-09-12 08:53:36 +000045std::string getNamespaceComment(const NamespaceDecl *ND, bool InsertLineBreak,
46 unsigned SpacesBeforeComments) {
Alexander Kornienkobef51cd2014-05-19 16:39:08 +000047 std::string Fix = "// namespace";
48 if (!ND->isAnonymousNamespace())
Alexander Kornienko6e0cbc82014-09-12 08:53:36 +000049 Fix.append(std::string(SpacesBeforeComments, ' '))
50 .append(ND->getNameAsString());
Alexander Kornienkobef51cd2014-05-19 16:39:08 +000051 if (InsertLineBreak)
52 Fix.append("\n");
53 return Fix;
54}
55
56void NamespaceCommentCheck::check(const MatchFinder::MatchResult &Result) {
57 const NamespaceDecl *ND = Result.Nodes.getNodeAs<NamespaceDecl>("namespace");
58 const SourceManager &Sources = *Result.SourceManager;
59
60 if (!locationsInSameFile(Sources, ND->getLocStart(), ND->getRBraceLoc()))
61 return;
62
63 // Don't require closing comments for namespaces spanning less than certain
64 // number of lines.
65 unsigned StartLine = Sources.getSpellingLineNumber(ND->getLocStart());
66 unsigned EndLine = Sources.getSpellingLineNumber(ND->getRBraceLoc());
67 if (EndLine - StartLine + 1 <= ShortNamespaceLines)
68 return;
69
70 // Find next token after the namespace closing brace.
71 SourceLocation AfterRBrace = ND->getRBraceLoc().getLocWithOffset(1);
72 SourceLocation Loc = AfterRBrace;
73 Token Tok;
74 // Skip whitespace until we find the next token.
75 while (Lexer::getRawToken(Loc, Tok, Sources, Result.Context->getLangOpts())) {
76 Loc = Loc.getLocWithOffset(1);
77 }
78 if (!locationsInSameFile(Sources, ND->getRBraceLoc(), Loc))
79 return;
80
81 bool NextTokenIsOnSameLine = Sources.getSpellingLineNumber(Loc) == EndLine;
Alexander Kornienko16ff5c72014-05-19 17:46:28 +000082 // If we insert a line comment before the token in the same line, we need
83 // to insert a line break.
Alexander Kornienkobef51cd2014-05-19 16:39:08 +000084 bool NeedLineBreak = NextTokenIsOnSameLine && Tok.isNot(tok::eof);
85
86 // Try to find existing namespace closing comment on the same line.
87 if (Tok.is(tok::comment) && NextTokenIsOnSameLine) {
88 StringRef Comment(Sources.getCharacterData(Loc), Tok.getLength());
89 SmallVector<StringRef, 6> Groups;
90 if (NamespaceCommentPattern.match(Comment, &Groups)) {
91 StringRef NamespaceNameInComment = Groups.size() >= 6 ? Groups[5] : "";
92
93 // Check if the namespace in the comment is the same.
94 if ((ND->isAnonymousNamespace() && NamespaceNameInComment.empty()) ||
95 ND->getNameAsString() == NamespaceNameInComment) {
96 // FIXME: Maybe we need a strict mode, where we always fix namespace
97 // comments with different format.
98 return;
99 }
100
101 // Otherwise we need to fix the comment.
102 NeedLineBreak = Comment.startswith("/*");
103 CharSourceRange OldCommentRange = CharSourceRange::getCharRange(
104 SourceRange(Loc, Loc.getLocWithOffset(Tok.getLength())));
105 diag(Loc, "namespace closing comment refers to a wrong namespace '%0'")
106 << NamespaceNameInComment
107 << FixItHint::CreateReplacement(
Alexander Kornienko6e0cbc82014-09-12 08:53:36 +0000108 OldCommentRange,
109 getNamespaceComment(ND, NeedLineBreak, SpacesBeforeComments));
Alexander Kornienkobef51cd2014-05-19 16:39:08 +0000110 return;
111 }
112
113 // This is not a recognized form of a namespace closing comment.
114 // Leave line comment on the same line. Move block comment to the next line,
115 // as it can be multi-line or there may be other tokens behind it.
116 if (Comment.startswith("//"))
117 NeedLineBreak = false;
118 }
119
120 diag(ND->getLocation(), "namespace not terminated with a closing comment")
121 << FixItHint::CreateInsertion(
Alexander Kornienko6e0cbc82014-09-12 08:53:36 +0000122 AfterRBrace,
123 " " + getNamespaceComment(ND, NeedLineBreak, SpacesBeforeComments));
Alexander Kornienkobef51cd2014-05-19 16:39:08 +0000124}
125
126} // namespace tidy
127} // namespace clang