[Seaside-dev] Grease-Core-lr.48

[Philippe Marschall]

2010/9/10 Lukas Renggli <renggli at gmail.com>:
>> I'm not happy at all with the #hash changes. It's a really bad
>> implementation of #hash, only slightly better than:
>>
>> #hash
>>   ^ 42
>
> The problem with the old implementation was that it was incorrect, as
> its method comment stated itself. Dictionaries that were equal did not
> necessary have the same hash.
>
>> I'm also not happy with the #= changes. Does #isDictionary cause
>> problems on any dialect? The change introduces commutativity problems:
>>
>> Dictionary new = GRSmallDictionary new -> true
>> GRSmallDictionary new = Dictionary new -> false
>
> Pharo and Squeak are likely the only Smalltalk that have #isDictionary.

Then we should probably move it to Grease-Pharo-Core or kick it.

> Actually #species would be the thing to use to avoid such problems,
> but Pharo doesn't do that.
>
> Where is this functionality used? This is fixable, but is it really
> worth the trouble?

#= is quite helpful for unit tests, but yes, given the option that we
can only do it wrong, we should probably kick them both and work
around it in the unit tests.

Cheers
Philippe