Issue 7013 in pharo: RBMethodNode should store class

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

New issue 7013 by [hidden email]: RBMethodNode should store class
http://code.google.com/p/pharo/issues/detail?id=7013

CompiledMethods hold on to the class binding. The RB AST does not, which  
means
that when doing name analysis, one needs to supply the class again.

The attached SLICE:
-> add classBinding iVar to RBMethodNode
-> provide #methodClass to return the class
-> #parseTree on CompiledMethod set the binding to the current binding.


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

Updates:
        Status: FixReviewNeeded

Comment #1 on issue 7013 by [hidden email]: RBMethodNode should  
store class
http://code.google.com/p/pharo/issues/detail?id=7013

Name: SLICE-Issue-7013-RBMethodNode-should-store-class-MarcusDenker.1
Author: MarcusDenker
Time: 20 November 2012, 2:49:58.498 pm
UUID: 7a11b520-646c-40de-96d6-6932d113e2e7
Ancestors:
Dependencies: AST-Core-MarcusDenker.106

Issue 7013: RBMethodNode should store class


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

Updates:
        Status: Integrated

Comment #2 on issue 7013 by [hidden email]: RBMethodNode should  
store class
http://code.google.com/p/pharo/issues/detail?id=7013

in 2.0 413


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

Comment #3 on issue 7013 by [hidden email]: RBMethodNode should  
store class
http://code.google.com/p/pharo/issues/detail?id=7013

this should actually go into the class-scope and be accessed via there...


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

Comment #4 on issue 7013 by [hidden email]: RBMethodNode should  
store class
http://code.google.com/p/pharo/issues/detail?id=7013

the problem is that the semantic analysis is different in Opal... I need to  
merge them.


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