<div dir="ltr">On Tue, Aug 20, 2013 at 2:44 PM, Hans Wennborg <span dir="ltr"><<a href="mailto:hans@chromium.org" target="_blank">hans@chromium.org</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On Tue, Aug 20, 2013 at 2:26 PM, Reid Kleckner <<a href="mailto:rnk@google.com">rnk@google.com</a>> wrote:<br>

>>    std::string ProgName(llvm::sys::path::stem(ArgVector[0]));<br>
>> +  for (size_t i = 0, e = ProgName.length(); i != e; ++i)<br>
>> +    ProgName[i] = toLowercase(ProgName[i]);<br>
><br>
><br>
> This is modern C++, we can't use for loops, what are we, farmers!?<br>
> std::transform(ProgName.begin(), ProgName.end(), ProgName.begin(),<br>
> ::toLowerCase);<br>
<br>
</div>Sure, I've updated the patch.<br>
<div class="im"><br>
> Also, should we do this transform on non-Windows platforms?<br>
<br>
</div>Yes, I think we should. Mostly because I don't see any reason not to,<br>
and the less platform-specific code, the better.<br></blockquote><div><br></div><div>OK.  LGTM</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
New patch uploaded.<br>
</blockquote></div><br></div></div>