Issue 5841 in pharo: referredInstVars

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

Issue 5841 in pharo: referredInstVars

pharo
Status: Accepted
Owner: [hidden email]
Labels: Type-Bug Milestone-2.0

New issue 5841 by [hidden email]: referredInstVars
http://code.google.com/p/pharo/issues/detail?id=5841

referredInstVars
        | allInstVarNames instVarNames |
        allInstVarNames := self methodClass allInstVarNames.
        self isReturnField ifTrue:
                [^Set with: (allInstVarNames at: self returnField + 1)].
        instVarNames := Set new.
        self abstractBytecodeMessagesDo:
                [:msg|
                (#(#popIntoReceiverVariable:
                    #pushReceiverVariable:
                    #storeIntoReceiverVariable:) includes: msg selector) ifTrue:
                        [instVarNames add: (allInstVarNames at: msg argument + 1)]].
        ^instVarNames

        "Dictionary newFromPairs: (Point selectors collect: [:s| { s. (Point >> s)  
referredInstVars}])"



Attachments:
        CompiledMethod-referredInstVars.st  724 bytes


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

Re: Issue 5841 in pharo: referredInstVars

pharo
Updates:
        Status: FixToInclude

Comment #1 on issue 5841 by [hidden email]: referredInstVars
http://code.google.com/p/pharo/issues/detail?id=5841

(No comment was entered for this change.)


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

Re: Issue 5841 in pharo: referredInstVars

pharo

Comment #2 on issue 5841 by marianopeck: referredInstVars
http://code.google.com/p/pharo/issues/detail?id=5841

it would be nice to include a test case, like "Dictionary newFromPairs:  
(Point selectors collect: [:s| { s. (Point >> s) referredInstVars}])"  but  
with a method autogenerated.


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

Re: Issue 5841 in pharo: referredInstVars

pharo
Updates:
        Status: Integrated

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

in 2.0 061


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

Re: Issue 5841 in pharo: referredInstVars

pharo
Updates:
        Status: FixToInclude

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

not integrated because updating is broken


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

Re: Issue 5841 in pharo: referredInstVars

pharo
Updates:
        Status: Integrated

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

(No comment was entered for this change.)


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