<div dir="ltr">To be clear, I don't think we need both; I'd like for us to drop one of them.  I just want to make sure that that the final state, post merge, works well on all of our platforms. :)</div><div class="gmail_extra">
<br><br><div class="gmail_quote">On Fri, Jul 25, 2014 at 2:14 PM, Jordan Rose <span dir="ltr"><<a href="mailto:jordan_rose@apple.com" target="_blank">jordan_rose@apple.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="HOEnZb"><div class="h5"><br>
On Jul 25, 2014, at 13:44 , Rafael Espíndola <<a href="mailto:rafael.espindola@gmail.com">rafael.espindola@gmail.com</a>> wrote:<br>
<br>
> On 25 July 2014 16:31, David Majnemer <<a href="mailto:david.majnemer@gmail.com">david.majnemer@gmail.com</a>> wrote:<br>
>> I have a few concerns:<br>
>> 1. GetTempPathW is buggy, we should be manually using<br>
>> GetEnvironmentVariableW with TMP, TEMP and USERPROFILE arguments and use the<br>
>> first that exists.<br>
>> 2. We should make sure that the platform specific pieces live in their<br>
>> respective Path.inc files.<br>
>><br>
><br>
> But that is orthogonal, no?<br>
><br>
> If I understand correctly, Jordan's question is "do we need both<br>
> system_temp_directory and TempDir"? I doesn't look like it, it seems<br>
> that every user of TempDir could be replaced with<br>
> system_temp_directory.<br>
<br>
</div></div>Well, part of my original point is that TempDir is smarter than system_temp_directory in a few ways. I suppose the Windows implementation isn't one of them.</blockquote></div><br></div>