chaoren added inline comments.

================
Comment at: test/dosep.py:142
@@ +141,3 @@
+    if exit_status == 0:
+        # stdout does not have any useful information from 'dotest.py', only 
stderr does.
+        report_test_pass(name, output[1])
----------------
To clarify, I mean something like:

```
with output_lock:
    if output_on_success:
        print >> sys.stderr, output
        print >> sys.stderr, "[%s] PASSED" % name # silent
    update_progress()
```

in `report_test_pass`.

and

```
sys.stderr.write("\r%*d out of %d test suites processed - [%s]\n" %
            (len(str(total_tests)), test_counter.value, total_tests), name)
```
in `update_progress`.


http://reviews.llvm.org/D11816



_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to