================ @@ -1591,24 +1591,24 @@ class CommandObjectProcessHandle : public CommandObjectParsed { Options *GetOptions() override { return &m_options; } - bool VerifyCommandOptionValue(const std::string &option, int &real_value) { - bool okay = true; + std::optional<bool> VerifyCommandOptionValue(const std::string &option) { ---------------- clayborg wrote:
We should modify `OptionArgParser::ToBoolean(...)` to return a `std::optional<bool>`. it already handles "0" and "1" strings correctly, so this integer conversion stuff is just not doing anything useful https://github.com/llvm/llvm-project/pull/79901 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits