Thanks!<br><br><div class="gmail_quote">On Fri, Sep 16, 2011 at 11:38 AM, Eli Friedman <span dir="ltr"><<a href="mailto:eli.friedman@gmail.com">eli.friedman@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 Fri, Sep 16, 2011 at 11:24 AM, Albert Wong (王重傑) <<a href="mailto:ajwong@google.com">ajwong@google.com</a>> wrote:<br>
> If you could commit it, that'd be great.  I'm not a project committer. :)<br>
<br>
</div>r139928.<br>
<font color="#888888"><br>
-Eli<br>
</font><div><div></div><div class="h5"><br>
> -Albert<br>
><br>
> On Fri, Sep 16, 2011 at 11:23 AM, Eli Friedman <<a href="mailto:eli.friedman@gmail.com">eli.friedman@gmail.com</a>><br>
> wrote:<br>
>><br>
>> On Thu, Sep 15, 2011 at 7:00 PM, Albert Wong (王重傑) <<a href="mailto:ajwong@google.com">ajwong@google.com</a>><br>
>> wrote:<br>
>> > Hey all,<br>
>> > I found a small bug in PathV2's implementation of create_directories().<br>
>> >  If<br>
>> > you try to call it with a path like /some/new/path, it will only create<br>
>> > the<br>
>> > first non-existent directory due to an improper early return.<br>
>> > I've attached a patch with a fix.  I also file a bug for it here:<br>
>> >   <a href="http://llvm.org/bugs/show_bug.cgi?id=10940" target="_blank">http://llvm.org/bugs/show_bug.cgi?id=10940</a><br>
>> > Let me know if there are any questions.<br>
>><br>
>> Patch looks fine.  (Not sure if you need me to commit this for you.)<br>
>><br>
>> -Eli<br>
><br>
><br>
</div></div></blockquote></div><br>