Refactor Keys and Values iterators to only have one type parameter.

[reem]

[apasel422]

Is there a way to reduce the duplication of the RawKeyBucket and RawValueBucket structs and their impls with a generic type?

Edit: It seems like we can just replace those structs with a single RawItemBucket<T> struct and a single set of generic impls?

[Gankra]

I'm not sure it's advisable to make the source of this deviate this substantially from std. I was under the impression that this was largely intended to just add missing APIs.

[reem]

Yeah, this has just bugged me for a while and I don't think it's possible to fix in std (it would be a breaking change).

[bluss]

Just philosophizing, but having both the K and V parameters means that the internal memory layout has greater freedom to change, so in that sense dropping them is limiting future internal refactoring.

[reem]

@bluss this is true. I originally figured I would need to erase the key/value type by storing its size, I only later discovered it's unnecessary right now.