<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Sun, Oct 11, 2015 at 7:30 AM, Rafael Espíndola <span dir="ltr"><<a href="mailto:rafael.espindola@gmail.com" target="_blank">rafael.espindola@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 11 October 2015 at 10:24, Rui Ueyama <<a href="mailto:ruiu@google.com">ruiu@google.com</a>> wrote:<br>
> Sorry about the style change. I'll separate them in future.<br>
><br>
> I think the use of unordered_set is reasonable because the number of<br>
> segments are very small.<br>
<br>
</span>That is precisely why it is not reasonable :-)<br>
<br>
Playing with the code a bit it seems the best thing is to just replace<br>
it with at SmallPtrSet.<br></blockquote><div><br></div><div>I'm not objecting to use SmallPtrSet, and that's probably good because it wouldn't raise this kind of discussion and also hint to the reader that the set is, well, a set of small pointers. But thinking about the function is called only once and the set is probably looked up less than 100 times, I'd still call it reasonable (at least is not unreasonable). :)</div></div></div></div>