PackageInfo>>addMethod:

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
2 messages Options
Reply | Threaded
Open this post in threaded view
|

PackageInfo>>addMethod:

Alexandre Bergel
This method is defined as:

-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=
addMethod: aMethodReference
        (self includesClass: aMethodReference class)
                ifTrue: [self addCoreMethod: aMethodReference]
                ifFalse: [self addExtensionMethod: aMethodReference]
-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=

I would have expected #actualClass to be sent to aMethodReference.
This method has been written a long time ago, it probably not relevant  
to fix the bug, since method extension are handled by Monticello.

Alexandre

--
_,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:
Alexandre Bergel  http://www.bergel.eu
^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;.






_______________________________________________
Pharo-project mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
Reply | Threaded
Open this post in threaded view
|

Re: PackageInfo>>addMethod:

Stéphane Ducasse
any bug is good to fix :)
this is a nice catch.

Stef

On Aug 13, 2009, at 4:40 PM, Alexandre Bergel wrote:

> This method is defined as:
>
> -=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=
> addMethod: aMethodReference
> (self includesClass: aMethodReference class)
> ifTrue: [self addCoreMethod: aMethodReference]
> ifFalse: [self addExtensionMethod: aMethodReference]
> -=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=
>
> I would have expected #actualClass to be sent to aMethodReference.
> This method has been written a long time ago, it probably not relevant
> to fix the bug, since method extension are handled by Monticello.
>
> Alexandre
>
> --
> _,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:
> Alexandre Bergel  http://www.bergel.eu
> ^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;.
>
>
>
>
>
>
> _______________________________________________
> 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