chaoren added a comment.
Zach, there were two more comments since the LGTM.
================
Comment at: test/dosep.py:1
@@ -1,2 +1,2 @@
-#!/usr/bin/env python
+#!/usr/bin/env python
----------------
chaoren wrote:
> I think you added a <U+FEFF> character here. Could you please double check
> before submitting?
Please remove the byte-order-mark that your editor added.
================
Comment at: test/dosep.py:95
@@ -79,1 +94,3 @@
+
+def update_progress(test_name, result):
global output_lock, test_counter, total_tests
----------------
chaoren wrote:
> The order of `test_name` and `result` here is different from the invocation.
The order of test_name and result here is different from the invocation.
http://reviews.llvm.org/D11816
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
