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

[David Spickett via lldb-commits]

================
@@ -147,6 +149,62 @@ class CommandObjectRegisterRead : public CommandObjectParsed {
   }
 
 protected:
+  uint32_t GetNameSize(const RegisterInfo * reg_info, bool use_primary_name) {
----------------
DavidSpickett wrote:

Also this name doesn't make sense to me. It's a member of CommandObjectRegisterWhatever, so GetNameSize would imply the name of this class.

I would call it GetRegisterNameSize and make it a static method.

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