Issue 3417 in pharo: BehaviorTest>>#testAllLocalCallsOn fails

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

Issue 3417 in pharo: BehaviorTest>>#testAllLocalCallsOn fails

pharo
Status: Accepted
Owner: marcus.denker
Labels: Milestone-1.2

New issue 3417 by marcus.denker: BehaviorTest>>#testAllLocalCallsOn  fails
http://code.google.com/p/pharo/issues/detail?id=3417

BehaviorTest>>#testAllLocalCallsOn  fails

... maybe because of MethodRereference is never equal to  
SourceMethodReference?


Reply | Threaded
Open this post in threaded view
|

Re: Issue 3417 in pharo: BehaviorTest>>#testAllLocalCallsOn fails

pharo
Updates:
        Cc: Benjamin.VanRyseghem.Pharo

Comment #1 on issue 3417 by luc.fabresse:  
BehaviorTest>>#testAllLocalCallsOn  fails
http://code.google.com/p/pharo/issues/detail?id=3417

yes it is the problem!
maybe the test could be changed to use SourcedMethodReference instead?
Benjamin, can you have a look at this?



Reply | Threaded
Open this post in threaded view
|

Re: Issue 3417 in pharo: BehaviorTest>>#testAllLocalCallsOn fails

pharo

Comment #2 on issue 3417 by rydier: BehaviorTest>>#testAllLocalCallsOn  
fails
http://code.google.com/p/pharo/issues/detail?id=3417

SourcedMethodReference also lacks an implementation of hash that  
corresponds to its = method.


Reply | Threaded
Open this post in threaded view
|

Re: Issue 3417 in pharo: BehaviorTest>>#testAllLocalCallsOn fails

pharo
Updates:
        Status: closed

Comment #3 on issue 3417 by [hidden email]:  
BehaviorTest>>#testAllLocalCallsOn  fails
http://code.google.com/p/pharo/issues/detail?id=3417

A possible solution would be

hash

        ^ super hash bitXor: self sourcePointer hash




Reply | Threaded
Open this post in threaded view
|

Re: Issue 3417 in pharo: BehaviorTest>>#testAllLocalCallsOn fails

pharo

Comment #4 on issue 3417 by [hidden email]:  
BehaviorTest>>#testAllLocalCallsOn  fails
http://code.google.com/p/pharo/issues/detail?id=3417

in 12305