================
@@ -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

Reply via email to