$cacheFactory inefficient when no capacity

[mvolkmann]

When no capacity is specified for a cache created with the $cacheFactory service, it still maintains a linked list of all entries and checks each time an entry is added to see if the capacity has been exceeded. It would be easy to make this more efficient by skipping maintaining the linked list when it is not an LRU cache.

[IgorMinar]

yeah. this is silly. can someone send a PR? it should be a simple refactoring.

[revolunet]

Even if we don't limit the capacity, i think we still have to maintain the cache size info from the info() method, right ? If so we still need to check the item existence to maintain a correct size value. If so the change is very minor.

[IgorMinar]

@revolunet yup. I landed the patch