Skip to content
This repository has been archived by the owner on Dec 7, 2020. It is now read-only.

Replace LruCache with an external library #119

Merged
merged 1 commit into from
Jul 5, 2016
Merged

Replace LruCache with an external library #119

merged 1 commit into from
Jul 5, 2016

Conversation

belak
Copy link
Contributor

@belak belak commented Jul 2, 2016

I looked at using the python stdlib functools.lru_cache decorator as well, but it doesn't lend itself very well to this use case because there are multiple ways of getting something into the cache.

This drops some code and replaces it with the external library cachetools because it doesn't make much sense to copy an LRU cache in when we could just use a pre-existing one.

@jodal
Copy link
Member

jodal commented Jul 5, 2016

👍

Easy choice as cachetools is maintained by @tkem, an active Mopidy contributor, and it is already used by other Mopidy extensions.

@jodal jodal merged commit 5e8d637 into mopidy:develop Jul 5, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants