Issue 7349 in pharo: rootApplication does not exist in Pharo

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

Issue 7349 in pharo: rootApplication does not exist in Pharo

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

New issue 7349 by [hidden email]: rootApplication does not exist in  
Pharo
http://code.google.com/p/pharo/issues/detail?id=7349

So

canCall: aSelector in: aClass from: anApplication
        | methodApp root |
        (aClass canUnderstand: aSelector) ifFalse: [^false].
        root := anApplication rootApplication.
        methodApp := ((aClass whichClassIncludesSelector: aSelector)  
compiledMethodAt: aSelector)
                                application rootApplication.
        ^methodApp == root or: [root isBasedOn: methodApp]

looks bogus and the methods that invoke it too :)


_______________________________________________
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 7349 in pharo: rootApplication does not exist in Pharo

pharo

Comment #1 on issue 7349 by [hidden email]: rootApplication does not  
exist in Pharo
http://code.google.com/p/pharo/issues/detail?id=7349

false negative, we added a comment to the method


_______________________________________________
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 7349 in pharo: rootApplication does not exist in Pharo

pharo

Comment #2 on issue 7349 by [hidden email]: rootApplication does not  
exist in Pharo
http://code.google.com/p/pharo/issues/detail?id=7349

This is fixed, I can not edit the tag


_______________________________________________
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 7349 in pharo: rootApplication does not exist in Pharo

pharo
Updates:
        Status: Closed

Comment #3 on issue 7349 by [hidden email]: rootApplication does not  
exist in Pharo
http://code.google.com/p/pharo/issues/detail?id=7349

(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 7349 in pharo: rootApplication does not exist in Pharo

pharo
Updates:
        Status: FixToInclude

Comment #4 on issue 7349 by [hidden email]: rootApplication does not  
exist in Pharo
http://code.google.com/p/pharo/issues/detail?id=7349

We should harvest the comment.
I do not want in 2 months from now get the same problem


_______________________________________________
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 7349 in pharo: rootApplication does not exist in Pharo

pharo
Updates:
        Status: Closed

Comment #5 on issue 7349 by [hidden email]: rootApplication does not  
exist in Pharo
http://code.google.com/p/pharo/issues/detail?id=7349

integrated in 20496


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