[clang-tools-extra] r348133 - [clangd] Fix a stale comment, NFC.

Haojian Wu via cfe-commits cfe-commits at lists.llvm.org
Mon Dec 3 05:16:04 PST 2018


Author: hokein
Date: Mon Dec  3 05:16:04 2018
New Revision: 348133

URL: http://llvm.org/viewvc/llvm-project?rev=348133&view=rev
Log:
[clangd] Fix a stale comment, NFC.

Modified:
    clang-tools-extra/trunk/clangd/index/SymbolCollector.cpp
    clang-tools-extra/trunk/clangd/index/SymbolCollector.h

Modified: clang-tools-extra/trunk/clangd/index/SymbolCollector.cpp
URL: http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/clangd/index/SymbolCollector.cpp?rev=348133&r1=348132&r2=348133&view=diff
==============================================================================
--- clang-tools-extra/trunk/clangd/index/SymbolCollector.cpp (original)
+++ clang-tools-extra/trunk/clangd/index/SymbolCollector.cpp Mon Dec  3 05:16:04 2018
@@ -260,7 +260,7 @@ void SymbolCollector::initialize(ASTCont
 }
 
 bool SymbolCollector::shouldCollectSymbol(const NamedDecl &ND,
-                                          ASTContext &ASTCtx,
+                                          const ASTContext &ASTCtx,
                                           const Options &Opts) {
   if (ND.isImplicit())
     return false;

Modified: clang-tools-extra/trunk/clangd/index/SymbolCollector.h
URL: http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/clangd/index/SymbolCollector.h?rev=348133&r1=348132&r2=348133&view=diff
==============================================================================
--- clang-tools-extra/trunk/clangd/index/SymbolCollector.h (original)
+++ clang-tools-extra/trunk/clangd/index/SymbolCollector.h Mon Dec  3 05:16:04 2018
@@ -80,8 +80,7 @@ public:
   SymbolCollector(Options Opts);
 
   /// Returns true is \p ND should be collected.
-  /// AST matchers require non-const ASTContext.
-  static bool shouldCollectSymbol(const NamedDecl &ND, ASTContext &ASTCtx,
+  static bool shouldCollectSymbol(const NamedDecl &ND, const ASTContext &ASTCtx,
                                   const Options &Opts);
 
   void initialize(ASTContext &Ctx) override;




More information about the cfe-commits mailing list