Stop caching iterator() result

This was clearly a stupid idea as you can only iterate over the map once when it's cached.