Timeline for Reducing contention in thread-safe LruCache
Current License: CC BY-SA 4.0
5 events
| when toggle format | what | by | license | comment | |
|---|---|---|---|---|---|
| Nov 28, 2022 at 15:21 | comment | added | AngelWarrior | LruCache in both Add and GetOrCreate, after calling RemoveLeastRecentlyUsedIfNecessary(); - it seems both also need _lruList.AddLast(lruNode); Otherwise it was only adding and finding it in the cacheMap, but it was not in the list | |
| Aug 17, 2019 at 20:10 | history | edited | Trauer | CC BY-SA 4.0 |
added 238 characters in body
|
| Aug 17, 2019 at 20:08 | comment | added | Trauer | I was pretty satisfied with the code, and I didn't think it would benefit from another round of review; but now that you commented I'm not so sure anymore, hehe. But yeah, I wanted to share the final result with the community, but without modifying the question and making the answers nonsensical. I don't think I'll choose this as an answer because I don't want to "frag-steal". But anyway, here what we produced :) | |
| Aug 17, 2019 at 20:03 | comment | added | dfhwze | A self-answer with a summary of changes you've made, and which input you appreciated from other answers, is a good way to sum it all up. Note, though, that if you want this code reviewed, you should ask a follow-up question, and we'll be happy to review that question as well. | |
| Aug 17, 2019 at 19:57 | history | answered | Trauer | CC BY-SA 4.0 |