Hello,  

       I'm trying to become forward some objects, these objects are stored in IdentitySets so I need to copy the identityHash of the older objects to the new ones. So that the IdentitySets are still well constructed.

I have read the comments in Array >> elementsForwardIdentityTo: copyHash: , and Array >> elementsForwardIdentityTo:, so I realise that the operation I want to use is the first with copyHash in true or the later (that should be doing the same, from the comments).

But after trying it, it is not working as expected. 

If I try with:

x := Object new.

y := Object new.

z := Object new.

set := IdentitySet new.

set add: x.

set add: y.

hash := x identityHash.

bhash := x basicIdentityHash.

self assert:(set isHealthy).

z becomeForward: x copyHash: true.

set isHealthy "false"

z basicIdentityHash  = bhash. "false"

z identityHash = hash "false"

It is not working, but if I changed the copyHash parameter to false, it works. 

Something is not right (the comment or the implementation), because the comment says:

"If copyHash is true, the identityHashes remain with the pointers rather than with the

objects so that the objects in the receiver should still be properly indexed in any

existing hashed structures after the mutation.  If copyHash is false, then the hashes

of the objects in otherArray remain unchanged."

Do I have understanded everything wrong or there is something that is not working (or at least the comment is not clear)

Is somebody using these operations?  How do you expect to work?

Maybe If you can  help me to understand the situation I can fix the comments. 

Thanks in advance.

--

Pablo Tesone.
tesonep@gmail.com