[Lldb-commits] [lldb] [lldb][NFCI] Minor refactor to CommandObjectProcessHandle::VerifyCommandOptionValue (PR #79901)

Greg Clayton via lldb-commits lldb-commits at lists.llvm.org
Mon Jan 29 16:50:08 PST 2024


https://github.com/clayborg requested changes to this pull request.

We should be modify `OptionArgParser::ToBoolean(...)` to return a `std::optional<bool>` instead of a local function. It already handles integer strings "0" and "1" correctly. Granted it doesn't allow "00" or "01" to be specified, but I am not sure we care. If we do, we can fix `OptionArgParser::ToBoolean(...)`

https://github.com/llvm/llvm-project/pull/79901


More information about the lldb-commits mailing list