[clang-tools-extra] r320574 - [clangd] Remove the const specifier of the takeSymbol method

Haojian Wu via cfe-commits cfe-commits at lists.llvm.org
Wed Dec 13 04:39:06 PST 2017


Author: hokein
Date: Wed Dec 13 04:39:06 2017
New Revision: 320574

URL: http://llvm.org/viewvc/llvm-project?rev=320574&view=rev
Log:
[clangd] Remove the const specifier of the takeSymbol method

otherwise we will copy an object.

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

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=320574&r1=320573&r2=320574&view=diff
==============================================================================
--- clang-tools-extra/trunk/clangd/index/SymbolCollector.h (original)
+++ clang-tools-extra/trunk/clangd/index/SymbolCollector.h Wed Dec 13 04:39:06 2017
@@ -32,7 +32,7 @@ public:
 
   void finish() override;
 
-  SymbolSlab takeSymbols() const { return std::move(Symbols); }
+  SymbolSlab takeSymbols() { return std::move(Symbols); }
 
 private:
   // All Symbols collected from the AST.




More information about the cfe-commits mailing list