<div dir="ltr">Was trying to circle back to write this test. But I realize that the objects in the set are pointers so moving or copying them is trivial.</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Aug 19, 2014 at 11:27 PM, Chandler Carruth <span dir="ltr"><<a href="mailto:chandlerc@google.com" target="_blank">chandlerc@google.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote"><div class="">On Tue, Aug 19, 2014 at 10:19 PM, Craig Topper <span dir="ltr"><<a href="mailto:craig.topper@gmail.com" target="_blank">craig.topper@gmail.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Not sure how. How do can we verify if its still 'small' or not from outside?</blockquote><div><br></div></div>
<div>I thought we had a way to test for that, mostly to check things in tests?</div>
<div><br></div><div>You can still verify it without that though: put objects in it that track when they are moved (or copied) and move a set. When the set is small, it will move the elements. When it is large, it will just re-point the pointers.</div>
<div class="">
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"> With the off by 1 fixed, the test did catch the bad move constructor.</blockquote><div><br></div></div><div>
Yea, it was just the off-by-one issue i was wondering about. </div>
</div><br><br></div></div>
</blockquote></div><br><br clear="all"><div><br></div>-- <br>~Craig
</div>