Issue 4551 in pharo: MessageAsTempNode does not understand beMethodArg

Status: Accepted
Owner: [hidden email]
Labels: Milestone-1.4

New issue 4551 by [hidden email]: MessageAsTempNode does not  
understand beMethodArg
http://code.google.com/p/pharo/issues/detail?id=4551



DoItIn: ThisContext
        ^ self
                do: [:each | ((ThisContext namedTempAt: 1)
                                        includes: each)
                                ifFalse: [^ false]]


I selected (ThisContext namedTempAt: 1

and I got MessageAsTempNode does not understand beMethodArg

method: doit context: ctxt
        " pattern [ | temporaries ] block => MethodNode."

        | sap blk prim temps messageComment methodNode |
        properties := AdditionalMethodState new.
        sap := self pattern: doit inContext: ctxt.
        "sap={selector, arguments, precedence}"
        properties selector: (sap at: 1).
        encoder selector: (sap at: 1).
        (sap at: 2) do: [:argNode | argNode beMethodArg].
                                                ^^^^
        doit ifFalse: [self pragmaSequence].
        temps := self temporaries.
        messageComment := currentComment.
        currentComment := nil.
        doit ifFalse: [self pragmaSequence].
        prim := self pragmaPrimitives.
        self statements: #() innerBlock: doit.
        blk := parseNode.
        doit ifTrue: [blk returnLast]
                ifFalse: [blk returnSelfIfNoOther: encoder].
        hereType == #doIt ifFalse: [^self expected: 'Nothing more'].
        self interactive ifTrue: [self removeUnusedTemps].
        methodNode := self newMethodNode comment: messageComment.
        ^methodNode
                selector: (sap at: 1)
                arguments: (sap at: 2)
                precedence: (sap at: 3)
                temporaries: temps
                block: blk
                encoder: encoder
                primitive: prim
                properties: properties


argNode is a MessageAsTempNode and beMethodArg is only defined on  
TempVariableNode


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

Comment #1 on issue 4551 by [hidden email]: MessageAsTempNode does  
not understand beMethodArg
http://code.google.com/p/pharo/issues/detail?id=4551


What about licensing issues?

I don't know. The implementation is proprietary and there is no source
code available. I would just take the architecture/API as inspiration
for an implementation in Smalltalk. Clutter
(http://www.clutter-project.org/) and an uncountable number of
Javascript libraries contain open-source implementation of such
animation frameworks (LGPL though).



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

Updates:
        Labels: Type-Bug

Comment #2 on issue 4551 by [hidden email]: MessageAsTempNode does  
not understand beMethodArg
http://code.google.com/p/pharo/issues/detail?id=4551

(No comment was entered for this change.)


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

Comment #3 on issue 4551 by [hidden email]: MessageAsTempNode does  
not understand beMethodArg
http://code.google.com/p/pharo/issues/detail?id=4551

How can this be reproduced?


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

Updates:
        Status: Invalid

Comment #4 on issue 4551 by [hidden email]: MessageAsTempNode does  
not understand beMethodArg
http://code.google.com/p/pharo/issues/detail?id=4551

I can not reproduce it.


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