[Lldb-commits] [PATCH] Add -stack-info-frame command (MI)

Greg Clayton clayborg at gmail.com
Thu Feb 19 11:35:48 PST 2015


Just a quick reminder you might want to use triple quotes instead of desensitizing every other character in the python test case code.

Looks good otherwise.


================
Comment at: test/tools/lldb-mi/TestMiStack.py:204
@@ +203,3 @@
+        self.runCmd("-stack-info-frame")
+        self.expect("\^error,msg=\"Command 'stack-info-frame'. Invalid process during debug session\"")
+
----------------
Use triple quotes?

 self.expect('''^error,msg="Command 'stack-info-frame'. Invalid process during debug session"''')

================
Comment at: test/tools/lldb-mi/TestMiStack.py:212
@@ +211,3 @@
+        self.runCmd("-break-insert -f main")
+        self.expect("\^done,bkpt={number=\"1\"")
+        self.runCmd("-exec-run")
----------------
Triple quotes?

        self.expect('''^done,bkpt={number="1"''')


================
Comment at: test/tools/lldb-mi/TestMiStack.py:219
@@ +218,3 @@
+        self.runCmd("-stack-info-frame")
+        self.expect("\^done,frame=\{level=\"0\",addr=\".+\",func=\"main\",file=\"main\.c\",fullname=\".*main\.c\",line=\"\d+\"\}")
+
----------------
triple quotes?

http://reviews.llvm.org/D7750

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the lldb-commits mailing list