<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Dec 4, 2014 at 5:43 PM, Duncan P. N. Exon Smith <span dir="ltr"><<a href="mailto:dexonsmith@apple.com" target="_blank">dexonsmith@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class=""><br>
> On 2014-Dec-04, at 17:10, Duncan P. N. Exon Smith <<a href="mailto:dexonsmith@apple.com">dexonsmith@apple.com</a>> wrote:<br>
><br>
>> On 2014-Dec-04, at 15:44, Alexey Samsonov <<a href="mailto:vonosmas@gmail.com">vonosmas@gmail.com</a>> wrote:<br>
>><br>
>> FYI, StringMapEntry<>::GetStringMapEntryFromValue is inherently broken - I've failed to fix it in r223402, but failed. We can't just "get the offset" of a non-standard-layout object field.<br>
><br>
> Crap.<br>
><br>
>> I'd be happy to delete GetStringMapEntryFromValue from the codebase completely, but currently MDString::getString() is its only user.<br>
>><br>
><br>
> I'll remove that use, and kill the function myself.  Thanks.<br>
<br>
</span>r223438 and r223439.  Sorry I missed the bot failure (if there was one).<br></blockquote><div><br></div><div>Thank you! </div></div><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr">Alexey Samsonov<br><a href="mailto:vonosmas@gmail.com" target="_blank">vonosmas@gmail.com</a></div></div>
</div></div>