Fariborz Jahanian | ca76bf8 | 2013-03-05 19:52:24 +0000 | [diff] [blame] | 1 | //===--- CommentSema.cpp - Doxygen comment semantic analysis --------------===// |
Dmitri Gribenko | 8d3ba23 | 2012-07-06 00:28:32 +0000 | [diff] [blame] | 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 "clang/AST/CommentSema.h" |
Benjamin Kramer | 2fa67ef | 2012-12-01 15:09:41 +0000 | [diff] [blame] | 11 | #include "clang/AST/Attr.h" |
Dmitri Gribenko | aa58081 | 2012-08-09 00:03:17 +0000 | [diff] [blame] | 12 | #include "clang/AST/CommentCommandTraits.h" |
Benjamin Kramer | 2fa67ef | 2012-12-01 15:09:41 +0000 | [diff] [blame] | 13 | #include "clang/AST/CommentDiagnostic.h" |
Dmitri Gribenko | a5ef44f | 2012-07-11 21:38:39 +0000 | [diff] [blame] | 14 | #include "clang/AST/Decl.h" |
Dmitri Gribenko | 96b0986 | 2012-07-31 22:37:06 +0000 | [diff] [blame] | 15 | #include "clang/AST/DeclTemplate.h" |
Dmitri Gribenko | a5ef44f | 2012-07-11 21:38:39 +0000 | [diff] [blame] | 16 | #include "clang/Basic/SourceManager.h" |
Dmitri Gribenko | 1952354 | 2012-09-29 11:40:46 +0000 | [diff] [blame] | 17 | #include "clang/Lex/Preprocessor.h" |
Dmitri Gribenko | 1952354 | 2012-09-29 11:40:46 +0000 | [diff] [blame] | 18 | #include "llvm/ADT/SmallString.h" |
Chandler Carruth | 55fc873 | 2012-12-04 09:13:33 +0000 | [diff] [blame] | 19 | #include "llvm/ADT/StringSwitch.h" |
Dmitri Gribenko | 8d3ba23 | 2012-07-06 00:28:32 +0000 | [diff] [blame] | 20 | |
| 21 | namespace clang { |
| 22 | namespace comments { |
| 23 | |
Dmitri Gribenko | c24a76e | 2012-08-31 02:21:44 +0000 | [diff] [blame] | 24 | namespace { |
| 25 | #include "clang/AST/CommentHTMLTagsProperties.inc" |
| 26 | } // unnamed namespace |
| 27 | |
Dmitri Gribenko | a5ef44f | 2012-07-11 21:38:39 +0000 | [diff] [blame] | 28 | Sema::Sema(llvm::BumpPtrAllocator &Allocator, const SourceManager &SourceMgr, |
Dmitri Gribenko | 1952354 | 2012-09-29 11:40:46 +0000 | [diff] [blame] | 29 | DiagnosticsEngine &Diags, CommandTraits &Traits, |
| 30 | const Preprocessor *PP) : |
Dmitri Gribenko | aa58081 | 2012-08-09 00:03:17 +0000 | [diff] [blame] | 31 | Allocator(Allocator), SourceMgr(SourceMgr), Diags(Diags), Traits(Traits), |
Fariborz Jahanian | f843a58 | 2013-01-31 23:12:39 +0000 | [diff] [blame] | 32 | PP(PP), ThisDeclInfo(NULL), BriefCommand(NULL), ReturnsCommand(NULL), |
| 33 | HeaderfileCommand(NULL) { |
Dmitri Gribenko | a5ef44f | 2012-07-11 21:38:39 +0000 | [diff] [blame] | 34 | } |
| 35 | |
| 36 | void Sema::setDecl(const Decl *D) { |
Dmitri Gribenko | 1ca7ecc | 2012-08-01 23:08:09 +0000 | [diff] [blame] | 37 | if (!D) |
| 38 | return; |
| 39 | |
| 40 | ThisDeclInfo = new (Allocator) DeclInfo; |
Fariborz Jahanian | bf967be | 2012-10-10 18:34:52 +0000 | [diff] [blame] | 41 | ThisDeclInfo->CommentDecl = D; |
Dmitri Gribenko | 651f8ce | 2012-08-01 23:21:57 +0000 | [diff] [blame] | 42 | ThisDeclInfo->IsFilled = false; |
Dmitri Gribenko | 8d3ba23 | 2012-07-06 00:28:32 +0000 | [diff] [blame] | 43 | } |
| 44 | |
| 45 | ParagraphComment *Sema::actOnParagraphComment( |
| 46 | ArrayRef<InlineContentComment *> Content) { |
| 47 | return new (Allocator) ParagraphComment(Content); |
| 48 | } |
| 49 | |
Dmitri Gribenko | 808383d | 2013-03-04 23:06:15 +0000 | [diff] [blame] | 50 | BlockCommandComment *Sema::actOnBlockCommandStart( |
| 51 | SourceLocation LocBegin, |
| 52 | SourceLocation LocEnd, |
| 53 | unsigned CommandID, |
| 54 | CommandMarkerKind CommandMarker) { |
Fariborz Jahanian | 28c1cd2 | 2013-03-07 23:33:11 +0000 | [diff] [blame] | 55 | BlockCommandComment *BC = new (Allocator) BlockCommandComment(LocBegin, LocEnd, |
| 56 | CommandID, |
| 57 | CommandMarker); |
| 58 | checkContainerDecl(BC); |
| 59 | return BC; |
Dmitri Gribenko | 8d3ba23 | 2012-07-06 00:28:32 +0000 | [diff] [blame] | 60 | } |
| 61 | |
Dmitri Gribenko | 7d9b511 | 2012-08-06 19:03:12 +0000 | [diff] [blame] | 62 | void Sema::actOnBlockCommandArgs(BlockCommandComment *Command, |
| 63 | ArrayRef<BlockCommandComment::Argument> Args) { |
Dmitri Gribenko | 8d3ba23 | 2012-07-06 00:28:32 +0000 | [diff] [blame] | 64 | Command->setArgs(Args); |
Dmitri Gribenko | 8d3ba23 | 2012-07-06 00:28:32 +0000 | [diff] [blame] | 65 | } |
| 66 | |
Dmitri Gribenko | 7d9b511 | 2012-08-06 19:03:12 +0000 | [diff] [blame] | 67 | void Sema::actOnBlockCommandFinish(BlockCommandComment *Command, |
| 68 | ParagraphComment *Paragraph) { |
Dmitri Gribenko | 8d3ba23 | 2012-07-06 00:28:32 +0000 | [diff] [blame] | 69 | Command->setParagraph(Paragraph); |
Dmitri Gribenko | a5ef44f | 2012-07-11 21:38:39 +0000 | [diff] [blame] | 70 | checkBlockCommandEmptyParagraph(Command); |
Dmitri Gribenko | 9443c57 | 2012-08-06 17:08:27 +0000 | [diff] [blame] | 71 | checkBlockCommandDuplicate(Command); |
Dmitri Gribenko | 89ab7d0 | 2012-08-03 21:15:32 +0000 | [diff] [blame] | 72 | checkReturnsCommand(Command); |
Dmitri Gribenko | 0bd9838 | 2012-09-22 21:47:50 +0000 | [diff] [blame] | 73 | checkDeprecatedCommand(Command); |
Dmitri Gribenko | 8d3ba23 | 2012-07-06 00:28:32 +0000 | [diff] [blame] | 74 | } |
| 75 | |
Dmitri Gribenko | 808383d | 2013-03-04 23:06:15 +0000 | [diff] [blame] | 76 | ParamCommandComment *Sema::actOnParamCommandStart( |
| 77 | SourceLocation LocBegin, |
| 78 | SourceLocation LocEnd, |
| 79 | unsigned CommandID, |
| 80 | CommandMarkerKind CommandMarker) { |
Dmitri Gribenko | a5ef44f | 2012-07-11 21:38:39 +0000 | [diff] [blame] | 81 | ParamCommandComment *Command = |
Dmitri Gribenko | 808383d | 2013-03-04 23:06:15 +0000 | [diff] [blame] | 82 | new (Allocator) ParamCommandComment(LocBegin, LocEnd, CommandID, |
| 83 | CommandMarker); |
Dmitri Gribenko | a5ef44f | 2012-07-11 21:38:39 +0000 | [diff] [blame] | 84 | |
Dmitri Gribenko | 8487c52 | 2012-07-23 17:40:30 +0000 | [diff] [blame] | 85 | if (!isFunctionDecl()) |
Dmitri Gribenko | a5ef44f | 2012-07-11 21:38:39 +0000 | [diff] [blame] | 86 | Diag(Command->getLocation(), |
| 87 | diag::warn_doc_param_not_attached_to_a_function_decl) |
Dmitri Gribenko | 808383d | 2013-03-04 23:06:15 +0000 | [diff] [blame] | 88 | << CommandMarker |
Dmitri Gribenko | e4330a3 | 2012-09-10 20:32:42 +0000 | [diff] [blame] | 89 | << Command->getCommandNameRange(Traits); |
Dmitri Gribenko | a5ef44f | 2012-07-11 21:38:39 +0000 | [diff] [blame] | 90 | |
| 91 | return Command; |
Dmitri Gribenko | 8d3ba23 | 2012-07-06 00:28:32 +0000 | [diff] [blame] | 92 | } |
| 93 | |
Fariborz Jahanian | 2a268f2 | 2013-03-05 01:05:07 +0000 | [diff] [blame] | 94 | void Sema::checkFunctionDeclVerbatimLine(const BlockCommandComment *Comment) { |
| 95 | const CommandInfo *Info = Traits.getCommandInfo(Comment->getCommandID()); |
Fariborz Jahanian | 99a7057 | 2013-03-05 22:46:07 +0000 | [diff] [blame] | 96 | if (!Info->IsFunctionDeclarationCommand) |
| 97 | return; |
| 98 | StringRef Name = Info->Name; |
Fariborz Jahanian | 88f070f | 2013-03-06 17:36:51 +0000 | [diff] [blame] | 99 | unsigned DiagSelect = llvm::StringSwitch<unsigned>(Name) |
| 100 | .Case("function", !isAnyFunctionDecl() ? 1 : 0) |
| 101 | .Case("method", !isObjCMethodDecl() ? 2 : 0) |
| 102 | .Case("callback", !isFunctionPointerVarDecl() ? 3 : 0) |
Fariborz Jahanian | 99a7057 | 2013-03-05 22:46:07 +0000 | [diff] [blame] | 103 | .Default(0); |
| 104 | |
Fariborz Jahanian | 88f070f | 2013-03-06 17:36:51 +0000 | [diff] [blame] | 105 | if (DiagSelect) |
| 106 | Diag(Comment->getLocation(), diag::warn_doc_function_method_decl_mismatch) |
| 107 | << Comment->getCommandMarker() |
| 108 | << (DiagSelect-1) << (DiagSelect-1) |
Fariborz Jahanian | 2a268f2 | 2013-03-05 01:05:07 +0000 | [diff] [blame] | 109 | << Comment->getSourceRange(); |
| 110 | } |
Fariborz Jahanian | 28c1cd2 | 2013-03-07 23:33:11 +0000 | [diff] [blame] | 111 | |
| 112 | void Sema::checkContainerDeclVerbatimLine(const BlockCommandComment *Comment) { |
| 113 | const CommandInfo *Info = Traits.getCommandInfo(Comment->getCommandID()); |
| 114 | if (!Info->IsContainerDeclarationCommand) |
| 115 | return; |
| 116 | StringRef Name = Info->Name; |
| 117 | unsigned DiagSelect = llvm::StringSwitch<unsigned>(Name) |
| 118 | .Case("class", !isClassStructDecl() ? 1 : 0) |
| 119 | .Case("interface", !isObjCInterfaceDecl() ? 2 : 0) |
| 120 | .Case("protocol", !isObjCProtocolDecl() ? 3 : 0) |
| 121 | .Case("struct", !isClassStructDecl() ? 4 : 0) |
| 122 | .Case("union", !isUnionDecl() ? 5 : 0) |
| 123 | .Default(0); |
| 124 | |
| 125 | if (DiagSelect) |
| 126 | Diag(Comment->getLocation(), diag::warn_doc_api_container_decl_mismatch) |
| 127 | << Comment->getCommandMarker() |
| 128 | << (DiagSelect-1) << (DiagSelect-1) |
| 129 | << Comment->getSourceRange(); |
| 130 | } |
| 131 | |
| 132 | void Sema::checkContainerDecl(const BlockCommandComment *Comment) { |
| 133 | const CommandInfo *Info = Traits.getCommandInfo(Comment->getCommandID()); |
| 134 | if (!Info->IsContainerDetailCommand || isContainerDecl()) |
| 135 | return; |
| 136 | StringRef Name = Info->Name; |
| 137 | unsigned DiagSelect = llvm::StringSwitch<unsigned>(Name) |
| 138 | .Case("classdesign", 1) |
| 139 | .Case("coclass", 2) |
| 140 | .Case("dependency", 3) |
| 141 | .Case("helper", 4) |
| 142 | .Case("helperclass", 5) |
| 143 | .Case("helps", 6) |
| 144 | .Case("instancesize", 7) |
| 145 | .Case("ownership", 8) |
| 146 | .Case("performance", 9) |
| 147 | .Case("security", 10) |
| 148 | .Case("superclass", 11) |
| 149 | .Default(0); |
| 150 | |
| 151 | if (DiagSelect) |
| 152 | Diag(Comment->getLocation(), diag::warn_doc_container_decl_mismatch) |
| 153 | << Comment->getCommandMarker() |
| 154 | << (DiagSelect-1) |
| 155 | << Comment->getSourceRange(); |
| 156 | } |
Fariborz Jahanian | 2a268f2 | 2013-03-05 01:05:07 +0000 | [diff] [blame] | 157 | |
Dmitri Gribenko | 7d9b511 | 2012-08-06 19:03:12 +0000 | [diff] [blame] | 158 | void Sema::actOnParamCommandDirectionArg(ParamCommandComment *Command, |
| 159 | SourceLocation ArgLocBegin, |
| 160 | SourceLocation ArgLocEnd, |
| 161 | StringRef Arg) { |
Dmitri Gribenko | a5ef44f | 2012-07-11 21:38:39 +0000 | [diff] [blame] | 162 | ParamCommandComment::PassDirection Direction; |
| 163 | std::string ArgLower = Arg.lower(); |
| 164 | // TODO: optimize: lower Name first (need an API in SmallString for that), |
| 165 | // after that StringSwitch. |
| 166 | if (ArgLower == "[in]") |
| 167 | Direction = ParamCommandComment::In; |
| 168 | else if (ArgLower == "[out]") |
| 169 | Direction = ParamCommandComment::Out; |
| 170 | else if (ArgLower == "[in,out]" || ArgLower == "[out,in]") |
| 171 | Direction = ParamCommandComment::InOut; |
| 172 | else { |
| 173 | // Remove spaces. |
| 174 | std::string::iterator O = ArgLower.begin(); |
| 175 | for (std::string::iterator I = ArgLower.begin(), E = ArgLower.end(); |
| 176 | I != E; ++I) { |
| 177 | const char C = *I; |
| 178 | if (C != ' ' && C != '\n' && C != '\r' && |
| 179 | C != '\t' && C != '\v' && C != '\f') |
| 180 | *O++ = C; |
| 181 | } |
| 182 | ArgLower.resize(O - ArgLower.begin()); |
Dmitri Gribenko | 8d3ba23 | 2012-07-06 00:28:32 +0000 | [diff] [blame] | 183 | |
Dmitri Gribenko | a5ef44f | 2012-07-11 21:38:39 +0000 | [diff] [blame] | 184 | bool RemovingWhitespaceHelped = false; |
| 185 | if (ArgLower == "[in]") { |
| 186 | Direction = ParamCommandComment::In; |
| 187 | RemovingWhitespaceHelped = true; |
| 188 | } else if (ArgLower == "[out]") { |
| 189 | Direction = ParamCommandComment::Out; |
| 190 | RemovingWhitespaceHelped = true; |
| 191 | } else if (ArgLower == "[in,out]" || ArgLower == "[out,in]") { |
| 192 | Direction = ParamCommandComment::InOut; |
| 193 | RemovingWhitespaceHelped = true; |
Dmitri Gribenko | 8d3ba23 | 2012-07-06 00:28:32 +0000 | [diff] [blame] | 194 | } else { |
Dmitri Gribenko | a5ef44f | 2012-07-11 21:38:39 +0000 | [diff] [blame] | 195 | Direction = ParamCommandComment::In; |
| 196 | RemovingWhitespaceHelped = false; |
Dmitri Gribenko | 8d3ba23 | 2012-07-06 00:28:32 +0000 | [diff] [blame] | 197 | } |
Dmitri Gribenko | a5ef44f | 2012-07-11 21:38:39 +0000 | [diff] [blame] | 198 | |
| 199 | SourceRange ArgRange(ArgLocBegin, ArgLocEnd); |
| 200 | if (RemovingWhitespaceHelped) |
| 201 | Diag(ArgLocBegin, diag::warn_doc_param_spaces_in_direction) |
| 202 | << ArgRange |
| 203 | << FixItHint::CreateReplacement( |
| 204 | ArgRange, |
| 205 | ParamCommandComment::getDirectionAsString(Direction)); |
| 206 | else |
| 207 | Diag(ArgLocBegin, diag::warn_doc_param_invalid_direction) |
| 208 | << ArgRange; |
Dmitri Gribenko | 8d3ba23 | 2012-07-06 00:28:32 +0000 | [diff] [blame] | 209 | } |
Dmitri Gribenko | a5ef44f | 2012-07-11 21:38:39 +0000 | [diff] [blame] | 210 | Command->setDirection(Direction, /* Explicit = */ true); |
Dmitri Gribenko | a5ef44f | 2012-07-11 21:38:39 +0000 | [diff] [blame] | 211 | } |
| 212 | |
Dmitri Gribenko | 7d9b511 | 2012-08-06 19:03:12 +0000 | [diff] [blame] | 213 | void Sema::actOnParamCommandParamNameArg(ParamCommandComment *Command, |
| 214 | SourceLocation ArgLocBegin, |
| 215 | SourceLocation ArgLocEnd, |
| 216 | StringRef Arg) { |
Dmitri Gribenko | a5ef44f | 2012-07-11 21:38:39 +0000 | [diff] [blame] | 217 | // Parser will not feed us more arguments than needed. |
Dmitri Gribenko | 0eaf69d | 2012-07-13 19:02:42 +0000 | [diff] [blame] | 218 | assert(Command->getNumArgs() == 0); |
Dmitri Gribenko | a5ef44f | 2012-07-11 21:38:39 +0000 | [diff] [blame] | 219 | |
| 220 | if (!Command->isDirectionExplicit()) { |
| 221 | // User didn't provide a direction argument. |
| 222 | Command->setDirection(ParamCommandComment::In, /* Explicit = */ false); |
| 223 | } |
| 224 | typedef BlockCommandComment::Argument Argument; |
| 225 | Argument *A = new (Allocator) Argument(SourceRange(ArgLocBegin, |
| 226 | ArgLocEnd), |
| 227 | Arg); |
| 228 | Command->setArgs(llvm::makeArrayRef(A, 1)); |
Dmitri Gribenko | 8d3ba23 | 2012-07-06 00:28:32 +0000 | [diff] [blame] | 229 | } |
| 230 | |
Dmitri Gribenko | 7d9b511 | 2012-08-06 19:03:12 +0000 | [diff] [blame] | 231 | void Sema::actOnParamCommandFinish(ParamCommandComment *Command, |
| 232 | ParagraphComment *Paragraph) { |
Dmitri Gribenko | 8d3ba23 | 2012-07-06 00:28:32 +0000 | [diff] [blame] | 233 | Command->setParagraph(Paragraph); |
Dmitri Gribenko | a5ef44f | 2012-07-11 21:38:39 +0000 | [diff] [blame] | 234 | checkBlockCommandEmptyParagraph(Command); |
Dmitri Gribenko | 8d3ba23 | 2012-07-06 00:28:32 +0000 | [diff] [blame] | 235 | } |
| 236 | |
Dmitri Gribenko | 808383d | 2013-03-04 23:06:15 +0000 | [diff] [blame] | 237 | TParamCommandComment *Sema::actOnTParamCommandStart( |
| 238 | SourceLocation LocBegin, |
| 239 | SourceLocation LocEnd, |
| 240 | unsigned CommandID, |
| 241 | CommandMarkerKind CommandMarker) { |
Dmitri Gribenko | 96b0986 | 2012-07-31 22:37:06 +0000 | [diff] [blame] | 242 | TParamCommandComment *Command = |
Dmitri Gribenko | 808383d | 2013-03-04 23:06:15 +0000 | [diff] [blame] | 243 | new (Allocator) TParamCommandComment(LocBegin, LocEnd, CommandID, |
| 244 | CommandMarker); |
Dmitri Gribenko | 96b0986 | 2012-07-31 22:37:06 +0000 | [diff] [blame] | 245 | |
Dmitri Gribenko | 04bf29e | 2012-08-06 21:31:15 +0000 | [diff] [blame] | 246 | if (!isTemplateOrSpecialization()) |
Dmitri Gribenko | 96b0986 | 2012-07-31 22:37:06 +0000 | [diff] [blame] | 247 | Diag(Command->getLocation(), |
| 248 | diag::warn_doc_tparam_not_attached_to_a_template_decl) |
Dmitri Gribenko | 808383d | 2013-03-04 23:06:15 +0000 | [diff] [blame] | 249 | << CommandMarker |
Dmitri Gribenko | e4330a3 | 2012-09-10 20:32:42 +0000 | [diff] [blame] | 250 | << Command->getCommandNameRange(Traits); |
Dmitri Gribenko | 96b0986 | 2012-07-31 22:37:06 +0000 | [diff] [blame] | 251 | |
| 252 | return Command; |
| 253 | } |
| 254 | |
Dmitri Gribenko | 7d9b511 | 2012-08-06 19:03:12 +0000 | [diff] [blame] | 255 | void Sema::actOnTParamCommandParamNameArg(TParamCommandComment *Command, |
| 256 | SourceLocation ArgLocBegin, |
| 257 | SourceLocation ArgLocEnd, |
| 258 | StringRef Arg) { |
Dmitri Gribenko | 96b0986 | 2012-07-31 22:37:06 +0000 | [diff] [blame] | 259 | // Parser will not feed us more arguments than needed. |
| 260 | assert(Command->getNumArgs() == 0); |
| 261 | |
| 262 | typedef BlockCommandComment::Argument Argument; |
| 263 | Argument *A = new (Allocator) Argument(SourceRange(ArgLocBegin, |
| 264 | ArgLocEnd), |
| 265 | Arg); |
| 266 | Command->setArgs(llvm::makeArrayRef(A, 1)); |
| 267 | |
Dmitri Gribenko | 04bf29e | 2012-08-06 21:31:15 +0000 | [diff] [blame] | 268 | if (!isTemplateOrSpecialization()) { |
Dmitri Gribenko | 96b0986 | 2012-07-31 22:37:06 +0000 | [diff] [blame] | 269 | // We already warned that this \\tparam is not attached to a template decl. |
Dmitri Gribenko | 7d9b511 | 2012-08-06 19:03:12 +0000 | [diff] [blame] | 270 | return; |
Dmitri Gribenko | 96b0986 | 2012-07-31 22:37:06 +0000 | [diff] [blame] | 271 | } |
| 272 | |
Dmitri Gribenko | 1ca7ecc | 2012-08-01 23:08:09 +0000 | [diff] [blame] | 273 | const TemplateParameterList *TemplateParameters = |
| 274 | ThisDeclInfo->TemplateParameters; |
Dmitri Gribenko | 96b0986 | 2012-07-31 22:37:06 +0000 | [diff] [blame] | 275 | SmallVector<unsigned, 2> Position; |
| 276 | if (resolveTParamReference(Arg, TemplateParameters, &Position)) { |
| 277 | Command->setPosition(copyArray(llvm::makeArrayRef(Position))); |
| 278 | llvm::StringMap<TParamCommandComment *>::iterator PrevCommandIt = |
| 279 | TemplateParameterDocs.find(Arg); |
| 280 | if (PrevCommandIt != TemplateParameterDocs.end()) { |
| 281 | SourceRange ArgRange(ArgLocBegin, ArgLocEnd); |
| 282 | Diag(ArgLocBegin, diag::warn_doc_tparam_duplicate) |
| 283 | << Arg << ArgRange; |
| 284 | TParamCommandComment *PrevCommand = PrevCommandIt->second; |
| 285 | Diag(PrevCommand->getLocation(), diag::note_doc_tparam_previous) |
| 286 | << PrevCommand->getParamNameRange(); |
| 287 | } |
| 288 | TemplateParameterDocs[Arg] = Command; |
Dmitri Gribenko | 7d9b511 | 2012-08-06 19:03:12 +0000 | [diff] [blame] | 289 | return; |
Dmitri Gribenko | 96b0986 | 2012-07-31 22:37:06 +0000 | [diff] [blame] | 290 | } |
| 291 | |
| 292 | SourceRange ArgRange(ArgLocBegin, ArgLocEnd); |
| 293 | Diag(ArgLocBegin, diag::warn_doc_tparam_not_found) |
| 294 | << Arg << ArgRange; |
| 295 | |
| 296 | if (!TemplateParameters || TemplateParameters->size() == 0) |
Dmitri Gribenko | 7d9b511 | 2012-08-06 19:03:12 +0000 | [diff] [blame] | 297 | return; |
Dmitri Gribenko | 96b0986 | 2012-07-31 22:37:06 +0000 | [diff] [blame] | 298 | |
| 299 | StringRef CorrectedName; |
| 300 | if (TemplateParameters->size() == 1) { |
| 301 | const NamedDecl *Param = TemplateParameters->getParam(0); |
| 302 | const IdentifierInfo *II = Param->getIdentifier(); |
| 303 | if (II) |
| 304 | CorrectedName = II->getName(); |
| 305 | } else { |
| 306 | CorrectedName = correctTypoInTParamReference(Arg, TemplateParameters); |
| 307 | } |
| 308 | |
| 309 | if (!CorrectedName.empty()) { |
| 310 | Diag(ArgLocBegin, diag::note_doc_tparam_name_suggestion) |
| 311 | << CorrectedName |
| 312 | << FixItHint::CreateReplacement(ArgRange, CorrectedName); |
| 313 | } |
| 314 | |
Dmitri Gribenko | 7d9b511 | 2012-08-06 19:03:12 +0000 | [diff] [blame] | 315 | return; |
Dmitri Gribenko | 96b0986 | 2012-07-31 22:37:06 +0000 | [diff] [blame] | 316 | } |
| 317 | |
Dmitri Gribenko | 7d9b511 | 2012-08-06 19:03:12 +0000 | [diff] [blame] | 318 | void Sema::actOnTParamCommandFinish(TParamCommandComment *Command, |
| 319 | ParagraphComment *Paragraph) { |
Dmitri Gribenko | 96b0986 | 2012-07-31 22:37:06 +0000 | [diff] [blame] | 320 | Command->setParagraph(Paragraph); |
| 321 | checkBlockCommandEmptyParagraph(Command); |
Dmitri Gribenko | 96b0986 | 2012-07-31 22:37:06 +0000 | [diff] [blame] | 322 | } |
| 323 | |
Dmitri Gribenko | 8d3ba23 | 2012-07-06 00:28:32 +0000 | [diff] [blame] | 324 | InlineCommandComment *Sema::actOnInlineCommand(SourceLocation CommandLocBegin, |
| 325 | SourceLocation CommandLocEnd, |
Dmitri Gribenko | e4330a3 | 2012-09-10 20:32:42 +0000 | [diff] [blame] | 326 | unsigned CommandID) { |
Dmitri Gribenko | 8d3ba23 | 2012-07-06 00:28:32 +0000 | [diff] [blame] | 327 | ArrayRef<InlineCommandComment::Argument> Args; |
Dmitri Gribenko | e4330a3 | 2012-09-10 20:32:42 +0000 | [diff] [blame] | 328 | StringRef CommandName = Traits.getCommandInfo(CommandID)->Name; |
Dmitri Gribenko | 2d66a50 | 2012-07-23 16:43:01 +0000 | [diff] [blame] | 329 | return new (Allocator) InlineCommandComment( |
| 330 | CommandLocBegin, |
| 331 | CommandLocEnd, |
Dmitri Gribenko | e4330a3 | 2012-09-10 20:32:42 +0000 | [diff] [blame] | 332 | CommandID, |
Dmitri Gribenko | 2d66a50 | 2012-07-23 16:43:01 +0000 | [diff] [blame] | 333 | getInlineCommandRenderKind(CommandName), |
| 334 | Args); |
Dmitri Gribenko | 8d3ba23 | 2012-07-06 00:28:32 +0000 | [diff] [blame] | 335 | } |
| 336 | |
| 337 | InlineCommandComment *Sema::actOnInlineCommand(SourceLocation CommandLocBegin, |
| 338 | SourceLocation CommandLocEnd, |
Dmitri Gribenko | e4330a3 | 2012-09-10 20:32:42 +0000 | [diff] [blame] | 339 | unsigned CommandID, |
Dmitri Gribenko | 8d3ba23 | 2012-07-06 00:28:32 +0000 | [diff] [blame] | 340 | SourceLocation ArgLocBegin, |
| 341 | SourceLocation ArgLocEnd, |
| 342 | StringRef Arg) { |
| 343 | typedef InlineCommandComment::Argument Argument; |
| 344 | Argument *A = new (Allocator) Argument(SourceRange(ArgLocBegin, |
| 345 | ArgLocEnd), |
| 346 | Arg); |
Dmitri Gribenko | e4330a3 | 2012-09-10 20:32:42 +0000 | [diff] [blame] | 347 | StringRef CommandName = Traits.getCommandInfo(CommandID)->Name; |
Dmitri Gribenko | 8d3ba23 | 2012-07-06 00:28:32 +0000 | [diff] [blame] | 348 | |
Dmitri Gribenko | 2d66a50 | 2012-07-23 16:43:01 +0000 | [diff] [blame] | 349 | return new (Allocator) InlineCommandComment( |
| 350 | CommandLocBegin, |
| 351 | CommandLocEnd, |
Dmitri Gribenko | e4330a3 | 2012-09-10 20:32:42 +0000 | [diff] [blame] | 352 | CommandID, |
Dmitri Gribenko | 2d66a50 | 2012-07-23 16:43:01 +0000 | [diff] [blame] | 353 | getInlineCommandRenderKind(CommandName), |
| 354 | llvm::makeArrayRef(A, 1)); |
Dmitri Gribenko | 8d3ba23 | 2012-07-06 00:28:32 +0000 | [diff] [blame] | 355 | } |
| 356 | |
| 357 | InlineContentComment *Sema::actOnUnknownCommand(SourceLocation LocBegin, |
| 358 | SourceLocation LocEnd, |
Dmitri Gribenko | b0b8a96 | 2012-09-11 19:22:03 +0000 | [diff] [blame] | 359 | StringRef CommandName) { |
| 360 | unsigned CommandID = Traits.registerUnknownCommand(CommandName)->getID(); |
| 361 | return actOnUnknownCommand(LocBegin, LocEnd, CommandID); |
| 362 | } |
| 363 | |
| 364 | InlineContentComment *Sema::actOnUnknownCommand(SourceLocation LocBegin, |
| 365 | SourceLocation LocEnd, |
| 366 | unsigned CommandID) { |
Dmitri Gribenko | 8d3ba23 | 2012-07-06 00:28:32 +0000 | [diff] [blame] | 367 | ArrayRef<InlineCommandComment::Argument> Args; |
Dmitri Gribenko | 2d66a50 | 2012-07-23 16:43:01 +0000 | [diff] [blame] | 368 | return new (Allocator) InlineCommandComment( |
Dmitri Gribenko | e4330a3 | 2012-09-10 20:32:42 +0000 | [diff] [blame] | 369 | LocBegin, LocEnd, CommandID, |
Dmitri Gribenko | 2d66a50 | 2012-07-23 16:43:01 +0000 | [diff] [blame] | 370 | InlineCommandComment::RenderNormal, |
| 371 | Args); |
Dmitri Gribenko | 8d3ba23 | 2012-07-06 00:28:32 +0000 | [diff] [blame] | 372 | } |
| 373 | |
| 374 | TextComment *Sema::actOnText(SourceLocation LocBegin, |
| 375 | SourceLocation LocEnd, |
| 376 | StringRef Text) { |
| 377 | return new (Allocator) TextComment(LocBegin, LocEnd, Text); |
| 378 | } |
| 379 | |
| 380 | VerbatimBlockComment *Sema::actOnVerbatimBlockStart(SourceLocation Loc, |
Dmitri Gribenko | e4330a3 | 2012-09-10 20:32:42 +0000 | [diff] [blame] | 381 | unsigned CommandID) { |
| 382 | StringRef CommandName = Traits.getCommandInfo(CommandID)->Name; |
Dmitri Gribenko | 8d3ba23 | 2012-07-06 00:28:32 +0000 | [diff] [blame] | 383 | return new (Allocator) VerbatimBlockComment( |
| 384 | Loc, |
Dmitri Gribenko | e4330a3 | 2012-09-10 20:32:42 +0000 | [diff] [blame] | 385 | Loc.getLocWithOffset(1 + CommandName.size()), |
| 386 | CommandID); |
Dmitri Gribenko | 8d3ba23 | 2012-07-06 00:28:32 +0000 | [diff] [blame] | 387 | } |
| 388 | |
| 389 | VerbatimBlockLineComment *Sema::actOnVerbatimBlockLine(SourceLocation Loc, |
| 390 | StringRef Text) { |
| 391 | return new (Allocator) VerbatimBlockLineComment(Loc, Text); |
| 392 | } |
| 393 | |
Dmitri Gribenko | 7d9b511 | 2012-08-06 19:03:12 +0000 | [diff] [blame] | 394 | void Sema::actOnVerbatimBlockFinish( |
Dmitri Gribenko | 8d3ba23 | 2012-07-06 00:28:32 +0000 | [diff] [blame] | 395 | VerbatimBlockComment *Block, |
| 396 | SourceLocation CloseNameLocBegin, |
| 397 | StringRef CloseName, |
| 398 | ArrayRef<VerbatimBlockLineComment *> Lines) { |
| 399 | Block->setCloseName(CloseName, CloseNameLocBegin); |
| 400 | Block->setLines(Lines); |
Dmitri Gribenko | 8d3ba23 | 2012-07-06 00:28:32 +0000 | [diff] [blame] | 401 | } |
| 402 | |
| 403 | VerbatimLineComment *Sema::actOnVerbatimLine(SourceLocation LocBegin, |
Dmitri Gribenko | e4330a3 | 2012-09-10 20:32:42 +0000 | [diff] [blame] | 404 | unsigned CommandID, |
Dmitri Gribenko | 8d3ba23 | 2012-07-06 00:28:32 +0000 | [diff] [blame] | 405 | SourceLocation TextBegin, |
| 406 | StringRef Text) { |
Fariborz Jahanian | bca9788 | 2013-03-05 19:40:47 +0000 | [diff] [blame] | 407 | VerbatimLineComment *VL = new (Allocator) VerbatimLineComment( |
Dmitri Gribenko | 8d3ba23 | 2012-07-06 00:28:32 +0000 | [diff] [blame] | 408 | LocBegin, |
| 409 | TextBegin.getLocWithOffset(Text.size()), |
Dmitri Gribenko | e4330a3 | 2012-09-10 20:32:42 +0000 | [diff] [blame] | 410 | CommandID, |
Dmitri Gribenko | 8d3ba23 | 2012-07-06 00:28:32 +0000 | [diff] [blame] | 411 | TextBegin, |
| 412 | Text); |
Fariborz Jahanian | bca9788 | 2013-03-05 19:40:47 +0000 | [diff] [blame] | 413 | checkFunctionDeclVerbatimLine(VL); |
Fariborz Jahanian | 28c1cd2 | 2013-03-07 23:33:11 +0000 | [diff] [blame] | 414 | checkContainerDeclVerbatimLine(VL); |
Fariborz Jahanian | bca9788 | 2013-03-05 19:40:47 +0000 | [diff] [blame] | 415 | return VL; |
Dmitri Gribenko | 8d3ba23 | 2012-07-06 00:28:32 +0000 | [diff] [blame] | 416 | } |
| 417 | |
Dmitri Gribenko | 3f38bf2 | 2012-07-13 00:44:24 +0000 | [diff] [blame] | 418 | HTMLStartTagComment *Sema::actOnHTMLStartTagStart(SourceLocation LocBegin, |
| 419 | StringRef TagName) { |
| 420 | return new (Allocator) HTMLStartTagComment(LocBegin, TagName); |
Dmitri Gribenko | 8d3ba23 | 2012-07-06 00:28:32 +0000 | [diff] [blame] | 421 | } |
| 422 | |
Dmitri Gribenko | 7d9b511 | 2012-08-06 19:03:12 +0000 | [diff] [blame] | 423 | void Sema::actOnHTMLStartTagFinish( |
Dmitri Gribenko | 3f38bf2 | 2012-07-13 00:44:24 +0000 | [diff] [blame] | 424 | HTMLStartTagComment *Tag, |
| 425 | ArrayRef<HTMLStartTagComment::Attribute> Attrs, |
Dmitri Gribenko | a5ef44f | 2012-07-11 21:38:39 +0000 | [diff] [blame] | 426 | SourceLocation GreaterLoc, |
| 427 | bool IsSelfClosing) { |
Dmitri Gribenko | 8d3ba23 | 2012-07-06 00:28:32 +0000 | [diff] [blame] | 428 | Tag->setAttrs(Attrs); |
| 429 | Tag->setGreaterLoc(GreaterLoc); |
Dmitri Gribenko | a5ef44f | 2012-07-11 21:38:39 +0000 | [diff] [blame] | 430 | if (IsSelfClosing) |
| 431 | Tag->setSelfClosing(); |
Dmitri Gribenko | 3f38bf2 | 2012-07-13 00:44:24 +0000 | [diff] [blame] | 432 | else if (!isHTMLEndTagForbidden(Tag->getTagName())) |
Dmitri Gribenko | a5ef44f | 2012-07-11 21:38:39 +0000 | [diff] [blame] | 433 | HTMLOpenTags.push_back(Tag); |
Dmitri Gribenko | 8d3ba23 | 2012-07-06 00:28:32 +0000 | [diff] [blame] | 434 | } |
| 435 | |
Dmitri Gribenko | 3f38bf2 | 2012-07-13 00:44:24 +0000 | [diff] [blame] | 436 | HTMLEndTagComment *Sema::actOnHTMLEndTag(SourceLocation LocBegin, |
| 437 | SourceLocation LocEnd, |
| 438 | StringRef TagName) { |
| 439 | HTMLEndTagComment *HET = |
| 440 | new (Allocator) HTMLEndTagComment(LocBegin, LocEnd, TagName); |
| 441 | if (isHTMLEndTagForbidden(TagName)) { |
| 442 | Diag(HET->getLocation(), diag::warn_doc_html_end_forbidden) |
| 443 | << TagName << HET->getSourceRange(); |
| 444 | return HET; |
Dmitri Gribenko | 3d98698 | 2012-07-12 23:37:09 +0000 | [diff] [blame] | 445 | } |
| 446 | |
Dmitri Gribenko | a5ef44f | 2012-07-11 21:38:39 +0000 | [diff] [blame] | 447 | bool FoundOpen = false; |
Dmitri Gribenko | 3f38bf2 | 2012-07-13 00:44:24 +0000 | [diff] [blame] | 448 | for (SmallVectorImpl<HTMLStartTagComment *>::const_reverse_iterator |
Dmitri Gribenko | a5ef44f | 2012-07-11 21:38:39 +0000 | [diff] [blame] | 449 | I = HTMLOpenTags.rbegin(), E = HTMLOpenTags.rend(); |
| 450 | I != E; ++I) { |
| 451 | if ((*I)->getTagName() == TagName) { |
| 452 | FoundOpen = true; |
| 453 | break; |
| 454 | } |
| 455 | } |
| 456 | if (!FoundOpen) { |
Dmitri Gribenko | 3f38bf2 | 2012-07-13 00:44:24 +0000 | [diff] [blame] | 457 | Diag(HET->getLocation(), diag::warn_doc_html_end_unbalanced) |
| 458 | << HET->getSourceRange(); |
| 459 | return HET; |
Dmitri Gribenko | a5ef44f | 2012-07-11 21:38:39 +0000 | [diff] [blame] | 460 | } |
| 461 | |
| 462 | while (!HTMLOpenTags.empty()) { |
Dmitri Gribenko | 3f38bf2 | 2012-07-13 00:44:24 +0000 | [diff] [blame] | 463 | const HTMLStartTagComment *HST = HTMLOpenTags.back(); |
Dmitri Gribenko | a5ef44f | 2012-07-11 21:38:39 +0000 | [diff] [blame] | 464 | HTMLOpenTags.pop_back(); |
Dmitri Gribenko | 3f38bf2 | 2012-07-13 00:44:24 +0000 | [diff] [blame] | 465 | StringRef LastNotClosedTagName = HST->getTagName(); |
Dmitri Gribenko | a5ef44f | 2012-07-11 21:38:39 +0000 | [diff] [blame] | 466 | if (LastNotClosedTagName == TagName) |
| 467 | break; |
| 468 | |
Dmitri Gribenko | 3f38bf2 | 2012-07-13 00:44:24 +0000 | [diff] [blame] | 469 | if (isHTMLEndTagOptional(LastNotClosedTagName)) |
Dmitri Gribenko | a5ef44f | 2012-07-11 21:38:39 +0000 | [diff] [blame] | 470 | continue; |
| 471 | |
| 472 | bool OpenLineInvalid; |
| 473 | const unsigned OpenLine = SourceMgr.getPresumedLineNumber( |
Dmitri Gribenko | 3f38bf2 | 2012-07-13 00:44:24 +0000 | [diff] [blame] | 474 | HST->getLocation(), |
Dmitri Gribenko | a5ef44f | 2012-07-11 21:38:39 +0000 | [diff] [blame] | 475 | &OpenLineInvalid); |
| 476 | bool CloseLineInvalid; |
| 477 | const unsigned CloseLine = SourceMgr.getPresumedLineNumber( |
Dmitri Gribenko | 3f38bf2 | 2012-07-13 00:44:24 +0000 | [diff] [blame] | 478 | HET->getLocation(), |
Dmitri Gribenko | a5ef44f | 2012-07-11 21:38:39 +0000 | [diff] [blame] | 479 | &CloseLineInvalid); |
| 480 | |
| 481 | if (OpenLineInvalid || CloseLineInvalid || OpenLine == CloseLine) |
Dmitri Gribenko | 3f38bf2 | 2012-07-13 00:44:24 +0000 | [diff] [blame] | 482 | Diag(HST->getLocation(), diag::warn_doc_html_start_end_mismatch) |
| 483 | << HST->getTagName() << HET->getTagName() |
| 484 | << HST->getSourceRange() << HET->getSourceRange(); |
Dmitri Gribenko | a5ef44f | 2012-07-11 21:38:39 +0000 | [diff] [blame] | 485 | else { |
Dmitri Gribenko | 3f38bf2 | 2012-07-13 00:44:24 +0000 | [diff] [blame] | 486 | Diag(HST->getLocation(), diag::warn_doc_html_start_end_mismatch) |
| 487 | << HST->getTagName() << HET->getTagName() |
| 488 | << HST->getSourceRange(); |
| 489 | Diag(HET->getLocation(), diag::note_doc_html_end_tag) |
| 490 | << HET->getSourceRange(); |
Dmitri Gribenko | a5ef44f | 2012-07-11 21:38:39 +0000 | [diff] [blame] | 491 | } |
| 492 | } |
| 493 | |
Dmitri Gribenko | 3f38bf2 | 2012-07-13 00:44:24 +0000 | [diff] [blame] | 494 | return HET; |
Dmitri Gribenko | 8d3ba23 | 2012-07-06 00:28:32 +0000 | [diff] [blame] | 495 | } |
| 496 | |
| 497 | FullComment *Sema::actOnFullComment( |
| 498 | ArrayRef<BlockContentComment *> Blocks) { |
Fariborz Jahanian | 749ace6 | 2012-10-11 23:52:50 +0000 | [diff] [blame] | 499 | FullComment *FC = new (Allocator) FullComment(Blocks, ThisDeclInfo); |
Dmitri Gribenko | 9edd2c8 | 2012-08-24 17:45:39 +0000 | [diff] [blame] | 500 | resolveParamCommandIndexes(FC); |
| 501 | return FC; |
Dmitri Gribenko | 8d3ba23 | 2012-07-06 00:28:32 +0000 | [diff] [blame] | 502 | } |
| 503 | |
Dmitri Gribenko | a5ef44f | 2012-07-11 21:38:39 +0000 | [diff] [blame] | 504 | void Sema::checkBlockCommandEmptyParagraph(BlockCommandComment *Command) { |
Dmitri Gribenko | abcf0dc | 2012-09-13 20:36:01 +0000 | [diff] [blame] | 505 | if (Traits.getCommandInfo(Command->getCommandID())->IsEmptyParagraphAllowed) |
| 506 | return; |
| 507 | |
Dmitri Gribenko | a5ef44f | 2012-07-11 21:38:39 +0000 | [diff] [blame] | 508 | ParagraphComment *Paragraph = Command->getParagraph(); |
| 509 | if (Paragraph->isWhitespace()) { |
| 510 | SourceLocation DiagLoc; |
Dmitri Gribenko | 0eaf69d | 2012-07-13 19:02:42 +0000 | [diff] [blame] | 511 | if (Command->getNumArgs() > 0) |
| 512 | DiagLoc = Command->getArgRange(Command->getNumArgs() - 1).getEnd(); |
Dmitri Gribenko | a5ef44f | 2012-07-11 21:38:39 +0000 | [diff] [blame] | 513 | if (!DiagLoc.isValid()) |
Dmitri Gribenko | e4330a3 | 2012-09-10 20:32:42 +0000 | [diff] [blame] | 514 | DiagLoc = Command->getCommandNameRange(Traits).getEnd(); |
Dmitri Gribenko | a5ef44f | 2012-07-11 21:38:39 +0000 | [diff] [blame] | 515 | Diag(DiagLoc, diag::warn_doc_block_command_empty_paragraph) |
Dmitri Gribenko | 808383d | 2013-03-04 23:06:15 +0000 | [diff] [blame] | 516 | << Command->getCommandMarker() |
Dmitri Gribenko | e4330a3 | 2012-09-10 20:32:42 +0000 | [diff] [blame] | 517 | << Command->getCommandName(Traits) |
Dmitri Gribenko | a5ef44f | 2012-07-11 21:38:39 +0000 | [diff] [blame] | 518 | << Command->getSourceRange(); |
| 519 | } |
| 520 | } |
| 521 | |
Dmitri Gribenko | 89ab7d0 | 2012-08-03 21:15:32 +0000 | [diff] [blame] | 522 | void Sema::checkReturnsCommand(const BlockCommandComment *Command) { |
Dmitri Gribenko | e4330a3 | 2012-09-10 20:32:42 +0000 | [diff] [blame] | 523 | if (!Traits.getCommandInfo(Command->getCommandID())->IsReturnsCommand) |
Dmitri Gribenko | 89ab7d0 | 2012-08-03 21:15:32 +0000 | [diff] [blame] | 524 | return; |
| 525 | if (isFunctionDecl()) { |
| 526 | if (ThisDeclInfo->ResultType->isVoidType()) { |
| 527 | unsigned DiagKind; |
Fariborz Jahanian | bf967be | 2012-10-10 18:34:52 +0000 | [diff] [blame] | 528 | switch (ThisDeclInfo->CommentDecl->getKind()) { |
Dmitri Gribenko | 89ab7d0 | 2012-08-03 21:15:32 +0000 | [diff] [blame] | 529 | default: |
Dmitri Gribenko | 88815f3 | 2012-08-06 16:29:26 +0000 | [diff] [blame] | 530 | if (ThisDeclInfo->IsObjCMethod) |
| 531 | DiagKind = 3; |
| 532 | else |
| 533 | DiagKind = 0; |
Dmitri Gribenko | 89ab7d0 | 2012-08-03 21:15:32 +0000 | [diff] [blame] | 534 | break; |
| 535 | case Decl::CXXConstructor: |
| 536 | DiagKind = 1; |
| 537 | break; |
| 538 | case Decl::CXXDestructor: |
| 539 | DiagKind = 2; |
| 540 | break; |
| 541 | } |
| 542 | Diag(Command->getLocation(), |
| 543 | diag::warn_doc_returns_attached_to_a_void_function) |
Dmitri Gribenko | 808383d | 2013-03-04 23:06:15 +0000 | [diff] [blame] | 544 | << Command->getCommandMarker() |
Dmitri Gribenko | e4330a3 | 2012-09-10 20:32:42 +0000 | [diff] [blame] | 545 | << Command->getCommandName(Traits) |
Dmitri Gribenko | 89ab7d0 | 2012-08-03 21:15:32 +0000 | [diff] [blame] | 546 | << DiagKind |
| 547 | << Command->getSourceRange(); |
| 548 | } |
| 549 | return; |
| 550 | } |
Fariborz Jahanian | 664e860 | 2013-02-27 00:46:06 +0000 | [diff] [blame] | 551 | else if (isObjCPropertyDecl()) |
| 552 | return; |
| 553 | |
Dmitri Gribenko | 89ab7d0 | 2012-08-03 21:15:32 +0000 | [diff] [blame] | 554 | Diag(Command->getLocation(), |
| 555 | diag::warn_doc_returns_not_attached_to_a_function_decl) |
Dmitri Gribenko | 808383d | 2013-03-04 23:06:15 +0000 | [diff] [blame] | 556 | << Command->getCommandMarker() |
Dmitri Gribenko | e4330a3 | 2012-09-10 20:32:42 +0000 | [diff] [blame] | 557 | << Command->getCommandName(Traits) |
Dmitri Gribenko | 89ab7d0 | 2012-08-03 21:15:32 +0000 | [diff] [blame] | 558 | << Command->getSourceRange(); |
| 559 | } |
| 560 | |
Dmitri Gribenko | 9443c57 | 2012-08-06 17:08:27 +0000 | [diff] [blame] | 561 | void Sema::checkBlockCommandDuplicate(const BlockCommandComment *Command) { |
Dmitri Gribenko | e4330a3 | 2012-09-10 20:32:42 +0000 | [diff] [blame] | 562 | const CommandInfo *Info = Traits.getCommandInfo(Command->getCommandID()); |
Dmitri Gribenko | 9443c57 | 2012-08-06 17:08:27 +0000 | [diff] [blame] | 563 | const BlockCommandComment *PrevCommand = NULL; |
Dmitri Gribenko | e4330a3 | 2012-09-10 20:32:42 +0000 | [diff] [blame] | 564 | if (Info->IsBriefCommand) { |
Dmitri Gribenko | 9443c57 | 2012-08-06 17:08:27 +0000 | [diff] [blame] | 565 | if (!BriefCommand) { |
| 566 | BriefCommand = Command; |
| 567 | return; |
| 568 | } |
| 569 | PrevCommand = BriefCommand; |
Dmitri Gribenko | e4330a3 | 2012-09-10 20:32:42 +0000 | [diff] [blame] | 570 | } else if (Info->IsReturnsCommand) { |
Dmitri Gribenko | 9443c57 | 2012-08-06 17:08:27 +0000 | [diff] [blame] | 571 | if (!ReturnsCommand) { |
| 572 | ReturnsCommand = Command; |
| 573 | return; |
| 574 | } |
| 575 | PrevCommand = ReturnsCommand; |
Fariborz Jahanian | f843a58 | 2013-01-31 23:12:39 +0000 | [diff] [blame] | 576 | } else if (Info->IsHeaderfileCommand) { |
| 577 | if (!HeaderfileCommand) { |
| 578 | HeaderfileCommand = Command; |
| 579 | return; |
| 580 | } |
| 581 | PrevCommand = HeaderfileCommand; |
Dmitri Gribenko | 9443c57 | 2012-08-06 17:08:27 +0000 | [diff] [blame] | 582 | } else { |
| 583 | // We don't want to check this command for duplicates. |
| 584 | return; |
| 585 | } |
Dmitri Gribenko | e4330a3 | 2012-09-10 20:32:42 +0000 | [diff] [blame] | 586 | StringRef CommandName = Command->getCommandName(Traits); |
| 587 | StringRef PrevCommandName = PrevCommand->getCommandName(Traits); |
Dmitri Gribenko | 9443c57 | 2012-08-06 17:08:27 +0000 | [diff] [blame] | 588 | Diag(Command->getLocation(), diag::warn_doc_block_command_duplicate) |
Dmitri Gribenko | 808383d | 2013-03-04 23:06:15 +0000 | [diff] [blame] | 589 | << Command->getCommandMarker() |
Dmitri Gribenko | e4330a3 | 2012-09-10 20:32:42 +0000 | [diff] [blame] | 590 | << CommandName |
Dmitri Gribenko | 9443c57 | 2012-08-06 17:08:27 +0000 | [diff] [blame] | 591 | << Command->getSourceRange(); |
Dmitri Gribenko | e4330a3 | 2012-09-10 20:32:42 +0000 | [diff] [blame] | 592 | if (CommandName == PrevCommandName) |
Dmitri Gribenko | 9443c57 | 2012-08-06 17:08:27 +0000 | [diff] [blame] | 593 | Diag(PrevCommand->getLocation(), diag::note_doc_block_command_previous) |
Dmitri Gribenko | 808383d | 2013-03-04 23:06:15 +0000 | [diff] [blame] | 594 | << PrevCommand->getCommandMarker() |
Dmitri Gribenko | e4330a3 | 2012-09-10 20:32:42 +0000 | [diff] [blame] | 595 | << PrevCommandName |
| 596 | << PrevCommand->getSourceRange(); |
Dmitri Gribenko | 9443c57 | 2012-08-06 17:08:27 +0000 | [diff] [blame] | 597 | else |
| 598 | Diag(PrevCommand->getLocation(), |
| 599 | diag::note_doc_block_command_previous_alias) |
Dmitri Gribenko | 808383d | 2013-03-04 23:06:15 +0000 | [diff] [blame] | 600 | << PrevCommand->getCommandMarker() |
Dmitri Gribenko | e4330a3 | 2012-09-10 20:32:42 +0000 | [diff] [blame] | 601 | << PrevCommandName |
| 602 | << CommandName; |
Dmitri Gribenko | 9443c57 | 2012-08-06 17:08:27 +0000 | [diff] [blame] | 603 | } |
| 604 | |
Dmitri Gribenko | 0bd9838 | 2012-09-22 21:47:50 +0000 | [diff] [blame] | 605 | void Sema::checkDeprecatedCommand(const BlockCommandComment *Command) { |
| 606 | if (!Traits.getCommandInfo(Command->getCommandID())->IsDeprecatedCommand) |
| 607 | return; |
| 608 | |
Fariborz Jahanian | bf967be | 2012-10-10 18:34:52 +0000 | [diff] [blame] | 609 | const Decl *D = ThisDeclInfo->CommentDecl; |
Dmitri Gribenko | 0bd9838 | 2012-09-22 21:47:50 +0000 | [diff] [blame] | 610 | if (!D) |
| 611 | return; |
| 612 | |
| 613 | if (D->hasAttr<DeprecatedAttr>() || |
| 614 | D->hasAttr<AvailabilityAttr>() || |
| 615 | D->hasAttr<UnavailableAttr>()) |
| 616 | return; |
| 617 | |
| 618 | Diag(Command->getLocation(), |
| 619 | diag::warn_doc_deprecated_not_sync) |
| 620 | << Command->getSourceRange(); |
| 621 | |
| 622 | // Try to emit a fixit with a deprecation attribute. |
| 623 | if (const FunctionDecl *FD = dyn_cast<FunctionDecl>(D)) { |
| 624 | // Don't emit a Fix-It for non-member function definitions. GCC does not |
| 625 | // accept attributes on them. |
| 626 | const DeclContext *Ctx = FD->getDeclContext(); |
| 627 | if ((!Ctx || !Ctx->isRecord()) && |
| 628 | FD->doesThisDeclarationHaveABody()) |
| 629 | return; |
| 630 | |
Dmitri Gribenko | 1952354 | 2012-09-29 11:40:46 +0000 | [diff] [blame] | 631 | StringRef AttributeSpelling = "__attribute__((deprecated))"; |
| 632 | if (PP) { |
| 633 | TokenValue Tokens[] = { |
| 634 | tok::kw___attribute, tok::l_paren, tok::l_paren, |
| 635 | PP->getIdentifierInfo("deprecated"), |
| 636 | tok::r_paren, tok::r_paren |
| 637 | }; |
| 638 | StringRef MacroName = PP->getLastMacroWithSpelling(FD->getLocation(), |
| 639 | Tokens); |
| 640 | if (!MacroName.empty()) |
| 641 | AttributeSpelling = MacroName; |
| 642 | } |
| 643 | |
| 644 | SmallString<64> TextToInsert(" "); |
| 645 | TextToInsert += AttributeSpelling; |
Dmitri Gribenko | 0bd9838 | 2012-09-22 21:47:50 +0000 | [diff] [blame] | 646 | Diag(FD->getLocEnd(), |
| 647 | diag::note_add_deprecation_attr) |
| 648 | << FixItHint::CreateInsertion(FD->getLocEnd().getLocWithOffset(1), |
Dmitri Gribenko | 1952354 | 2012-09-29 11:40:46 +0000 | [diff] [blame] | 649 | TextToInsert); |
Dmitri Gribenko | 0bd9838 | 2012-09-22 21:47:50 +0000 | [diff] [blame] | 650 | } |
| 651 | } |
| 652 | |
Dmitri Gribenko | 9edd2c8 | 2012-08-24 17:45:39 +0000 | [diff] [blame] | 653 | void Sema::resolveParamCommandIndexes(const FullComment *FC) { |
| 654 | if (!isFunctionDecl()) { |
| 655 | // We already warned that \\param commands are not attached to a function |
| 656 | // decl. |
| 657 | return; |
| 658 | } |
| 659 | |
Dmitri Gribenko | cfa88f8 | 2013-01-12 19:30:44 +0000 | [diff] [blame] | 660 | SmallVector<ParamCommandComment *, 8> UnresolvedParamCommands; |
Dmitri Gribenko | 9edd2c8 | 2012-08-24 17:45:39 +0000 | [diff] [blame] | 661 | |
| 662 | // Comment AST nodes that correspond to \c ParamVars for which we have |
| 663 | // found a \\param command or NULL if no documentation was found so far. |
Dmitri Gribenko | cfa88f8 | 2013-01-12 19:30:44 +0000 | [diff] [blame] | 664 | SmallVector<ParamCommandComment *, 8> ParamVarDocs; |
Dmitri Gribenko | 9edd2c8 | 2012-08-24 17:45:39 +0000 | [diff] [blame] | 665 | |
| 666 | ArrayRef<const ParmVarDecl *> ParamVars = getParamVars(); |
| 667 | ParamVarDocs.resize(ParamVars.size(), NULL); |
| 668 | |
| 669 | // First pass over all \\param commands: resolve all parameter names. |
| 670 | for (Comment::child_iterator I = FC->child_begin(), E = FC->child_end(); |
| 671 | I != E; ++I) { |
| 672 | ParamCommandComment *PCC = dyn_cast<ParamCommandComment>(*I); |
| 673 | if (!PCC || !PCC->hasParamName()) |
| 674 | continue; |
Fariborz Jahanian | 262e60c | 2012-10-18 21:42:42 +0000 | [diff] [blame] | 675 | StringRef ParamName = PCC->getParamNameAsWritten(); |
Dmitri Gribenko | 9edd2c8 | 2012-08-24 17:45:39 +0000 | [diff] [blame] | 676 | |
| 677 | // Check that referenced parameter name is in the function decl. |
| 678 | const unsigned ResolvedParamIndex = resolveParmVarReference(ParamName, |
| 679 | ParamVars); |
| 680 | if (ResolvedParamIndex == ParamCommandComment::InvalidParamIndex) { |
| 681 | UnresolvedParamCommands.push_back(PCC); |
| 682 | continue; |
| 683 | } |
| 684 | PCC->setParamIndex(ResolvedParamIndex); |
| 685 | if (ParamVarDocs[ResolvedParamIndex]) { |
| 686 | SourceRange ArgRange = PCC->getParamNameRange(); |
| 687 | Diag(ArgRange.getBegin(), diag::warn_doc_param_duplicate) |
| 688 | << ParamName << ArgRange; |
| 689 | ParamCommandComment *PrevCommand = ParamVarDocs[ResolvedParamIndex]; |
| 690 | Diag(PrevCommand->getLocation(), diag::note_doc_param_previous) |
| 691 | << PrevCommand->getParamNameRange(); |
| 692 | } |
| 693 | ParamVarDocs[ResolvedParamIndex] = PCC; |
| 694 | } |
| 695 | |
| 696 | // Find parameter declarations that have no corresponding \\param. |
Dmitri Gribenko | cfa88f8 | 2013-01-12 19:30:44 +0000 | [diff] [blame] | 697 | SmallVector<const ParmVarDecl *, 8> OrphanedParamDecls; |
Dmitri Gribenko | 9edd2c8 | 2012-08-24 17:45:39 +0000 | [diff] [blame] | 698 | for (unsigned i = 0, e = ParamVarDocs.size(); i != e; ++i) { |
| 699 | if (!ParamVarDocs[i]) |
| 700 | OrphanedParamDecls.push_back(ParamVars[i]); |
| 701 | } |
| 702 | |
| 703 | // Second pass over unresolved \\param commands: do typo correction. |
| 704 | // Suggest corrections from a set of parameter declarations that have no |
| 705 | // corresponding \\param. |
| 706 | for (unsigned i = 0, e = UnresolvedParamCommands.size(); i != e; ++i) { |
| 707 | const ParamCommandComment *PCC = UnresolvedParamCommands[i]; |
| 708 | |
| 709 | SourceRange ArgRange = PCC->getParamNameRange(); |
Fariborz Jahanian | 262e60c | 2012-10-18 21:42:42 +0000 | [diff] [blame] | 710 | StringRef ParamName = PCC->getParamNameAsWritten(); |
Dmitri Gribenko | 9edd2c8 | 2012-08-24 17:45:39 +0000 | [diff] [blame] | 711 | Diag(ArgRange.getBegin(), diag::warn_doc_param_not_found) |
| 712 | << ParamName << ArgRange; |
| 713 | |
| 714 | // All parameters documented -- can't suggest a correction. |
| 715 | if (OrphanedParamDecls.size() == 0) |
| 716 | continue; |
| 717 | |
| 718 | unsigned CorrectedParamIndex = ParamCommandComment::InvalidParamIndex; |
| 719 | if (OrphanedParamDecls.size() == 1) { |
| 720 | // If one parameter is not documented then that parameter is the only |
| 721 | // possible suggestion. |
| 722 | CorrectedParamIndex = 0; |
| 723 | } else { |
| 724 | // Do typo correction. |
| 725 | CorrectedParamIndex = correctTypoInParmVarReference(ParamName, |
| 726 | OrphanedParamDecls); |
| 727 | } |
| 728 | if (CorrectedParamIndex != ParamCommandComment::InvalidParamIndex) { |
| 729 | const ParmVarDecl *CorrectedPVD = OrphanedParamDecls[CorrectedParamIndex]; |
| 730 | if (const IdentifierInfo *CorrectedII = CorrectedPVD->getIdentifier()) |
| 731 | Diag(ArgRange.getBegin(), diag::note_doc_param_name_suggestion) |
| 732 | << CorrectedII->getName() |
| 733 | << FixItHint::CreateReplacement(ArgRange, CorrectedII->getName()); |
| 734 | } |
| 735 | } |
| 736 | } |
| 737 | |
Dmitri Gribenko | 8487c52 | 2012-07-23 17:40:30 +0000 | [diff] [blame] | 738 | bool Sema::isFunctionDecl() { |
Dmitri Gribenko | 1ca7ecc | 2012-08-01 23:08:09 +0000 | [diff] [blame] | 739 | if (!ThisDeclInfo) |
| 740 | return false; |
| 741 | if (!ThisDeclInfo->IsFilled) |
Dmitri Gribenko | 00c59f7 | 2012-07-24 20:58:46 +0000 | [diff] [blame] | 742 | inspectThisDecl(); |
Dmitri Gribenko | af19a6a | 2012-08-02 21:45:39 +0000 | [diff] [blame] | 743 | return ThisDeclInfo->getKind() == DeclInfo::FunctionKind; |
Dmitri Gribenko | 8487c52 | 2012-07-23 17:40:30 +0000 | [diff] [blame] | 744 | } |
Fariborz Jahanian | 99a7057 | 2013-03-05 22:46:07 +0000 | [diff] [blame] | 745 | |
Fariborz Jahanian | eb8f69f | 2013-03-05 23:20:29 +0000 | [diff] [blame] | 746 | bool Sema::isAnyFunctionDecl() { |
| 747 | return isFunctionDecl() && ThisDeclInfo->CurrentDecl && |
| 748 | isa<FunctionDecl>(ThisDeclInfo->CurrentDecl); |
| 749 | } |
| 750 | |
Fariborz Jahanian | 99a7057 | 2013-03-05 22:46:07 +0000 | [diff] [blame] | 751 | bool Sema::isObjCMethodDecl() { |
| 752 | return isFunctionDecl() && ThisDeclInfo->CurrentDecl && |
| 753 | isa<ObjCMethodDecl>(ThisDeclInfo->CurrentDecl); |
| 754 | } |
Fariborz Jahanian | 664e860 | 2013-02-27 00:46:06 +0000 | [diff] [blame] | 755 | |
Fariborz Jahanian | 99a7057 | 2013-03-05 22:46:07 +0000 | [diff] [blame] | 756 | /// isFunctionPointerVarDecl - returns 'true' if declaration is a pointer to |
| 757 | /// function decl. |
| 758 | bool Sema::isFunctionPointerVarDecl() { |
Fariborz Jahanian | bca9788 | 2013-03-05 19:40:47 +0000 | [diff] [blame] | 759 | if (!ThisDeclInfo) |
| 760 | return false; |
| 761 | if (!ThisDeclInfo->IsFilled) |
| 762 | inspectThisDecl(); |
| 763 | if (ThisDeclInfo->getKind() == DeclInfo::VariableKind) { |
| 764 | if (const VarDecl *VD = dyn_cast_or_null<VarDecl>(ThisDeclInfo->CurrentDecl)) { |
| 765 | QualType QT = VD->getType(); |
| 766 | return QT->isFunctionPointerType(); |
| 767 | } |
| 768 | } |
| 769 | return false; |
| 770 | } |
| 771 | |
Fariborz Jahanian | 664e860 | 2013-02-27 00:46:06 +0000 | [diff] [blame] | 772 | bool Sema::isObjCPropertyDecl() { |
| 773 | if (!ThisDeclInfo) |
| 774 | return false; |
| 775 | if (!ThisDeclInfo->IsFilled) |
| 776 | inspectThisDecl(); |
| 777 | return ThisDeclInfo->CurrentDecl->getKind() == Decl::ObjCProperty; |
| 778 | } |
Dmitri Gribenko | 8487c52 | 2012-07-23 17:40:30 +0000 | [diff] [blame] | 779 | |
Dmitri Gribenko | 04bf29e | 2012-08-06 21:31:15 +0000 | [diff] [blame] | 780 | bool Sema::isTemplateOrSpecialization() { |
Dmitri Gribenko | 1ca7ecc | 2012-08-01 23:08:09 +0000 | [diff] [blame] | 781 | if (!ThisDeclInfo) |
| 782 | return false; |
| 783 | if (!ThisDeclInfo->IsFilled) |
Dmitri Gribenko | 96b0986 | 2012-07-31 22:37:06 +0000 | [diff] [blame] | 784 | inspectThisDecl(); |
Dmitri Gribenko | 04bf29e | 2012-08-06 21:31:15 +0000 | [diff] [blame] | 785 | return ThisDeclInfo->getTemplateKind() != DeclInfo::NotTemplate; |
Dmitri Gribenko | 96b0986 | 2012-07-31 22:37:06 +0000 | [diff] [blame] | 786 | } |
| 787 | |
Fariborz Jahanian | 28c1cd2 | 2013-03-07 23:33:11 +0000 | [diff] [blame] | 788 | bool Sema::isContainerDecl() { |
| 789 | if (!ThisDeclInfo) |
| 790 | return false; |
| 791 | if (!ThisDeclInfo->IsFilled) |
| 792 | inspectThisDecl(); |
| 793 | return isUnionDecl() || isClassStructDecl() |
| 794 | || isObjCInterfaceDecl() || isObjCProtocolDecl(); |
| 795 | } |
| 796 | |
| 797 | bool Sema::isUnionDecl() { |
| 798 | if (!ThisDeclInfo) |
| 799 | return false; |
| 800 | if (!ThisDeclInfo->IsFilled) |
| 801 | inspectThisDecl(); |
| 802 | if (const RecordDecl *RD = |
| 803 | dyn_cast_or_null<RecordDecl>(ThisDeclInfo->CurrentDecl)) |
| 804 | return RD->isUnion(); |
| 805 | return false; |
| 806 | } |
| 807 | |
| 808 | bool Sema::isClassStructDecl() { |
| 809 | if (!ThisDeclInfo) |
| 810 | return false; |
| 811 | if (!ThisDeclInfo->IsFilled) |
| 812 | inspectThisDecl(); |
| 813 | return ThisDeclInfo->CurrentDecl && |
| 814 | isa<RecordDecl>(ThisDeclInfo->CurrentDecl) && |
| 815 | !isUnionDecl(); |
| 816 | } |
| 817 | |
| 818 | bool Sema::isObjCInterfaceDecl() { |
| 819 | if (!ThisDeclInfo) |
| 820 | return false; |
| 821 | if (!ThisDeclInfo->IsFilled) |
| 822 | inspectThisDecl(); |
| 823 | return ThisDeclInfo->CurrentDecl && |
| 824 | isa<ObjCInterfaceDecl>(ThisDeclInfo->CurrentDecl); |
| 825 | } |
| 826 | |
| 827 | bool Sema::isObjCProtocolDecl() { |
| 828 | if (!ThisDeclInfo) |
| 829 | return false; |
| 830 | if (!ThisDeclInfo->IsFilled) |
| 831 | inspectThisDecl(); |
| 832 | return ThisDeclInfo->CurrentDecl && |
| 833 | isa<ObjCProtocolDecl>(ThisDeclInfo->CurrentDecl); |
| 834 | } |
| 835 | |
Dmitri Gribenko | 8487c52 | 2012-07-23 17:40:30 +0000 | [diff] [blame] | 836 | ArrayRef<const ParmVarDecl *> Sema::getParamVars() { |
Dmitri Gribenko | 1ca7ecc | 2012-08-01 23:08:09 +0000 | [diff] [blame] | 837 | if (!ThisDeclInfo->IsFilled) |
Dmitri Gribenko | 00c59f7 | 2012-07-24 20:58:46 +0000 | [diff] [blame] | 838 | inspectThisDecl(); |
Dmitri Gribenko | 1ca7ecc | 2012-08-01 23:08:09 +0000 | [diff] [blame] | 839 | return ThisDeclInfo->ParamVars; |
Dmitri Gribenko | 8487c52 | 2012-07-23 17:40:30 +0000 | [diff] [blame] | 840 | } |
| 841 | |
| 842 | void Sema::inspectThisDecl() { |
Dmitri Gribenko | 1ca7ecc | 2012-08-01 23:08:09 +0000 | [diff] [blame] | 843 | ThisDeclInfo->fill(); |
Dmitri Gribenko | 8487c52 | 2012-07-23 17:40:30 +0000 | [diff] [blame] | 844 | } |
| 845 | |
Dmitri Gribenko | a5ef44f | 2012-07-11 21:38:39 +0000 | [diff] [blame] | 846 | unsigned Sema::resolveParmVarReference(StringRef Name, |
Dmitri Gribenko | 8487c52 | 2012-07-23 17:40:30 +0000 | [diff] [blame] | 847 | ArrayRef<const ParmVarDecl *> ParamVars) { |
| 848 | for (unsigned i = 0, e = ParamVars.size(); i != e; ++i) { |
Dmitri Gribenko | a5ef44f | 2012-07-11 21:38:39 +0000 | [diff] [blame] | 849 | const IdentifierInfo *II = ParamVars[i]->getIdentifier(); |
| 850 | if (II && II->getName() == Name) |
| 851 | return i; |
| 852 | } |
| 853 | return ParamCommandComment::InvalidParamIndex; |
| 854 | } |
| 855 | |
Dmitri Gribenko | 96b0986 | 2012-07-31 22:37:06 +0000 | [diff] [blame] | 856 | namespace { |
| 857 | class SimpleTypoCorrector { |
| 858 | StringRef Typo; |
| 859 | const unsigned MaxEditDistance; |
| 860 | |
| 861 | const NamedDecl *BestDecl; |
| 862 | unsigned BestEditDistance; |
| 863 | unsigned BestIndex; |
| 864 | unsigned NextIndex; |
| 865 | |
| 866 | public: |
| 867 | SimpleTypoCorrector(StringRef Typo) : |
| 868 | Typo(Typo), MaxEditDistance((Typo.size() + 2) / 3), |
| 869 | BestDecl(NULL), BestEditDistance(MaxEditDistance + 1), |
| 870 | BestIndex(0), NextIndex(0) |
| 871 | { } |
| 872 | |
| 873 | void addDecl(const NamedDecl *ND); |
| 874 | |
| 875 | const NamedDecl *getBestDecl() const { |
| 876 | if (BestEditDistance > MaxEditDistance) |
| 877 | return NULL; |
| 878 | |
| 879 | return BestDecl; |
| 880 | } |
| 881 | |
| 882 | unsigned getBestDeclIndex() const { |
| 883 | assert(getBestDecl()); |
| 884 | return BestIndex; |
| 885 | } |
| 886 | }; |
| 887 | |
| 888 | void SimpleTypoCorrector::addDecl(const NamedDecl *ND) { |
| 889 | unsigned CurrIndex = NextIndex++; |
| 890 | |
| 891 | const IdentifierInfo *II = ND->getIdentifier(); |
| 892 | if (!II) |
| 893 | return; |
| 894 | |
| 895 | StringRef Name = II->getName(); |
| 896 | unsigned MinPossibleEditDistance = abs((int)Name.size() - (int)Typo.size()); |
| 897 | if (MinPossibleEditDistance > 0 && |
| 898 | Typo.size() / MinPossibleEditDistance < 3) |
| 899 | return; |
| 900 | |
| 901 | unsigned EditDistance = Typo.edit_distance(Name, true, MaxEditDistance); |
| 902 | if (EditDistance < BestEditDistance) { |
| 903 | BestEditDistance = EditDistance; |
| 904 | BestDecl = ND; |
| 905 | BestIndex = CurrIndex; |
| 906 | } |
| 907 | } |
| 908 | } // unnamed namespace |
| 909 | |
Dmitri Gribenko | a5ef44f | 2012-07-11 21:38:39 +0000 | [diff] [blame] | 910 | unsigned Sema::correctTypoInParmVarReference( |
| 911 | StringRef Typo, |
Dmitri Gribenko | 8487c52 | 2012-07-23 17:40:30 +0000 | [diff] [blame] | 912 | ArrayRef<const ParmVarDecl *> ParamVars) { |
Dmitri Gribenko | 96b0986 | 2012-07-31 22:37:06 +0000 | [diff] [blame] | 913 | SimpleTypoCorrector Corrector(Typo); |
| 914 | for (unsigned i = 0, e = ParamVars.size(); i != e; ++i) |
| 915 | Corrector.addDecl(ParamVars[i]); |
| 916 | if (Corrector.getBestDecl()) |
| 917 | return Corrector.getBestDeclIndex(); |
| 918 | else |
Dmitri Gribenko | 1ad23d6 | 2012-09-10 21:20:09 +0000 | [diff] [blame] | 919 | return ParamCommandComment::InvalidParamIndex; |
Dmitri Gribenko | 96b0986 | 2012-07-31 22:37:06 +0000 | [diff] [blame] | 920 | } |
Dmitri Gribenko | a5ef44f | 2012-07-11 21:38:39 +0000 | [diff] [blame] | 921 | |
Dmitri Gribenko | 96b0986 | 2012-07-31 22:37:06 +0000 | [diff] [blame] | 922 | namespace { |
| 923 | bool ResolveTParamReferenceHelper( |
| 924 | StringRef Name, |
| 925 | const TemplateParameterList *TemplateParameters, |
| 926 | SmallVectorImpl<unsigned> *Position) { |
| 927 | for (unsigned i = 0, e = TemplateParameters->size(); i != e; ++i) { |
| 928 | const NamedDecl *Param = TemplateParameters->getParam(i); |
| 929 | const IdentifierInfo *II = Param->getIdentifier(); |
| 930 | if (II && II->getName() == Name) { |
| 931 | Position->push_back(i); |
| 932 | return true; |
| 933 | } |
| 934 | |
| 935 | if (const TemplateTemplateParmDecl *TTP = |
| 936 | dyn_cast<TemplateTemplateParmDecl>(Param)) { |
| 937 | Position->push_back(i); |
| 938 | if (ResolveTParamReferenceHelper(Name, TTP->getTemplateParameters(), |
| 939 | Position)) |
| 940 | return true; |
| 941 | Position->pop_back(); |
Dmitri Gribenko | a5ef44f | 2012-07-11 21:38:39 +0000 | [diff] [blame] | 942 | } |
| 943 | } |
Dmitri Gribenko | 96b0986 | 2012-07-31 22:37:06 +0000 | [diff] [blame] | 944 | return false; |
| 945 | } |
| 946 | } // unnamed namespace |
Dmitri Gribenko | a5ef44f | 2012-07-11 21:38:39 +0000 | [diff] [blame] | 947 | |
Dmitri Gribenko | 96b0986 | 2012-07-31 22:37:06 +0000 | [diff] [blame] | 948 | bool Sema::resolveTParamReference( |
| 949 | StringRef Name, |
| 950 | const TemplateParameterList *TemplateParameters, |
| 951 | SmallVectorImpl<unsigned> *Position) { |
| 952 | Position->clear(); |
| 953 | if (!TemplateParameters) |
| 954 | return false; |
| 955 | |
| 956 | return ResolveTParamReferenceHelper(Name, TemplateParameters, Position); |
| 957 | } |
| 958 | |
| 959 | namespace { |
| 960 | void CorrectTypoInTParamReferenceHelper( |
| 961 | const TemplateParameterList *TemplateParameters, |
| 962 | SimpleTypoCorrector &Corrector) { |
| 963 | for (unsigned i = 0, e = TemplateParameters->size(); i != e; ++i) { |
| 964 | const NamedDecl *Param = TemplateParameters->getParam(i); |
| 965 | Corrector.addDecl(Param); |
| 966 | |
| 967 | if (const TemplateTemplateParmDecl *TTP = |
| 968 | dyn_cast<TemplateTemplateParmDecl>(Param)) |
| 969 | CorrectTypoInTParamReferenceHelper(TTP->getTemplateParameters(), |
| 970 | Corrector); |
| 971 | } |
| 972 | } |
| 973 | } // unnamed namespace |
| 974 | |
| 975 | StringRef Sema::correctTypoInTParamReference( |
| 976 | StringRef Typo, |
| 977 | const TemplateParameterList *TemplateParameters) { |
| 978 | SimpleTypoCorrector Corrector(Typo); |
| 979 | CorrectTypoInTParamReferenceHelper(TemplateParameters, Corrector); |
| 980 | if (const NamedDecl *ND = Corrector.getBestDecl()) { |
| 981 | const IdentifierInfo *II = ND->getIdentifier(); |
| 982 | assert(II && "SimpleTypoCorrector should not return this decl"); |
| 983 | return II->getName(); |
| 984 | } |
| 985 | return StringRef(); |
Dmitri Gribenko | a5ef44f | 2012-07-11 21:38:39 +0000 | [diff] [blame] | 986 | } |
| 987 | |
Dmitri Gribenko | 2d66a50 | 2012-07-23 16:43:01 +0000 | [diff] [blame] | 988 | InlineCommandComment::RenderKind |
| 989 | Sema::getInlineCommandRenderKind(StringRef Name) const { |
Dmitri Gribenko | e4330a3 | 2012-09-10 20:32:42 +0000 | [diff] [blame] | 990 | assert(Traits.getCommandInfo(Name)->IsInlineCommand); |
Dmitri Gribenko | 2d66a50 | 2012-07-23 16:43:01 +0000 | [diff] [blame] | 991 | |
| 992 | return llvm::StringSwitch<InlineCommandComment::RenderKind>(Name) |
| 993 | .Case("b", InlineCommandComment::RenderBold) |
| 994 | .Cases("c", "p", InlineCommandComment::RenderMonospaced) |
| 995 | .Cases("a", "e", "em", InlineCommandComment::RenderEmphasized) |
| 996 | .Default(InlineCommandComment::RenderNormal); |
| 997 | } |
| 998 | |
Dmitri Gribenko | 8d3ba23 | 2012-07-06 00:28:32 +0000 | [diff] [blame] | 999 | } // end namespace comments |
| 1000 | } // end namespace clang |
| 1001 | |