[Lldb-commits] [PATCH] D12612: Fix dotest on Windows after multiprocessing refactor
Todd Fiala via lldb-commits
lldb-commits at lists.llvm.org
Thu Sep 3 16:04:26 PDT 2015
tfiala added a comment.
In http://reviews.llvm.org/D12612#239698, @chying wrote:
> This patch works for me.
> Although I thought we could put less code in main() function.
> Then have
> if __name__ == "__main__":
>
> main()
>
> But that will require some more work to refactor the code. Probably that could wait for Todd's clean up.
Yep, let's wait on that. There is a metric ton of code that needs to shift around in there, be broken down into bite-sized chunks of functionality, etc. We'll get it cleaned up with a main() function soon enough when we do some of that decomposition.
http://reviews.llvm.org/D12612
More information about the lldb-commits
mailing list