bulbazord added a comment. Looks good to me. Might want to let @jingham or @JDevlieghere give it a look-over though.
================ Comment at: lldb/test/API/functionalities/interactive_scripted_process/interactive_scripted_process.py:9 + +import os,json,struct,signal +import time ---------------- 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. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D145297/new/ https://reviews.llvm.org/D145297 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits