#10304: PolynomialRing_field.lagrange_polynomial doesn't always return a 
polynomial
-----------------------------------+----------------------------------------
   Reporter:  mguaypaq             |       Owner:  mguaypaq                     
       
       Type:  defect               |      Status:  positive_review              
       
   Priority:  major                |   Milestone:  sage-4.6.1                   
       
  Component:  algebra              |    Keywords:  polynomial ring, lagrange 
polynomial
     Author:  Mathieu Guay-Paquet  |    Upstream:  N/A                          
       
   Reviewer:  Minh Van Nguyen      |      Merged:                               
       
Work_issues:                       |  
-----------------------------------+----------------------------------------
Changes (by mguaypaq):

  * status:  needs_review => positive_review


Comment:

 I thought the use of a mutable default was safe, since it is always
 copied, never modified. But since there's an idiom against this, I'm happy
 to comply! And thanks for fixing the documentation issues.

 Positive review for the reviewer patch.

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/10304#comment:3>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica, 
and MATLAB

-- 
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