[llvm-branch-commits] [clang-tools-extra] 82c22f1 - [clangd] Fix compile error after 20b69af7
Nathan James via llvm-branch-commits
llvm-branch-commits at lists.llvm.org
Sun Nov 22 02:53:20 PST 2020
Author: Nathan James
Date: 2020-11-22T10:48:48Z
New Revision: 82c22f124816d1f260dc8f0626d56b459d1358b8
URL: https://github.com/llvm/llvm-project/commit/82c22f124816d1f260dc8f0626d56b459d1358b8
DIFF: https://github.com/llvm/llvm-project/commit/82c22f124816d1f260dc8f0626d56b459d1358b8.diff
LOG: [clangd] Fix compile error after 20b69af7
Some of the buildbots were failing due to what seems to be them using a non c++14 compilant std::string implementation.
Since c++14 std::basic_string::append(const basic_string, size_t, size_t) has a defaulted 3rd paramater, but some of the build bots were reporting that it wasn't defaulted in their implementation.
Added:
Modified:
clang-tools-extra/clangd/ConfigCompile.cpp
Removed:
################################################################################
diff --git a/clang-tools-extra/clangd/ConfigCompile.cpp b/clang-tools-extra/clangd/ConfigCompile.cpp
index ff031238418d..3f4dcd3c036d 100644
--- a/clang-tools-extra/clangd/ConfigCompile.cpp
+++ b/clang-tools-extra/clangd/ConfigCompile.cpp
@@ -292,7 +292,8 @@ struct FragmentCompiler {
Out.Apply.push_back(
[Checks = std::move(Checks)](const Params &, Config &C) {
C.ClangTidy.Checks.append(
- Checks, C.ClangTidy.Checks.empty() ? /*skip comma*/ 1 : 0);
+ Checks, C.ClangTidy.Checks.empty() ? /*skip comma*/ 1 : 0,
+ std::string::npos);
});
if (!F.CheckOptions.empty()) {
std::vector<std::pair<std::string, std::string>> CheckOptions;
More information about the llvm-branch-commits
mailing list