[Lldb-commits] [lldb] [lldb] Improved formatting of 'register read' command. (PR #188049)

[David Spickett via lldb-commits]

================
@@ -75,7 +75,7 @@ class CommandObjectRegisterRead : public CommandObjectParsed {
 
   bool DumpRegister(const ExecutionContext &exe_ctx, Stream &strm,
                     RegisterContext &reg_ctx, const RegisterInfo &reg_info,
-                    bool print_flags) {
+                    bool print_flags, uint32_t reg_name_align_at = 0) {
----------------
DavidSpickett wrote:

align_at seems like a column number to align at, but it doesn't tell me what direction. As in, whether the start of the name is at that column or the end. In this case it's the end, or the "right" as it's sometimes called.

`name_end_align_at` ? You can drop `register` at least in this function because the function name already implies it.

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