[Lldb-commits] [PATCH] D96463: [LLDB] Arm64/Linux test case for MTE and Pointer Authentication regset

[Pavel Labath via Phabricator via lldb-commits]

labath accepted this revision.
labath added inline comments.


================
Comment at: lldb/test/API/commands/register/register/aarch64_dynamic_regset/TestArm64DynamicRegsets.py:59-60
+                         for _ in range(p_reg_size)) + '}'
+            self.runCmd('register write p%i' %
+                        (i) + " '" + p_regs_value + "'")
+            self.expect('register read p%i' % (i), substrs=[p_regs_value])
----------------
I find this combination of %-formatting and string concatenation unsettling. Just use % for the whole string?


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D96463/new/

https://reviews.llvm.org/D96463