mailgun / expiringdict

Dictionary with auto-expiring values for caching purposes.
Apache License 2.0
347 stars 76 forks source link

Clean up on length call to give accurate dictionary length based on expiration #40

Open chanchiem opened 5 years ago

chanchiem commented 5 years ago

Clean up on length call to give accurate dictionary length based on expiration

Performance wise, it shouldn't make a big impact. __setitem__ calls len() to check for the length of the dictionary. This will also make it much more accurate; only expired items will be popped from the list.

But the main motive for this push is because len(expiringdict) was inaccurate. Added unit tests.