Multi-thread: Infinite loop in trimToSize about disklrucache HOT 5 OPEN

Yup, it should definitely skip entries being edited.

from disklrucache.

Tried a fix, let me know what you think about it.

from disklrucache.

It's close to what we want. I think it's going to run faster as a single loop that runs until the size is sufficiently small, or there's nothing left to remove. (Since we only allocate one iterator). Even better if you can fast-fail at the front and avoid allocating the iterator at all if the cache is already sufficiently small.

from disklrucache.

Send a pull request?

from disklrucache.

And even better, send the pull request against OkHttp? Its DiskLruCache is developed further than this one. Someday we should just simplify this project into a repackaging of a subset of that code.

from disklrucache.