[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