<div dir="ltr"><br><div class="gmail_extra"><div class="gmail_quote">On Fri, Jul 12, 2013 at 4:21 PM, Chandler Carruth <span dir="ltr"><<a href="mailto:chandlerc@gmail.com" target="_blank">chandlerc@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 dir="ltr"><div class="gmail_extra">First of all...</div><div class="gmail_extra"><div class="im"><br><div class="gmail_quote">
On Fri, Jul 12, 2013 at 5:12 AM, Alexey Samsonov <span dir="ltr"><<a href="mailto:samsonov@google.com" target="_blank">samsonov@google.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div>Why? Manual for <span>mbstowcs</span> and friends explicitly tells that trailing '\0' is *not* always appended</div>


<div>(I've verified this by running these functions manually), so we actually have <span>to</span> consider different cases here,</div><div>as the original code does. I went on and reverted these changes in r186155.</div>

<div><br>
</div><div>Your change also broke our buildbots and MemorySanitizer tests ("make check-msan"). It is possible that</div><div>you haven't noticed this because MSan unit tests are not built in your build tree - most likely you don't</div>


<div>have "projects/libcxx" checked out - it is required <span>to</span> build MSan unittests.</div><div><div></div></div></blockquote></div><br></div>None of the mbstowcs changes were intentional... I don't know how they even happened. I blame either rogue commands to my editor or syncing to a weir revision... Very sorry about that.</div>
</div></blockquote><div><br></div><div><br></div><div>OK, I reverted all these changes in r186158.</div></div><div><br></div>-- <br><div>Alexey Samsonov, MSK</div>
</div></div>