<div dir="ltr"><div dir="ltr">On Tue, Jun 11, 2019 at 12:03 PM Ulf Hermann <<a href="mailto:ulf.hermann@qt.io">ulf.hermann@qt.io</a>> wrote:<br></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
3) Also sort the data on copying. Then you can still share the result. <br></blockquote><div><br>You mean at the point of the shallow-copy (i.e. ref-count increment)? If so, yes, that could work too, I think.<br><br></div><div><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Jun 11, 2019 at 12:46 PM Giuseppe D'Angelo via Development <<a href="mailto:development@qt-project.org">development@qt-project.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">It's fun to write containers and everything, but this stuff already <br></blockquote><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">exists, and it's ready to be used (under liberal licenses): <br>boost::container::flat_map, absl::flat_hash_map, and so on. Maybe the <br>only thing missing is a flat unordered container that doesn't use <br>hashing at all (just a vector of elements compared with op== for <br>detecting duplicates, for very small workloads).<br><br>Anyhow, given all of this isn't meant for public APIs, let's just use <br>them and move on?<br></blockquote><div><br>I find that unlikely to happen. Otherwise it'd raise a question in my mind why this[1] couldn't go in.<br><br></div></div>[1]: <a href="https://codereview.qt-project.org/c/qt/qtbase/+/261715">https://codereview.qt-project.org/c/qt/qtbase/+/261715</a></div></div></div>