[squeak-dev] Re: [Pharo-dev] IdentitySet>>collect:

Marcus Denker marcus.denker at inria.fr
Fri Nov 28 14:35:25 UTC 2014


> On 27 Nov 2014, at 08:21, Marcus Denker <marcus.denker at inria.fr> wrote:
> 
> 
>> On 26 Nov 2014, at 19:19, Eliot Miranda <eliot.miranda at gmail.com> wrote:
>> 
>> Hi All,
>> 
>>    IdentitySet>>collect: answers a Set, not an IdentitySet.  Anyone else agree this is a serious bug?  Anyone else disagree?
>> 
>> WTF??
>> 
>> (IdentitySet withAll: #(1 2 3 1.0 2.0 3.0)) collect: [:e| e] a Set(1.0 2 3)
> 
> Yes, I would say so. I think we fixed some others like that already (not 100% sure)
> 
> I added an issue tracker entry
> 
> 14535	IdentitySet>>collect: answers a Set, not an IdentitySet.
> 	https://pharo.fogbugz.com/f/cases/14535
> 

#collect: directly uses Set in Set>>#collect, I committed a fix (to the issue tracker, to be reviewed) 
to use “self species” instead (plus the example from above as a test in IdentitySetTest).

	Marcus



More information about the Squeak-dev mailing list