#5596: [with patch, needs review] refactor coercion to catch fewer exceptions
----------------------+-----------------------------------------------------
 Reporter:  robertwb  |       Owner:  robertwb  
     Type:  defect    |      Status:  new       
 Priority:  major     |   Milestone:  sage-4.1.2
Component:  coercion  |    Keywords:            
 Reviewer:            |      Author:            
   Merged:            |  
----------------------+-----------------------------------------------------

Comment(by GeorgSWeber):

 Sorry, but

 > Overall, +1 for setting positive review, but I would prefer if some
 expert could double check.

 although it is always a good thing to have "some expert" double checking a
 patch.
 But who else, but you two, could you think of for reviewing this specific
 patch?

 If you could point out to me certain "suspicious" lines, I would be happy
 to look at these, and listen to your explanations, and learn something
 from it, but I fear that's all I can do. There are damn few experts around
 for programming "code doing coercions" ...

 Cheers,
 Georg

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