search

Infogroep / imp

Ruben en Ayrton's Rare Audio Speler
MIT License
1 stars 0 forks source link

Cache must be deleted when older than x days. #6

Closed angryzor closed 10 years ago

RoelandMatthijssens commented 10 years ago

how about specifying a max cache size instead? no sense in throwing it away for no good reason... "but what if the song suddenly changes?" ... well shit

rdewaele commented 10 years ago

LRU seems to have the desired semantics

On 12/20/13 15:00, Roeland Matthijssens wrote:

how about specifying a max cache size instead? no sense in throwing it away for no good reason... "but what if the song suddenly changes?" ... well shit

— Reply to this email directly or view it on GitHub https://github.com/Infogroep/imp/issues/6#issuecomment-31010948.

angryzor commented 10 years ago

Btw, what I eventually did (and what I originally wanted to do but didn't word well) is have script register their temp files with the playlist (potentially while queuing) and have the playlist destroy them (still through the cache script mind) when they fall off the history.