I guess there's a possibility a test on Windows could return 124 and mean something completely different than timeout.  Also 124 is a magic number that people won't understand unless they check the man page for timeout and gtimeout.  <div><br></div><div>Can you move the return code checking into the try_timeout function, and then return an enum like eTimedOut, eFailed, eSucceeded?<br></div><div><br></div><div>Other than that, looks good.</div><br><div class="gmail_quote">On Thu Dec 04 2014 at 11:39:12 AM Chaoren Lin <<a href="mailto:chaorenl@google.com">chaorenl@google.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Timeout if command exists.<br>
<br>
<a href="http://reviews.llvm.org/D6364" target="_blank">http://reviews.llvm.org/D6364</a><br>
<br>
Files:<br>
  test/dosep.py<br>
</blockquote></div>