16 Mar
2012
16 Mar
'12
12:43 p.m.
I was wondering why camillo did the changes but this is good change since now we can do
union
on collection and not only set.
Stef
More precisely the crashes are due to the changes in
the Collection>>union:
The original definition was:
union: aCollection
^ self asSet addAll: aCollection; yourself
The new definition is:
union: aCollection
| set |
set := self asSet addAll: aCollection; yourself.
^ self class withAll: set asArray
This breaks the assumptions behind FMMultivalueLink because although FMMultivalueLink is
a subclass of Collection, it is not supposed to be created with a "withAll:"
constructor.