Issue 4964 in pharo: Number nthRoot: should be subclassResponsibility

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

Issue 4964 in pharo: Number nthRoot: should be subclassResponsibility

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

New issue 4964 by [hidden email]: Number nthRoot: should be  
subclassResponsibility
http://code.google.com/p/pharo/issues/detail?id=4964

When extending Number protocol in issue 4907 I forgot to implement  
#nthRoot: in Number. This is a bug, because the message is sent from  
Number>>raisedTo:, so a new Number subclass might fail.

The best thing to do is to implement it with a subclassResponsibility.

It is also better to now let #sqrt be of subclassResponsibility.


_______________________________________________
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 4964 in pharo: Number nthRoot: should be subclassResponsibility

pharo
Updates:
        Status: FixToInclude

Comment #1 on issue 4964 by [hidden email]: Number nthRoot: should  
be subclassResponsibility
http://code.google.com/p/pharo/issues/detail?id=4964

Name:  
SLICE-Issue-4964-Number-nthRoot-should-be-subclassResponsibility-nice.1
Author: nice
Time: 1 November 2011, 3:33:58 pm
UUID: 7ea4ce8c-68f7-413b-901a-13985d6ae2ee
Ancestors:
Dependencies: Kernel-nice.964

Fix issue 4964 Number nthRoot: should be subclassResponsibility
And #sqrt 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 4964 in pharo: Number nthRoot: should be subclassResponsibility

pharo
Updates:
        Status: Closed

Comment #2 on issue 4964 by [hidden email]: Number nthRoot: should  
be subclassResponsibility
http://code.google.com/p/pharo/issues/detail?id=4964

Thanks a lot nicolas. We are running after time and we deeply appreciate  
the duplicated effort you are doing (I mean pushing to squeak and pharo)..
Integrated in 14215


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