[PATCH] D44040: Add test for lldb-mi interpreter
Adrian Prantl via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Mar 2 14:05:27 PST 2018
aprantl accepted this revision.
aprantl added a comment.
This revision is now accepted and ready to land.
Thanks, this looks good!
================
Comment at: packages/Python/lldbsuite/test/tools/lldb-mi/interpreter/TestMiCliSupport.py:45
+
+ # Test that initially there are no targets
+ self.runCmd("target list")
----------------
Small nit: We usually try to use complete sentences in comments with a `.` at the end.
Repository:
rL LLVM
https://reviews.llvm.org/D44040
More information about the llvm-commits
mailing list