<div class="gmail_quote">Thanks for the review. I've committed the first version in r146729 so that fixes can happen in-tree. Almost all of your and David's feedback both here and on IRC has been addressed. Only one issue remains, and if you have ideas about how to solve it, I'll happily jump on that one.</div>
<div class="gmail_quote"><br></div><div class="gmail_quote">On Thu, Dec 15, 2011 at 10:19 AM, Douglas Gregor <span dir="ltr"><<a href="mailto:dgregor@apple.com">dgregor@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="HOEnZb"><div class="h5">Just one comment:</div></div>
<br>
+  VariadicFunction() {}<br>
+<br>
<br>
This is not trivial, which is unfortunate. Please just leave out the default constructor so we'll get the implicitly-generated trivial default constructor. (The same goes for all of the VariadicFunction* class templates).<br>
</blockquote><div><br></div><div>This sounds great, but it breaks declaring these as 'const'. It's really not clear to me why, but I may be having a dense moment here. Do you think its worth requiring these to be non-const to make the constructor trivial? Any other ideas of how to do this? I agree trivial would be much better here.</div>
<div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Also, I agree with David about the use of ArrayRef.<br></blockquote><div><br></div><div>Yea, I have no idea why I didn't fix that first thing. Thanks to you both for the prodding there, it makes the interface much nicer.</div>
</div>