#9438: IntegerMod_int.log has side-effect in Pari
--------------------------------+-------------------------------------------
Reporter: SimonKing | Owner: AlexGhitza
Type: defect | Status: needs_review
Priority: critical | Milestone: sage-4.5
Component: basic arithmetic | Keywords: log finite field pari
Author: Simon King | Upstream: N/A
Reviewer: | Merged:
Work_issues: |
--------------------------------+-------------------------------------------
Changes (by newvalueoldvalue):
* status: new => needs_review
* author: => Simon King
Comment:
The patch seems to fix the bug. The string that is evaluated by PARI
defines a variable 'b' and uses it exactly twice. The definition of 'b' is
not so long: It is {{{b=Mod(m,n)}}}. OK, m and n can be large numbers. But
still, I think the cleanest solution is to expand the definition and
insert {{{Mod(m,n)}}} twice in the string.
sage -t "devel/sage/sage/rings/finite_rings/integer_mod.pyx" passes, I now
do sage -testall
Of course, I added a doctest against the bug.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/9438#comment:1>
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.
