<div dir="ltr">Excellent. Thanks Rafael!</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Jun 18, 2013 at 5:31 AM, Rafael Espíndola <span dir="ltr"><<a href="mailto:rafael.espindola@gmail.com" target="_blank">rafael.espindola@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On 17 June 2013 18:09, Lang Hames <<a href="mailto:lhames@gmail.com">lhames@gmail.com</a>> wrote:<br>

> Hi Rafael,<br>
><br>
</div><div class="im">> No worries. Thanks for fixing it so quickly.<br>
><br>
> Just so I understand - the plan is ultimately to add these checks to<br>
> sys::fs::remove? (Or add a "safe" remove variant that does the checks)?<br>
<br>
</div>I added the check to sys::fs::remove, since according to the comment<br>
in unlinkFromDisk we don't want any unsafe uses in llvm.<br>
<div class="im"><br>
> Once that's done we should be able to re-apply your patch, right?<br>
<br>
</div>I have done it :-)<br>
<br>
Cheers,<br>
Rafael<br>
</blockquote></div><br></div>