#5627: [with patch, with positive review] Trivial typo in quadratic_nonresidue
-----------------------------+----------------------------------------------
Reporter: kcrisman | Owner: justin
Type: defect | Status: closed
Priority: trivial | Milestone: sage-3.4.2
Component: quadratic forms | Resolution: fixed
Keywords: |
-----------------------------+----------------------------------------------
Comment(by cremona):
Replying to [comment:12 mabshoff]:
> Merged both patches in Sage 3.4.2.alpha0.
>
> John: This is touching quadratic forms code, so I am CCing you. You
really ought to get rid of the "Oops!" bit when throwing exceptions :)
>
> Cheers,
>
> Michael
They were not my "oops"s, and I agree with you. I just did not remove
them. If we do this now with a new ticket/ patch then it will interfere
with #5834 but I think I could live with that.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/5627#comment:13>
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
-~----------~----~----~----~------~----~------~--~---
