<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Fri, Aug 22, 2014 at 10:43 AM, David Blaikie <span dir="ltr"><<a href="mailto:dblaikie@gmail.com" target="_blank">dblaikie@gmail.com</a>></span> wrote:<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Yep - the convenience of one-element->ArrayRef is "cute" at best, I<br>
think. Having to wrap it doesn't seem detrimental. Would have to look<br>
at some numbers, though (if we could easily gather the number of<br>
instances of this - I'm not sure of the easiest way to do that, given<br>
the build system likes to stop after it sees an error).<br>
<br>
I'm not sure if the "Temp" in the name would be a sufficient<br>
deterrent, but maybe.</blockquote><div><br></div><div>Wow, I think I finally understand how the temporary is being created.</div><div><br></div><div>Can we get by without the single element constructor? Callers would have to write:</div>
<div>const char *strs[] = { "string" };</div><div>foo(strs);</div><div><br></div><div>Instead of foo("string"), which is subtle anyway.</div></div></div></div>