Issue 6018 in pharo: Fixing MCSynchronisationTest

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

Issue 6018 in pharo: Fixing MCSynchronisationTest

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

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

Here is one of the new definition but it does not solve many problems

RPackageOrganizer>>systemMethodAddedActionFrom: ann

        |class selector protocol method rPackage |
        'methodAdded' crLog.
        class := ann itemClass.
        protocol := ann itemProtocol.
        selector := ann itemSelector.
        method := ann itemMethod.
       
        (protocol beginsWith: '*') "we should pattern search the right package but  
for now let us do it simply"
                ifTrue: [ rPackage := packages at: (protocol copyWithout: $*) asSymbol]
                ifFalse: [ rPackage := class package ].
       
        class isTrait
                        ifTrue: [
                                        "we do not use addMethod: since the method class is probably the trait  
user class and not the trait.
                                        So we would be adding the method in the package of the class using the  
trait instead of the one of the trait."
                                        class isMeta
                                                ifTrue: [rPackage addSelector: selector ofMetaclassName: class  
instanceSide name]
                                                ifFalse: [rPackage addSelector: selector ofClassName: class  
instanceSide name] ]
                        ifFalse: [ rPackage addMethod: 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 6018 in pharo: Fixing MCSynchronisationTest

pharo

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

Working on it but burned out....


_______________________________________________
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 6018 in pharo: Fixing MCSynchronisationTest

pharo

Comment #2 on issue 6018 by [hidden email]: Fixing  
MCSynchronisationTest
http://code.google.com/p/pharo/issues/detail?id=6018

Steph, I think this is a duplicated for Issue 5977.  I already proposed a  
solution+slice in there.


_______________________________________________
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 6018 in pharo: Fixing MCSynchronisationTest

pharo
Updates:
        Status: Duplicate
        Mergedinto: 5977

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

(No comment was entered for this change.)


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