#10112: random_prime does not handle erroneous input gracefully - it hangs
-----------------------------+----------------------------------------------
Reporter: drkirkby | Owner: was
Type: defect | Status: needs_review
Priority: major | Milestone: sage-4.6.1
Component: number theory | Keywords:
Author: Mike Hansen | Upstream: N/A
Reviewer: | Merged:
Work_issues: |
-----------------------------+----------------------------------------------
Comment(by drkirkby):
Mike,
thank you for fixing this. I'll try this later today, but here's a few
comments from reading the patch only - I've not tested it in Sage yet.
Should the example on line 1127 be
{{{random_prime(200, proof=False, lbound=100) }}} ?
Currently it's the same as the previous example.
One thing that would be nice is if you did like John did at #10105 and
have doctests for the commands that currently hang Sage. Then if there are
any regressions, and the bug(s) get back in, this will be detected.
Perhaps I'll get bored of running my little script that supplies junk to
commands to see what ones crash sage (like #10113) or hang Sage like this
ticket, #10108 and #10105.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/10112#comment:2>
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.
