teemperor accepted this revision. teemperor added a comment. This revision is now accepted and ready to land.
This patch sparks joy, but the fact that this all is dead code without Swift doesn't spark joy. But as it's up to me to revive this code and the downstream Swift test looks fine, I'll sign this off. Thanks! ================ Comment at: lldb/source/Interpreter/CommandInterpreter.cpp:2096 + LanguageSet repl_languages = Language::GetLanguagesSupportingREPLs(); + LanguageType language = {}; + if (auto main_repl_language = repl_languages.GetSingularLanguage()) ---------------- `= eLanguageTypeUnknown`? I don't think we usually initialise enums with braces and it makes the default value clearer. ================ Comment at: lldb/source/Interpreter/CommandInterpreter.cpp:2100 + if (language == LanguageType::eLanguageTypeUnknown) return; ---------------- This could now be `else return` to make the error control flow a bit clearer. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D87076/new/ https://reviews.llvm.org/D87076 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits