[PATCH] D54157: [clangd] [NFC] Fix clang-tidy warnings.
Kadir Cetinkaya via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Nov 7 04:27:57 PST 2018
This revision was automatically updated to reflect the committed changes.
Closed by commit rL346308: [clangd] [NFC] Fix clang-tidy warnings. (authored by kadircet, committed by ).
Herald added a subscriber: llvm-commits.
Repository:
rL LLVM
https://reviews.llvm.org/D54157
Files:
clang-tools-extra/trunk/clangd/index/FileIndex.cpp
clang-tools-extra/trunk/unittests/clangd/BackgroundIndexTests.cpp
Index: clang-tools-extra/trunk/clangd/index/FileIndex.cpp
===================================================================
--- clang-tools-extra/trunk/clangd/index/FileIndex.cpp
+++ clang-tools-extra/trunk/clangd/index/FileIndex.cpp
@@ -126,7 +126,7 @@
for (const auto &Sym : *Slab) {
auto I = Merged.try_emplace(Sym.ID, Sym);
if (!I.second)
- I.first->second = mergeSymbol(std::move(I.first->second), Sym);
+ I.first->second = mergeSymbol(I.first->second, Sym);
}
}
SymsStorage.reserve(Merged.size());
Index: clang-tools-extra/trunk/unittests/clangd/BackgroundIndexTests.cpp
===================================================================
--- clang-tools-extra/trunk/unittests/clangd/BackgroundIndexTests.cpp
+++ clang-tools-extra/trunk/unittests/clangd/BackgroundIndexTests.cpp
@@ -43,7 +43,7 @@
void f_b() {
(void)common;
})cpp";
- BackgroundIndex Idx(Context::empty(), "", FS, /*URISchmes=*/{"unittest"});
+ BackgroundIndex Idx(Context::empty(), "", FS, /*URISchemes=*/{"unittest"});
tooling::CompileCommand Cmd;
Cmd.Filename = testPath("root/A.cc");
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D54157.172923.patch
Type: text/x-patch
Size: 1155 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20181107/6a7aa343/attachment-0001.bin>
More information about the cfe-commits
mailing list