[PATCH] D96608: [clangd] Delay binding LSP methods until initialize. NFC

2021-02-15 Thread Sam McCall via Phabricator via cfe-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG6c5f17e701ff: [clangd] Delay binding LSP methods until initialize. NFC (authored by sammccall). Repository: rG LLVM Github Monorepo CHANGES SINCE

[PATCH] D96608: [clangd] Delay binding LSP methods until initialize. NFC

2021-02-15 Thread Sam McCall via Phabricator via cfe-commits
sammccall updated this revision to Diff 323793. sammccall marked an inline comment as done. sammccall added a comment. address comments & rebase Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D96608/new/ https://reviews.llvm.org/D96608 Files: clan

[PATCH] D96608: [clangd] Delay binding LSP methods until initialize. NFC

2021-02-15 Thread Sam McCall via Phabricator via cfe-commits
sammccall marked 2 inline comments as done. sammccall added inline comments. Comment at: clang-tools-extra/clangd/ClangdLSPServer.cpp:165 return false; -if (!Server.Server) { - elog("Notification {0} before initialization", Method); -} else if (Method == "$/ca

[PATCH] D96608: [clangd] Delay binding LSP methods until initialize. NFC

2021-02-12 Thread Kadir Cetinkaya via Phabricator via cfe-commits
kadircet accepted this revision. kadircet added inline comments. This revision is now accepted and ready to land. Comment at: clang-tools-extra/clangd/ClangdLSPServer.cpp:165 return false; -if (!Server.Server) { - elog("Notification {0} before initialization", Met

[PATCH] D96608: [clangd] Delay binding LSP methods until initialize. NFC

2021-02-12 Thread Sam McCall via Phabricator via cfe-commits
sammccall created this revision. sammccall added a reviewer: kadircet. Herald added subscribers: usaxena95, arphaman. sammccall requested review of this revision. Herald added subscribers: cfe-commits, MaskRay, ilya-biryukov. Herald added a project: clang. This is NFC because the MessageHandler re