Re: [Lldb-commits] [PATCH] D12612: Fix dotest on Windows after multiprocessing refactor

2015-10-11 Thread Zachary Turner via lldb-commits
Yes On Sat, Oct 10, 2015 at 9:18 PM Todd Fiala wrote: > tfiala added a comment. > > Did this go in, Zachary? I'm just looking to clear out my reviews and > this one seems to still be open. > > > http://reviews.llvm.org/D12612 > > > >

Re: [Lldb-commits] [PATCH] D12612: Fix dotest on Windows after multiprocessing refactor

2015-10-11 Thread Todd Fiala via lldb-commits
tfiala closed this revision. tfiala added a comment. Great, thanks! http://reviews.llvm.org/D12612 ___ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Re: [Lldb-commits] [PATCH] D12612: Fix dotest on Windows after multiprocessing refactor

2015-10-10 Thread Todd Fiala via lldb-commits
tfiala added a comment. Did this go in, Zachary? I'm just looking to clear out my reviews and this one seems to still be open. http://reviews.llvm.org/D12612 ___ lldb-commits mailing list lldb-commits@lists.llvm.org

Re: [Lldb-commits] [PATCH] D12612: Fix dotest on Windows after multiprocessing refactor

2015-09-03 Thread Todd Fiala via lldb-commits
tfiala accepted this revision. tfiala added a comment. This revision is now accepted and ready to land. Okay I see, makes sense. Particularly the discrepancy. That code will get much nicer (in dotest.py) when that gets some energy put into cleaning it up. Thanks for tracking it down, Zachary!