<div><div dir="auto">It was quite fast really as my PR doesn’t address the fundamental design issues nor much of the coding style. A quick win could be to move from a list to an unordered_map, it’s a quite trivial cache in the end. There’s a lot of machinery in there for such a simple data structure :/</div></div><div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sat, 4 Apr 2020 at 08:15, Amos Jeffries <<a href="mailto:squid3@treenet.co.nz">squid3@treenet.co.nz</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 4/04/20 7:49 pm, Francesco Chemolli wrote:<br>
> I am not sure about what you recommend to do here.<br>
> This cache is IMO over complicated and it breaks layering.<br>
> I’m mostly done in a PR replacing the dlink with a std::list but without<br>
> changing the overall design. It does kill a few tens of lines of code<br>
> and is clearer to read tho.<br>
> <br>
<br>
Well, if you have already done the work it migth as well finish up. I<br>
was thinking you had just done the investigation and not started<br>
refactoring yet.<br>
<br>
Amos<br>
</blockquote></div></div>-- <br><div dir="ltr" class="gmail_signature" data-smartmail="gmail_signature">@mobile</div>