Re: [Pharo-project] bug in the code of CompiledMethod >> =

You might also want to check last code published in trunk.

Nicolas

2010/3/22 Eliot Miranda <[hidden email]>:

>
>
> On Mon, Mar 22, 2010 at 2:25 PM, Stéphane Ducasse
> <[hidden email]> wrote:
>>
>> eliot
>>
>> Your changes fixed some of the problems. Now I tried to fix the test
>> testAnalogousCodeTo in MethodPropertiesTest and
>> the expression returns false.
>>
>> (#zork->'hello') analogousCodeTo: (#zork->'hello')  -> false
>>
>> (#zork->'hello') = (#zork->'hello') -> true
>
> For me
>     #(#zork -> 'hello') size 3
> In Pharo is it true that
> #(#zork -> 'hello') size 1
> ??!?!
> In that case I would implement analogousCodeTo: in Association, e.g.
> analogousCodeTo: anObject
>     ^anObject class == self class
>     and: [key = anObject key
>     and: [value = anObject value]]
>
> But if Association isn't a literal I would simply check
> MethodPropertiesTests.  It could be obsolete.
>>
>> Stef
>>
>> On Mar 22, 2010, at 6:20 PM, Eliot Miranda wrote:
>>
>> > Oops.  Find attached.
>> >
>> > 2010/3/22 Cyrille Delaunay <[hidden email]>
>> > try in a Pharo-1.0-10515-rc3 image :
>> >
>> > |set|
>> > set := Set new.
>> > Collection withAllSubclasses do: [:aClass |
>> >       set addAll: aClass methods
>> >       ].
>> >
>> > It will raise an exception telling: "MessageNotUnderstood:
>> > ByteSymbol>>analogousCodeTo:".
>> > Indeed, in the code of CompiledMethod >> = , the message
>> > 'analogousCodeTo:' is send to a
>> > symbol.
>> >
>> >
>> > I opened an Issue:
>> > http://code.google.com/p/pharo/issues/detail?id=2185
>> >
>> >
>> >
>> > _______________________________________________
>> > Pharo-project mailing list
>> > [hidden email]
>> > http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>> >
>> >
>> > <Object-analogousCodeTo.st>_______________________________________________
>> > Pharo-project mailing list
>> > [hidden email]
>> > http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>>
>>
>> _______________________________________________
>> Pharo-project mailing list
>> [hidden email]
>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>
>
> _______________________________________________
> Pharo-project mailing list
> [hidden email]
> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>

_______________________________________________
Pharo-project mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project