[Lldb-commits] [lldb] [lldb] Show signal number description (PR #164176)

David Spickett via lldb-commits lldb-commits at lists.llvm.org
Mon Oct 20 03:00:18 PDT 2025


================
@@ -1616,8 +1616,15 @@ class CommandObjectProcessHandle : public CommandObjectParsed {
     str.Format("{0, -11}  ", sig_name);
     if (signals_sp->GetSignalInfo(signo, suppress, stop, notify)) {
       bool pass = !suppress;
-      str.Printf("%s  %s  %s", (pass ? "true " : "false"),
+      str.Printf("%s  %s  %s  ", (pass ? "true " : "false"),
----------------
DavidSpickett wrote:

What's the logic with adding spaces here but also below?

After years of staring at Python test failures due to whitespace differences, I always try to add as few of them as possible, but that's just a personal preference most of the time.

Could you add no spaces here and then add 3 later if there is a description?

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


More information about the lldb-commits mailing list