<div dir="ltr">As for the original change proposed: My guiding principle would be "do whatever std::vector does". (& that's what I did when implementing GDB pretty printers for SmallVector/SmallString/ArrayRef, etc... )<br><br>An aside: We generally don't do time limited reviews like this. Either something needs review because you're not sure about it, or it doesn't. It sounds like the feedback you were looking for probably would've been fine a post-commit review feedback just as easily & perhaps might've been a better option. (while in this case it was fine - it's sort of a community habit/standards thing - we don't want to create the idea that lack of feedback is consent/approval in the review process)</div><div class="gmail_extra"><br><div class="gmail_quote">On Sun, Jun 12, 2016 at 7:01 PM, Mike Spertus via cfe-commits <span dir="ltr"><<a href="mailto:cfe-commits@lists.llvm.org" target="_blank">cfe-commits@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">mspertus closed this revision.<br>
mspertus added a comment.<br>
<div class="HOEnZb"><div class="h5"><br>
revision 272525<br>
<br>
<br>
<a href="http://reviews.llvm.org/D21256" rel="noreferrer" target="_blank">http://reviews.llvm.org/D21256</a><br>
<br>
<br>
<br>
_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@lists.llvm.org">cfe-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits</a><br>
</div></div></blockquote></div><br></div>