format

Author: Febbe

clang-tools-extra/clangd/ClangdLSPServer.cpp

@@ -856,8 +856,7 @@ void ClangdLSPServer::onCommandApplyRename(const RenameParams &R,
 void ClangdLSPServer::onMethodSearchAST(const SearchASTArgs &Args,
                                         Callback<llvm::json::Value> Reply) {
   Server->findAST(Args, [Reply = std::move(Reply)](
-                            llvm::Expected<BoundASTNodes>
-                                BoundNodes) mutable {
+                            llvm::Expected<BoundASTNodes> BoundNodes) mutable {
     if (!BoundNodes)
       return Reply(BoundNodes.takeError());
     return Reply(*BoundNodes);

clang-tools-extra/clangd/ClangdServer.cpp

@@ -812,35 +812,35 @@ void ClangdServer::locateSymbolAt(PathRef File, Position Pos,
 
 void ClangdServer::findAST(SearchASTArgs const &Args,
                            Callback<BoundASTNodes> CB) {
-  auto Action =
-      [Args, CB = std::move(CB)](llvm::Expected<InputsAndAST> InpAST) mutable {
-        if (!InpAST)
-          return CB(InpAST.takeError());
-        auto BoundNodes = clangd::locateASTQuery(InpAST->AST, Args);
-        if (!BoundNodes)
-          return CB(BoundNodes.takeError());
-        if (BoundNodes->empty())
-          return CB(error("No matching AST nodes found"));
-
-        auto &&AST = InpAST->AST;
-        // Convert BoundNodes to a vector of vectors to ASTNode's.
-        BoundASTNodes Result;
-        Result.reserve(BoundNodes->size());
-        for (auto &&BN : *BoundNodes) {
-          auto &&Map = BN.getMap();
-          BoundASTNodes::value_type BAN;
-          for (const auto &[Key, Value] : Map) {
-            BAN.emplace(Key, dumpAST(Value, AST.getTokens(), AST.getASTContext()));
-          }
-          if (BAN.empty())
-            continue;
-          Result.push_back(std::move(BAN));
-        }
-        if (Result.empty()) {
-          return CB(error("No AST nodes found for the query"));
-        }
-        CB(std::move(Result));
-      };
+  auto Action = [Args, CB = std::move(CB)](
+                    llvm::Expected<InputsAndAST> InpAST) mutable {
+    if (!InpAST)
+      return CB(InpAST.takeError());
+    auto BoundNodes = clangd::locateASTQuery(InpAST->AST, Args);
+    if (!BoundNodes)
+      return CB(BoundNodes.takeError());
+    if (BoundNodes->empty())
+      return CB(error("No matching AST nodes found"));
+
+    auto &&AST = InpAST->AST;
+    // Convert BoundNodes to a vector of vectors to ASTNode's.
+    BoundASTNodes Result;
+    Result.reserve(BoundNodes->size());
+    for (auto &&BN : *BoundNodes) {
+      auto &&Map = BN.getMap();
+      BoundASTNodes::value_type BAN;
+      for (const auto &[Key, Value] : Map) {
+        BAN.emplace(Key, dumpAST(Value, AST.getTokens(), AST.getASTContext()));
+      }
+      if (BAN.empty())
+        continue;
+      Result.push_back(std::move(BAN));
+    }
+    if (Result.empty()) {
+      return CB(error("No AST nodes found for the query"));
+    }
+    CB(std::move(Result));
+  };
 
   WorkScheduler->runWithAST("Definitions", Args.textDocument.uri.file(),
                             std::move(Action));

clang-tools-extra/clangd/ClangdServer.h

@@ -263,8 +263,7 @@ class ClangdServer {
   void locateSymbolAt(PathRef File, Position Pos,
                       Callback<std::vector<LocatedSymbol>> CB);
 
-  void findAST(const SearchASTArgs &Args,
-               Callback<BoundASTNodes> CB);
+  void findAST(const SearchASTArgs &Args, Callback<BoundASTNodes> CB);
 
   /// Switch to a corresponding source file when given a header file, and vice
   /// versa.

clang-tools-extra/clangd/Protocol.cpp

@@ -1651,13 +1651,16 @@ bool fromJSON(const llvm::json::Value &Params, SelectionRangeParams &S,
          O.map("positions", S.positions);
 }
 
-bool fromJSON(const llvm::json::Value &Params, SearchASTArgs &Args, llvm::json::Path P) {
+bool fromJSON(const llvm::json::Value &Params, SearchASTArgs &Args,
+              llvm::json::Path P) {
   llvm::json::ObjectMapper O(Params, P);
-  return O && O.map("query", Args.searchQuery)
-  && O.map("textDocument", Args.textDocument)
-  // && O.map("bindRoot", Args.bindRoot); TODO: add bindRoot to extend this feature
-  // && O.map("traversalKind", Args.tk); TODO: add traversalKind to extend this feature
-  ;
+  return O && O.map("query", Args.searchQuery) &&
+         O.map("textDocument", Args.textDocument)
+      // && O.map("bindRoot", Args.bindRoot); TODO: add bindRoot to extend this
+      // feature
+      // && O.map("traversalKind", Args.tk); TODO: add traversalKind to extend
+      // this feature
+      ;
 }
 
 llvm::json::Value toJSON(const SelectionRange &Out) {

clang-tools-extra/clangd/XRefs.cpp

@@ -799,7 +799,6 @@ auto locateASTQuery(ParsedAST &AST, SearchASTArgs const &Query)
     return error("Not a valid top-level matcher: {}.", Diag.toString());
   }
 
-
   struct CollectBoundNodes : MatchFinder::MatchCallback {
     std::vector<BoundNodes> *Bindings;
     CollectBoundNodes(std::vector<BoundNodes> &Bindings)