================ @@ -769,10 +768,10 @@ bool CommandObjectRaw::Execute(const char *args_string, handled = InvokeOverrideCallback(argv, result); } if (!handled) { - if (CheckRequirements(result)) - handled = DoExecute(args_string, result); + if (CheckRequirements(result)) { + DoExecute(args_string, result); + } ---------------- PortalPete wrote:
I took out the braces. Interesting the code_formatter test/bot didn't flag that or the other one-liners within curly braces. I typically write my code for safety first and then readability/maintainability, and I put in the curly braces in for those reasons. https://github.com/llvm/llvm-project/pull/69989 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits