From 6342b38c5fee74df94d7b0c34e5a93b9b22763df Mon Sep 17 00:00:00 2001 From: Sam McCall Date: Wed, 30 Sep 2020 10:56:43 +0200 Subject: [PATCH] [clangd] Fix member/type name conflict caught by buildbots. --- clang-tools-extra/clangd/ClangdLSPServer.cpp | 16 ++++++++-------- clang-tools-extra/clangd/ClangdLSPServer.h | 2 +- clang-tools-extra/clangd/tool/ClangdMain.cpp | 2 +- 3 files changed, 10 insertions(+), 10 deletions(-) diff --git a/clang-tools-extra/clangd/ClangdLSPServer.cpp b/clang-tools-extra/clangd/ClangdLSPServer.cpp index 4d9c0a4..dfd26ad 100644 --- a/clang-tools-extra/clangd/ClangdLSPServer.cpp +++ b/clang-tools-extra/clangd/ClangdLSPServer.cpp @@ -395,7 +395,7 @@ private: Context handlerContext() const { return Context::current().derive( kCurrentOffsetEncoding, - Server.Opts.OffsetEncoding.getValueOr(OffsetEncoding::UTF16)); + Server.Opts.Encoding.getValueOr(OffsetEncoding::UTF16)); } // We run cancelable requests in a context that does two things: @@ -465,11 +465,11 @@ static std::vector semanticTokenTypes() { void ClangdLSPServer::onInitialize(const InitializeParams &Params, Callback Reply) { // Determine character encoding first as it affects constructed ClangdServer. - if (Params.capabilities.offsetEncoding && !Opts.OffsetEncoding) { - Opts.OffsetEncoding = OffsetEncoding::UTF16; // fallback + if (Params.capabilities.offsetEncoding && !Opts.Encoding) { + Opts.Encoding = OffsetEncoding::UTF16; // fallback for (OffsetEncoding Supported : *Params.capabilities.offsetEncoding) if (Supported != OffsetEncoding::UnsupportedEncoding) { - Opts.OffsetEncoding = Supported; + Opts.Encoding = Supported; break; } } @@ -509,8 +509,8 @@ void ClangdLSPServer::onInitialize(const InitializeParams &Params, // Server, CDB, etc. WithContext MainContext(BackgroundContext.clone()); llvm::Optional WithOffsetEncoding; - if (Opts.OffsetEncoding) - WithOffsetEncoding.emplace(kCurrentOffsetEncoding, *Opts.OffsetEncoding); + if (Opts.Encoding) + WithOffsetEncoding.emplace(kCurrentOffsetEncoding, *Opts.Encoding); Server.emplace(*CDB, TFS, Opts, static_cast(this)); } @@ -620,8 +620,8 @@ void ClangdLSPServer::onInitialize(const InitializeParams &Params, }}, {"typeHierarchyProvider", true}, }}}}; - if (Opts.OffsetEncoding) - Result["offsetEncoding"] = *Opts.OffsetEncoding; + if (Opts.Encoding) + Result["offsetEncoding"] = *Opts.Encoding; if (Opts.TheiaSemanticHighlighting) Result.getObject("capabilities") ->insert( diff --git a/clang-tools-extra/clangd/ClangdLSPServer.h b/clang-tools-extra/clangd/ClangdLSPServer.h index 3dc679c..e8823d3 100644 --- a/clang-tools-extra/clangd/ClangdLSPServer.h +++ b/clang-tools-extra/clangd/ClangdLSPServer.h @@ -44,7 +44,7 @@ public: /// If not set, we search upward from the source file. llvm::Optional CompileCommandsDir; /// The offset-encoding to use, or None to negotiate it over LSP. - llvm::Optional OffsetEncoding; + llvm::Optional Encoding; /// Per-feature options. Generally ClangdServer lets these vary /// per-request, but LSP allows limited/no customizations. diff --git a/clang-tools-extra/clangd/tool/ClangdMain.cpp b/clang-tools-extra/clangd/tool/ClangdMain.cpp index 60a6c26..a897a9a 100644 --- a/clang-tools-extra/clangd/tool/ClangdMain.cpp +++ b/clang-tools-extra/clangd/tool/ClangdMain.cpp @@ -820,7 +820,7 @@ clangd accepts flags on the commandline, and in the CLANGD_FLAGS environment var return true; }; if (ForceOffsetEncoding != OffsetEncoding::UnsupportedEncoding) - Opts.OffsetEncoding = ForceOffsetEncoding; + Opts.Encoding = ForceOffsetEncoding; // Shall we allow to customize the file limit? Opts.Rename.AllowCrossFile = CrossFileRename; -- 2.7.4