Support multiple -init=

Initialization options are applied (deserialized to the same object) in the following order:

* "initializationOptions" from client
* first -init=
* second -init=
* ...

Scalar options will be overridden but arrays will get concatenated, e.g.

ccls -log-file=/dev/stderr -index . -init='{"clang":{"extraArgs":["-DA"]}}' -init='{"clang":{"extraArgs":["-DB"]}}'

results in clang.extraArgs: ["-DA", "-DB"]
This commit is contained in:
Fangrui Song 2018-12-09 13:50:58 -08:00
parent 259b9fefb3
commit 6945a56fb8
2 changed files with 32 additions and 29 deletions

View File

@ -42,7 +42,7 @@ using namespace llvm;
using namespace llvm::cl; using namespace llvm::cl;
namespace ccls { namespace ccls {
std::string g_init_options; std::vector<std::string> g_init_options;
} }
namespace { namespace {
@ -56,7 +56,7 @@ opt<std::string> opt_test_index("test-index", ValueOptional, init("!"),
opt<std::string> opt_index("index", opt<std::string> opt_index("index",
desc("standalone mode: index a project and exit"), desc("standalone mode: index a project and exit"),
value_desc("root"), cat(C)); value_desc("root"), cat(C));
opt<std::string> opt_init("init", desc("extra initialization options in JSON"), list<std::string> opt_init("init", desc("extra initialization options in JSON"),
cat(C)); cat(C));
opt<std::string> opt_log_file("log-file", desc("log"), value_desc("filename"), opt<std::string> opt_log_file("log-file", desc("log"), value_desc("filename"),
cat(C)); cat(C));
@ -118,9 +118,10 @@ int main(int argc, char **argv) {
if (!opt_init.empty()) { if (!opt_init.empty()) {
// We check syntax error here but override client-side // We check syntax error here but override client-side
// initializationOptions in messages/initialize.cc // initializationOptions in messages/initialize.cc
g_init_options = opt_init.getValue(); g_init_options = opt_init;
rapidjson::Document reader; rapidjson::Document reader;
rapidjson::ParseResult ok = reader.Parse(g_init_options.c_str()); for (const std::string &str : g_init_options) {
rapidjson::ParseResult ok = reader.Parse(str.c_str());
if (!ok) { if (!ok) {
fprintf(stderr, "Failed to parse --init as JSON: %s (%zd)\n", fprintf(stderr, "Failed to parse --init as JSON: %s (%zd)\n",
rapidjson::GetParseError_En(ok.Code()), ok.Offset()); rapidjson::GetParseError_En(ok.Code()), ok.Offset());
@ -137,6 +138,7 @@ int main(int argc, char **argv) {
return 1; return 1;
} }
} }
}
sys::ChangeStdinToBinary(); sys::ChangeStdinToBinary();
sys::ChangeStdoutToBinary(); sys::ChangeStdoutToBinary();

View File

@ -36,8 +36,7 @@ limitations under the License.
namespace ccls { namespace ccls {
using namespace llvm; using namespace llvm;
// TODO Cleanup global variables extern std::vector<std::string> g_init_options;
extern std::string g_init_options;
namespace { namespace {
enum class TextDocumentSyncKind { None = 0, Full = 1, Incremental = 2 }; enum class TextDocumentSyncKind { None = 0, Full = 1, Incremental = 2 };
@ -255,7 +254,8 @@ void Initialize(MessageHandler *m, InitializeParam &param, ReplyOnce &reply) {
{ {
g_config = new Config(param.initializationOptions); g_config = new Config(param.initializationOptions);
rapidjson::Document reader; rapidjson::Document reader;
reader.Parse(g_init_options.c_str()); for (const std::string &str : g_init_options) {
reader.Parse(str.c_str());
if (!reader.HasParseError()) { if (!reader.HasParseError()) {
JsonReader json_reader{&reader}; JsonReader json_reader{&reader};
try { try {
@ -265,6 +265,7 @@ void Initialize(MessageHandler *m, InitializeParam &param, ReplyOnce &reply) {
// MessageRegistry::Parse in lsp.cc // MessageRegistry::Parse in lsp.cc
} }
} }
}
rapidjson::StringBuffer output; rapidjson::StringBuffer output;
rapidjson::Writer<rapidjson::StringBuffer> writer(output); rapidjson::Writer<rapidjson::StringBuffer> writer(output);