<div dir="ltr">Hi Jonathan,<div><br></div><div>The race I was talking about in watch() is this:</div><div>--</div><div>1. Client opens Watchdog(...)</div><div>2. Watchdog.handler() fires, does nothing because self.popens is empty.</div><div>3. Client calls Watchdog.watch()</div><div>4. Now that process will never be killed by the timeout.</div><div>--<br><div class="gmail_extra"><br></div><div class="gmail_extra">For the second comment, you can't just set a param to change execute_external, you need to change it in the test suites lit.cfg file. If you can't reproduce with that I am not sure...</div><div class="gmail_extra"><br></div><div class="gmail_extra"> - Daniel</div><div class="gmail_extra"><br></div><div class="gmail_extra"><div class="gmail_quote">On Mon, May 11, 2015 at 5:24 PM, Jonathan Roelofs <span dir="ltr"><<a href="mailto:jonathan@codesourcery.com" target="_blank">jonathan@codesourcery.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">ping: @ddunbar, can you clarify your review comments please?<br>
<div class="HOEnZb"><div class="h5"><br>
<br>
<a href="http://reviews.llvm.org/D6584" target="_blank">http://reviews.llvm.org/D6584</a><br>
<br>
EMAIL PREFERENCES<br>
  <a href="http://reviews.llvm.org/settings/panel/emailpreferences/" target="_blank">http://reviews.llvm.org/settings/panel/emailpreferences/</a><br>
<br>
<br>
</div></div></blockquote></div><br></div></div></div>