1 //===--- AddUsing.cpp --------------------------------------------*- C++-*-===//
2 //
3 // Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
4 // See https://llvm.org/LICENSE.txt for license information.
5 // SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
6 //
7 //===----------------------------------------------------------------------===//
8
9 #include "AST.h"
10 #include "FindTarget.h"
11 #include "refactor/Tweak.h"
12 #include "support/Logger.h"
13 #include "clang/AST/Decl.h"
14 #include "clang/AST/RecursiveASTVisitor.h"
15
16 namespace clang {
17 namespace clangd {
18 namespace {
19
20 // Tweak for removing full namespace qualifier under cursor on DeclRefExpr and
21 // types and adding "using" statement instead.
22 //
23 // Only qualifiers that refer exclusively to namespaces (no record types) are
24 // supported. There is some guessing of appropriate place to insert the using
25 // declaration. If we find any existing usings, we insert it there. If not, we
26 // insert right after the inner-most relevant namespace declaration. If there is
27 // none, or there is, but it was declared via macro, we insert above the first
28 // top level decl.
29 //
30 // Currently this only removes qualifier from under the cursor. In the future,
31 // we should improve this to remove qualifier from all occurrences of this
32 // symbol.
33 class AddUsing : public Tweak {
34 public:
35 const char *id() const override;
36
37 bool prepare(const Selection &Inputs) override;
38 Expected<Effect> apply(const Selection &Inputs) override;
39 std::string title() const override;
intent() const40 Intent intent() const override { return Refactor; }
41
42 private:
43 // The qualifier to remove. Set by prepare().
44 NestedNameSpecifierLoc QualifierToRemove;
45 // The name following QualifierToRemove. Set by prepare().
46 llvm::StringRef Name;
47 };
REGISTER_TWEAK(AddUsing)48 REGISTER_TWEAK(AddUsing)
49
50 std::string AddUsing::title() const {
51 return std::string(llvm::formatv(
52 "Add using-declaration for {0} and remove qualifier.", Name));
53 }
54
55 // Locates all "using" statements relevant to SelectionDeclContext.
56 class UsingFinder : public RecursiveASTVisitor<UsingFinder> {
57 public:
UsingFinder(std::vector<const UsingDecl * > & Results,const DeclContext * SelectionDeclContext,const SourceManager & SM)58 UsingFinder(std::vector<const UsingDecl *> &Results,
59 const DeclContext *SelectionDeclContext, const SourceManager &SM)
60 : Results(Results), SelectionDeclContext(SelectionDeclContext), SM(SM) {}
61
VisitUsingDecl(UsingDecl * D)62 bool VisitUsingDecl(UsingDecl *D) {
63 auto Loc = D->getUsingLoc();
64 if (SM.getFileID(Loc) != SM.getMainFileID()) {
65 return true;
66 }
67 if (D->getDeclContext()->Encloses(SelectionDeclContext)) {
68 Results.push_back(D);
69 }
70 return true;
71 }
72
TraverseDecl(Decl * Node)73 bool TraverseDecl(Decl *Node) {
74 // There is no need to go deeper into nodes that do not enclose selection,
75 // since "using" there will not affect selection, nor would it make a good
76 // insertion point.
77 if (Node->getDeclContext()->Encloses(SelectionDeclContext)) {
78 return RecursiveASTVisitor<UsingFinder>::TraverseDecl(Node);
79 }
80 return true;
81 }
82
83 private:
84 std::vector<const UsingDecl *> &Results;
85 const DeclContext *SelectionDeclContext;
86 const SourceManager &SM;
87 };
88
89 struct InsertionPointData {
90 // Location to insert the "using" statement. If invalid then the statement
91 // should not be inserted at all (it already exists).
92 SourceLocation Loc;
93 // Extra suffix to place after the "using" statement. Depending on what the
94 // insertion point is anchored to, we may need one or more \n to ensure
95 // proper formatting.
96 std::string Suffix;
97 };
98
99 // Finds the best place to insert the "using" statement. Returns invalid
100 // SourceLocation if the "using" statement already exists.
101 //
102 // The insertion point might be a little awkward if the decl we're anchoring to
103 // has a comment in an unfortunate place (e.g. directly above function or using
104 // decl, or immediately following "namespace {". We should add some helpers for
105 // dealing with that and use them in other code modifications as well.
106 llvm::Expected<InsertionPointData>
findInsertionPoint(const Tweak::Selection & Inputs,const NestedNameSpecifierLoc & QualifierToRemove,const llvm::StringRef Name)107 findInsertionPoint(const Tweak::Selection &Inputs,
108 const NestedNameSpecifierLoc &QualifierToRemove,
109 const llvm::StringRef Name) {
110 auto &SM = Inputs.AST->getSourceManager();
111
112 // Search for all using decls that affect this point in file. We need this for
113 // two reasons: to skip adding "using" if one already exists and to find best
114 // place to add it, if it doesn't exist.
115 SourceLocation LastUsingLoc;
116 std::vector<const UsingDecl *> Usings;
117 UsingFinder(Usings, &Inputs.ASTSelection.commonAncestor()->getDeclContext(),
118 SM)
119 .TraverseAST(Inputs.AST->getASTContext());
120
121 for (auto &U : Usings) {
122 if (SM.isBeforeInTranslationUnit(Inputs.Cursor, U->getUsingLoc()))
123 // "Usings" is sorted, so we're done.
124 break;
125 if (U->getQualifier()->getAsNamespace()->getCanonicalDecl() ==
126 QualifierToRemove.getNestedNameSpecifier()
127 ->getAsNamespace()
128 ->getCanonicalDecl() &&
129 U->getName() == Name) {
130 return InsertionPointData();
131 }
132 // Insertion point will be before last UsingDecl that affects cursor
133 // position. For most cases this should stick with the local convention of
134 // add using inside or outside namespace.
135 LastUsingLoc = U->getUsingLoc();
136 }
137 if (LastUsingLoc.isValid()) {
138 InsertionPointData Out;
139 Out.Loc = LastUsingLoc;
140 return Out;
141 }
142
143 // No relevant "using" statements. Try the nearest namespace level.
144 const DeclContext *ParentDeclCtx =
145 &Inputs.ASTSelection.commonAncestor()->getDeclContext();
146 while (ParentDeclCtx && !ParentDeclCtx->isFileContext()) {
147 ParentDeclCtx = ParentDeclCtx->getLexicalParent();
148 }
149 if (auto *ND = llvm::dyn_cast_or_null<NamespaceDecl>(ParentDeclCtx)) {
150 auto Toks = Inputs.AST->getTokens().expandedTokens(ND->getSourceRange());
151 const auto *Tok = llvm::find_if(Toks, [](const syntax::Token &Tok) {
152 return Tok.kind() == tok::l_brace;
153 });
154 if (Tok == Toks.end() || Tok->endLocation().isInvalid()) {
155 return llvm::createStringError(llvm::inconvertibleErrorCode(),
156 "Namespace with no {");
157 }
158 if (!Tok->endLocation().isMacroID()) {
159 InsertionPointData Out;
160 Out.Loc = Tok->endLocation();
161 Out.Suffix = "\n";
162 return Out;
163 }
164 }
165 // No using, no namespace, no idea where to insert. Try above the first
166 // top level decl.
167 auto TLDs = Inputs.AST->getLocalTopLevelDecls();
168 if (TLDs.empty()) {
169 return llvm::createStringError(llvm::inconvertibleErrorCode(),
170 "Cannot find place to insert \"using\"");
171 }
172 InsertionPointData Out;
173 Out.Loc = SM.getExpansionLoc(TLDs[0]->getBeginLoc());
174 Out.Suffix = "\n\n";
175 return Out;
176 }
177
prepare(const Selection & Inputs)178 bool AddUsing::prepare(const Selection &Inputs) {
179 auto &SM = Inputs.AST->getSourceManager();
180
181 // Do not suggest "using" in header files. That way madness lies.
182 if (isHeaderFile(SM.getFileEntryForID(SM.getMainFileID())->getName(),
183 Inputs.AST->getLangOpts()))
184 return false;
185
186 auto *Node = Inputs.ASTSelection.commonAncestor();
187 if (Node == nullptr)
188 return false;
189
190 // If we're looking at a type or NestedNameSpecifier, walk up the tree until
191 // we find the "main" node we care about, which would be ElaboratedTypeLoc or
192 // DeclRefExpr.
193 for (; Node->Parent; Node = Node->Parent) {
194 if (Node->ASTNode.get<NestedNameSpecifierLoc>()) {
195 continue;
196 } else if (auto *T = Node->ASTNode.get<TypeLoc>()) {
197 if (T->getAs<ElaboratedTypeLoc>()) {
198 break;
199 } else if (Node->Parent->ASTNode.get<TypeLoc>() ||
200 Node->Parent->ASTNode.get<NestedNameSpecifierLoc>()) {
201 // Node is TypeLoc, but it's parent is either TypeLoc or
202 // NestedNameSpecifier. In both cases, we want to go up, to find
203 // the outermost TypeLoc.
204 continue;
205 }
206 }
207 break;
208 }
209 if (Node == nullptr)
210 return false;
211
212 if (auto *D = Node->ASTNode.get<DeclRefExpr>()) {
213 if (auto *II = D->getDecl()->getIdentifier()) {
214 QualifierToRemove = D->getQualifierLoc();
215 Name = II->getName();
216 }
217 } else if (auto *T = Node->ASTNode.get<TypeLoc>()) {
218 if (auto E = T->getAs<ElaboratedTypeLoc>()) {
219 if (auto *BaseTypeIdentifier =
220 E.getType().getUnqualifiedType().getBaseTypeIdentifier()) {
221 Name = BaseTypeIdentifier->getName();
222 QualifierToRemove = E.getQualifierLoc();
223 }
224 }
225 }
226
227 // FIXME: This only supports removing qualifiers that are made up of just
228 // namespace names. If qualifier contains a type, we could take the longest
229 // namespace prefix and remove that.
230 if (!QualifierToRemove.hasQualifier() ||
231 !QualifierToRemove.getNestedNameSpecifier()->getAsNamespace() ||
232 Name.empty()) {
233 return false;
234 }
235
236 // Macros are difficult. We only want to offer code action when what's spelled
237 // under the cursor is a namespace qualifier. If it's a macro that expands to
238 // a qualifier, user would not know what code action will actually change.
239 // On the other hand, if the qualifier is part of the macro argument, we
240 // should still support that.
241 if (SM.isMacroBodyExpansion(QualifierToRemove.getBeginLoc()) ||
242 !SM.isWrittenInSameFile(QualifierToRemove.getBeginLoc(),
243 QualifierToRemove.getEndLoc())) {
244 return false;
245 }
246
247 return true;
248 }
249
apply(const Selection & Inputs)250 Expected<Tweak::Effect> AddUsing::apply(const Selection &Inputs) {
251 auto &SM = Inputs.AST->getSourceManager();
252 auto &TB = Inputs.AST->getTokens();
253
254 // Determine the length of the qualifier under the cursor, then remove it.
255 auto SpelledTokens = TB.spelledForExpanded(
256 TB.expandedTokens(QualifierToRemove.getSourceRange()));
257 if (!SpelledTokens) {
258 return llvm::createStringError(
259 llvm::inconvertibleErrorCode(),
260 "Could not determine length of the qualifier");
261 }
262 unsigned Length =
263 syntax::Token::range(SM, SpelledTokens->front(), SpelledTokens->back())
264 .length();
265 tooling::Replacements R;
266 if (auto Err = R.add(tooling::Replacement(
267 SM, SpelledTokens->front().location(), Length, ""))) {
268 return std::move(Err);
269 }
270
271 auto InsertionPoint = findInsertionPoint(Inputs, QualifierToRemove, Name);
272 if (!InsertionPoint) {
273 return InsertionPoint.takeError();
274 }
275
276 if (InsertionPoint->Loc.isValid()) {
277 // Add the using statement at appropriate location.
278 std::string UsingText;
279 llvm::raw_string_ostream UsingTextStream(UsingText);
280 UsingTextStream << "using ";
281 QualifierToRemove.getNestedNameSpecifier()->print(
282 UsingTextStream, Inputs.AST->getASTContext().getPrintingPolicy());
283 UsingTextStream << Name << ";" << InsertionPoint->Suffix;
284
285 assert(SM.getFileID(InsertionPoint->Loc) == SM.getMainFileID());
286 if (auto Err = R.add(tooling::Replacement(SM, InsertionPoint->Loc, 0,
287 UsingTextStream.str()))) {
288 return std::move(Err);
289 }
290 }
291
292 return Effect::mainFileEdit(Inputs.AST->getASTContext().getSourceManager(),
293 std::move(R));
294 }
295
296 } // namespace
297 } // namespace clangd
298 } // namespace clang
299