#4746: [with patch, positive review] Bug in srange
---------------------+------------------------------------------------------
 Reporter:  vgermrk  |       Owner:  cwitty    
     Type:  defect   |      Status:  new       
 Priority:  major    |   Milestone:  sage-3.4.1
Component:  misc     |    Keywords:            
---------------------+------------------------------------------------------

Comment(by cwitty):

 I had to rebase the patch (the added/removed lines are unchanged).  Also,
 three doctests failed in interact.py; William said in IRC to change the
 expected results:
 {{{
  Should I just change the expected result?
 <wstein> The Got definitely looks more sensible.
  So yes, do change the expected result.
 }}}

 Code looks good, all doctests pass.  Positive review.  Apply 4746-srange-
 rebased.patch and 4746-referee.patch.

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/4746#comment:7>
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