[Lldb-commits] [lldb] [lldb] Part 2 of 2 - Refactor `CommandObject::DoExecute(...)` return `void` (not `bool`) (PR #69991)

[Alex Langford via lldb-commits]

bulbazord wrote:

> > > I think I looked at every changed line, looks good to me overall. Found one place with a small style issue but it's not a blocker. Thanks!
> > 
> > 
> > Thanks @bulbazord! Do you know why the `code_formatter` check/bot didn't flag that or the other one-line `if` statements?
> 
> This could be because it's part of the LLVM style guide but not enforced in the `.clang-format`.
> 
> Also looked at most of the lines, LGTM. I wonder however why did you decide to make 2 separate PRs for this ? Is there any reason for this ? It could have been 2 commits in the same PR.

Yeah, I think it's probably not enforced. Maybe it should be? 🤔 


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