[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