Hi,
as you mentioned earlier there is no real reason to use a identityhashmap as the units do not implement them. i thought this could avoid bugs in cases where someone does not use units. I think i will undo the changes to minimize possible sideeffects