On 28 Nov 2014, at 15:35, Marcus Denker <marcus.denker@inria.fr> wrote:
On 27 Nov 2014, at 08:21, Marcus Denker <marcus.denker@inria.fr> wrote:
On 26 Nov 2014, at 19:19, Eliot Miranda <eliot.miranda@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
I think that is the most logical thing to do, it is what I would expect. I know others have cases where that is not really what you want (some of Set's subclasses are pretty special), but it is still a good default, while #collect:as: can be used for all other cases (and subclasses can override #species again if they want). Sven