currentTimeCallback, do we need it?

[ChocoChipset]

Just wondering if we really need the currentTimeCallback parameter in SPTPersistentCacheOptions.

It makes the API more complex and I'm not sure if it's a common to require something different than the default implementation we provide.

@spotify/objc-dev

[8W9aG]

I do not think we need it 👍

[rastersize]

I agree.

In general we should try to simplify the API. For example by reducing the number of methods on SPTPersistentCache. There are lots of very similar methods that make it difficult to know which to use.

[dflems]

I originally thought we used that for testing purposes. Looks like we don't use it at all. Looks ok to remove 👍

[ChocoChipset]

Fixed and merged.