#5092: [with patch; needs work] Primes()?? gets hung in len call; also bring
coverage to 100% for primes.py
---------------------------+------------------------------------------------
 Reporter:  was            |        Owner:  was     
     Type:  defect         |       Status:  new     
 Priority:  major          |    Milestone:  sage-3.3
Component:  number theory  |   Resolution:          
 Keywords:                 |  
---------------------------+------------------------------------------------
Changes (by saliola):

  * summary:  [with patch; needs review] Primes()?? gets hung in len call;
              also bring coverage to 100% for primes.py =>
              [with patch; needs work] Primes()?? gets hung
              in len call; also bring coverage to 100% for
              primes.py

Comment:

 You don't need the first two lines below in __cmp__ anymore.
 {{{
         if isinstance(right, Primes_class):
             return 0
         return cmp(type(self), type(right))
 }}}

 Otherwise, it's a positive review.

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/5092#comment:3>
Sage <http://sagemath.org/>
Sage - Open Source Mathematical Software: Building the Car Instead of 
Reinventing the Wheel
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups 
"sage-trac" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to 
[email protected]
For more options, visit this group at 
http://groups.google.com/group/sage-trac?hl=en
-~----------~----~----~----~------~----~------~--~---

Reply via email to