
<html><head><meta http-equiv="Content-Type" content="text/html charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><div><blockquote type="cite" class=""><div class=""><div dir="ltr" class=""></div></div></blockquote><blockquote type="cite" class=""><div class=""><div class="gmail_extra"><br class=""><div class="gmail_quote">On Wed, Mar 23, 2016 at 11:36 AM, Pete Cooper <span dir="ltr" class=""><<a href="mailto:peter_cooper@apple.com" target="_blank" class="">peter_cooper@apple.com</a>></span> wrote:<br class=""><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Mehdi, David<br class="">

<br class="">

This is a spin off from the discussion on Allocator.h and requesting that we allocate 0 bytes.<br class="">

<br class="">

Independent from whatever gets decided on Allocate behaviour, I think it would be best if StringRef::copy returns StringRef(nullptr, 0) whenever we try to copy an empty string.  StringRef(“”) would also be fine if you prefer.<br class="">

<br class="">

This makes that change and adds test to the unit test for the new behavior.<br class="">

<br class="">

Cheers,<br class="">

Pete<br class="">

<br class="">

</blockquote></div><br class=""></div>

</div></blockquote></div><br class=""></body></html>