#4988: [with patch, needs review] major easy-to-fix but STUPID bug in gcd
---------------------------+------------------------------------------------
Reporter: was | Owner: was
Type: defect | Status: new
Priority: major | Milestone: sage-3.3
Component: number theory | Resolution:
Keywords: |
---------------------------+------------------------------------------------
Changes (by cremona):
* summary: major easy-to-fix but STUPID bug in gcd => [with patch, needs
review] major easy-to-fix but STUPID bug in gcd
Comment:
Patch attached. I deleted the now redundant integer parameter (which once
was used to tell the function to use integer-specific code and is now
redundant). I added a relevant doctest and some more so hopefully
William's confusion will never again occur (in someone else, I mean ;)).
I discovered some places which still had "integer=True" in gcd calls and
fixed those. I tested all rings/ and used search_src() to find any other
places where "integer=True" might have been used, and found that
search_src("integer=True") runs for ever while producing no output.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/4988#comment:1>
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
-~----------~----~----~----~------~----~------~--~---
