<div dir="ltr">I think it's needed. You may not see a problem with that code because it becomes visible only when the vector grows large enough so that a new vector is allocated and everything in the previous vector are moved to the new one.</div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Sep 19, 2016 at 6:47 AM, George Rimar <span dir="ltr"><<a href="mailto:grimar@accesssoftek.com" target="_blank">grimar@accesssoftek.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" style="font-size:12pt;color:#000000;background-color:#ffffff;font-family:Calibri,Arial,Helvetica,sans-serif"><div><div class="h5">
<p><span style="font-size:12pt;color:rgb(33,33,33)">>></span><span style="font-size:12pt;color:rgb(33,33,33)">Can you add a comment saying that list is used to avoid a compiler (libc++?) bug? std::vector is far more common, so it is nice to justify
 why a list is used.</span><br>
</p>
</div></div><div dir="ltr" style="font-size:12pt;color:#000000;background-color:#ffffff;font-family:Calibri,Arial,Helvetica,sans-serif">
<div>
<div style="color:rgb(33,33,33)">
<div>
<div dir="ltr">
<div class="gmail_extra">
<div class="gmail_quote"><div><div class="h5">
<div>>> </div>
<div>>> </div>
<div>>>Thanks,</div>
<div>>>Rafael</div>
<div>> </div>
<div>
<p style="font-family:Calibri,Arial,Helvetica,sans-serif;font-size:16px">​>Rui already changed that place in r<span style="font-size:12pt">281803, to use struct instead of pair,</span></p>
<p style="font-family:Calibri,Arial,Helvetica,sans-serif;font-size:16px"><span style="font-size:12pt">>but seems also had problems with vector (see <span style="background-color:rgb(255,255,255)"></span><span style="font-size:12pt;background-color:rgb(255,255,255)">r<span style="font-size:12pt">281805</span>)</span>,
 as comment now says:</span></p>
<p style="font-family:Calibri,Arial,Helvetica,sans-serif;font-size:16px"><span style="font-size:12pt">>
</span></p>
<div style="font-family:Calibri,Arial,Helvetica,sans-serif;font-size:16px">>  // We use std::list instead of std::vector because SectionPattern</div>
<div style="font-family:Calibri,Arial,Helvetica,sans-serif;font-size:16px">>  // do not support move assignment.<br>
</div>
<div style="font-family:Calibri,Arial,Helvetica,sans-serif;font-size:16px">> </div>
<div style="font-family:Calibri,Arial,Helvetica,sans-serif;font-size:16px">>I am not sure it is correct comment though, as I think struct without anything like desctructors,<br>
</div>
<div style="font-family:Calibri,Arial,Helvetica,sans-serif;font-size:16px">>and just that has 2 movable fields should be movable.<br>
</div>
<div style="font-family:Calibri,Arial,Helvetica,sans-serif;font-size:16px">> </div>
<div style="font-family:Calibri,Arial,Helvetica,sans-serif;font-size:16px">>There is no link on buildbot in commit message for 281805, </div>
<div style="font-family:Calibri,Arial,Helvetica,sans-serif;font-size:16px">>I`ll check if swiching to struct helped my FreeBSD VM where It was reproducible.<br>
</div>
<p style="font-family:Calibri,Arial,Helvetica,sans-serif;font-size:16px">​​><br>
</p>
<p style="font-family:Calibri,Arial,Helvetica,sans-serif;font-size:16px">>George.<br>
</p>
<br>
</div>
</div></div><div>And I just tried and issue does not reproduce for me on r<span style="font-size:12pt">281803 which uses struct</span></div>
<div><span style="font-size:12pt"></span><span style="font-size:12pt">instead of std::pair. So I am not sure. Is std::list is really needed after that, Rui ?</span></div><span class="HOEnZb"><font color="#888888">
<div><span style="font-size:12pt"><br>
</span></div>
<div><span style="font-size:12pt">George.​</span>
<div><br>
<br>
</div>
<br>
</div>
<div> </div>
</font></span></div>
</div>
</div>
</div>
</div>
</div>
</div>
</div>

</blockquote></div><br></div>