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; _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
