Makes non-const iterator values mutable

[howsohazard]

This allows for usage patterns like:

auto iter = map.find(key);
iter->second = new_value;

while respecting that const_iterator cannot update that value.

[tkhyn]

Thanks a lot for this fix, I bumped into this issue as well... Definitely preferring this over const_cast-ing results 😊