[Lldb-commits] [PATCH] D114509: [lldb] Introduce PlatformQemuUser
Michał Górny via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Wed Nov 24 02:20:45 PST 2021
mgorny added inline comments.
================
Comment at: lldb/packages/Python/lldbsuite/test/gdbclientutils.py:403
return self._connection.sendall(data)
+class TCPServerSocket(ServerSocket):
----------------
It's customary (read: PEP8) to use two empty lines between global-scope stuff like `class`es (and then one line between functions inside the class).
================
Comment at: lldb/test/API/qemu/TestQemuLaunch.py:31
+ e.write("sys.path = %r\n"%sys.path)
+ e.write("runpy.run_path('%s', run_name='__main__')\n"%
+ self.getSourcePath("qemu.py"))
----------------
`%r` will use the `repr()` form, i.e. suitable for direct reuse in Python with the right escaping and so on.
================
Comment at: lldb/test/API/qemu/TestQemuLaunch.py:42-43
+ target = self.dbg.CreateTarget(exe, '', 'qemu-user', False, error)
+ self.assertEqual(target.GetPlatform().GetName(), "qemu-user")
+ self.assertSuccess(error)
+ process = target.LaunchSimple(
----------------
Wouldn't it make sense to reverse these two? i.e. if it fails, then platform is unlikely to be correct, is it?
================
Comment at: lldb/test/API/qemu/qemu.py:38
+ main()
+
----------------
Oh no, trailing empty line!
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D114509/new/
https://reviews.llvm.org/D114509
More information about the lldb-commits
mailing list