FogBugz (Case [Issue]10615) Compiler Opal - RBMethodNode classBinding: move it to compilationContext

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

FogBugz (Case [Issue]10615) Compiler Opal - RBMethodNode classBinding: move it to compilationContext

Pharo Issue Tracker
A FogBugz case was assigned to Marcus Denker by Clement Bera.

Case ID:      10615
Title:        RBMethodNode classBinding: move it to compilationContext
Status:       Work Needed (On Hold)
Category:     Bug
Project:      Compiler Opal
Area:         Misc
Priority:     3 - Must Fix
Milestone:    Pharo3.0: 30/03/2014
Assigned To:  Marcus Denker

URL:          https://pharo.fogbugz.com/default.asp?10615


Changes:
Status changed from 'Work Needed' to 'Work Needed (On Hold)'.

The problem is that :

RBMethodNode>>#compilationContext

        ^ compilationContext ifNil: [ compilationContext := self methodClass compiler compilationContext]

 

RBMethodNode>>#methodClass

        ^ classBinding value

 

And the case where compilationContext is nil happens (In the case where you parse but you don't do the semantic analysis).

Then if you use compilationContext class instead of classBinding iVar you end up with an infinite loop.

 

We could imagine that you cannot parse without semantic analysis but I don't like it. So I guess this bug is not a bug but a feature :)


You are subscribed to this case.  If you do not want to receive automatic notifications in the future, unsubscribe (https://pharo.fogbugz.com/default.asp?pre=preUnsubscribe&pg=pgEditBug&command=view&ixBug=10615) from this case.

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