<div dir="ltr">Hi,<div class="gmail_extra"><br><div class="gmail_quote">On Sun, May 19, 2013 at 1:41 PM, Milian Wolff <span dir="ltr"><<a href="mailto:mail@milianw.de" target="_blank">mail@milianw.de</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On Sunday 12 May 2013 19:32:38 Aleix Pol wrote:<br>
> Hi,<br>
> I just remembered we have these classes in our codebase<br>
> (kdevplatform/utils/google) and I thought that it might be a good idea to<br>
> use a newer version [1]. According to the changelogs there's been many<br>
> improvements.<br>
><br>
> Also I saw that we're installing these classes, it's not being used in<br>
> KDevelop, so I'd suggest to stop installing those classes' headers. Also<br>
> they're not being exported as well.<br>
<br>
</div>Sure, sounds good. But... where do we actually use them? Do we need them?<br>
Should we use them in more places?<br></blockquote><div><br></div><div style>The two hash maps are specifically optimized for speed or memory-efficiency respectively. The duchain uses the fast variant for its ImportsHash type. Not sure what that is. According to the commit log it was introduced for usage with small cache's as it is (or was) twice as fast as stl's hash_map.</div>
<div style><br></div><div style>In the past it was used for various things in duchain, but it seems this is not the case anymore. So maybe its not needed or someone refactored the code and switched from the google types back to Qt types. For example one case was in duchainlock.cpp (the ReaderMap which does not exist anymore), another case was m_chainsByIndex in duchain.cpp which is not a map anymore.</div>
<div style><br></div><div style>Andreas</div><div> </div></div></div></div>