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