search

mbr / simplekv

A simple key-value store for binary data.
http://simplekv.readthedocs.io
MIT License
152 stars 50 forks source link

Fix problem that the user is able to change the dictonary while we iterate over it #54

Closed criemen closed 7 years ago

criemen commented 7 years ago

The test demonstrates the problem and fails without the fix.

criemen commented 7 years ago

This is not necessarily a good idea. We can also force the user to use keys() instead of iter_keys() when modifying the store during the iteration and everything works.