<div dir="ltr"><div class="gmail_extra">First of all...</div><div class="gmail_extra"><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" class="cremed">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 class="il">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 class="il">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 class="il">to</span> build MSan unittests.</div><div class="im"><div></div></div></blockquote></div><br>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>