[Lldb-commits] [PATCH] D145297: [lldb] Add an example of interactive scripted process debugging

[Jonas Devlieghere via Phabricator via lldb-commits]

JDevlieghere added inline comments.


================
Comment at: lldb/test/API/functionalities/interactive_scripted_process/interactive_scripted_process.py:9
+
+import os,json,struct,signal
+import time
----------------
bulbazord wrote:
> mib wrote:
> > bulbazord wrote:
> > > nit: import these all on different lines
> > why :p ?
> I don't actually think it matters a ton, which is why it was a nit, but I think there's something to be said for consistency with the rest of lldb.
Given the discussion in https://discourse.llvm.org/t/rfc-document-and-standardize-python-code-style, let's format new tests with `black`. It doesn't look like the tool has a preference between those two so I would vote for consistency. 


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

https://reviews.llvm.org/D145297