<div dir="ltr">On Mon, Sep 2, 2013 at 11:27 AM, Alexander Kornienko <span dir="ltr"><<a href="mailto:alexfh@google.com" target="_blank">alexfh@google.com</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"><br>
<br>
================<br>
Comment at: lib/Support/Locale.cpp:18<br>
@@ +17,3 @@<br>
+#if LLVM_ON_WIN32<br>
+  return ' ' <= UCS && UCS <= '~';<br>
+#else<br>
----------------<br>
</div><div class="im">Manuel Klimek wrote:<br>
> I think this could need a comment. Why ~? I assume it's some limitations of a win32 terminal?<br>
</div>This is the range of non-control ASCII characters (0x20 - 0x7E). I can add this in a comment if you think it makes sense.<br></blockquote><div><br></div><div>Well, especially explain why the upper latin1 is not included (I would have intuitively expected it to work).</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
<br>
<a href="http://llvm-reviews.chandlerc.com/D1559" target="_blank">http://llvm-reviews.chandlerc.com/D1559</a><br>
</blockquote></div><br></div></div>