<div dir="ltr">I won't be able to have a serious look until Monday, as I'm still remote.  Hopefully you arent working on weekends :)</div><br><div class="gmail_quote"><div dir="ltr">On Fri, Sep 25, 2015 at 6:30 PM Todd Fiala <<a href="mailto:todd.fiala@gmail.com">todd.fiala@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">tfiala added inline comments.<br>
<br>
================<br>
Comment at: test/dosep.py:245<br>
@@ +244,3 @@<br>
+        # binary should have called the results-generation code.<br>
+        raise Exception("no test results were generated whatsoever")<br>
+    return process_driver.results<br>
----------------<br>
This message should include the filename.<br>
<br>
================<br>
Comment at: test/process_control.py:273<br>
@@ +272,3 @@<br>
+<br>
+    def create_piped_process(self, command, new_process_group=True):<br>
+        if new_process_group:<br>
----------------<br>
This will want to store the popen_process.using_process_groups property like I do on the Posix side if the terminate would do something different based on the presence or absence of using process groups.  Oversight for not storing.<br>
<br>
<br>
<a href="http://reviews.llvm.org/D13124" rel="noreferrer" target="_blank">http://reviews.llvm.org/D13124</a><br>
<br>
<br>
<br>
</blockquote></div>