Re: Issue 1507 in pharo: is LocatedMethod needed?

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

Re: Issue 1507 in pharo: is LocatedMethod needed?

pharo
Updates:
        Status: FixProposed
        Labels: Milestone-1.3

Comment #3 on issue 1507 by [hidden email]: is LocatedMethod needed?
http://code.google.com/p/pharo/issues/detail?id=1507

As a first step, we can just replace it by MethodReference

Fix attached. besides the changeset, this method needs to be changed in  
TTransformationCompatibility

collectMethodsFor: aSelector into: methodDescription
        (self includesSelector: aSelector) ifTrue: [
                methodDescription addLocatedMethod: (
                        MethodReference
                                class: self
                                selector: aSelector)]


Attachments:
        LocatedMethodFix.4.cs  2.2 KB


Reply | Threaded
Open this post in threaded view
|

Re: Issue 1507 in pharo: is LocatedMethod needed?

pharo
Updates:
        Status: Fixed

Comment #4 on issue 1507 by [hidden email]: is LocatedMethod needed?
http://code.google.com/p/pharo/issues/detail?id=1507

(No comment was entered for this change.)


Reply | Threaded
Open this post in threaded view
|

Re: Issue 1507 in pharo: is LocatedMethod needed?

pharo
Updates:
        Status: FixProposed

Comment #5 on issue 1507 by [hidden email]: is LocatedMethod needed?
http://code.google.com/p/pharo/issues/detail?id=1507

(No comment was entered for this change.)


Reply | Threaded
Open this post in threaded view
|

Re: Issue 1507 in pharo: is LocatedMethod needed?

pharo
Updates:
        Status: Closed

Comment #6 on issue 1507 by [hidden email]: is LocatedMethod needed?
http://code.google.com/p/pharo/issues/detail?id=1507

in 13095