#18739: Wrap PARI functions for converting relative number field elements
-------------------------------------+-------------------------------------
       Reporter:  pbruin             |        Owner:
           Type:  enhancement        |       Status:  needs_work
       Priority:  minor              |    Milestone:  sage-6.8
      Component:  interfaces         |   Resolution:
       Keywords:  pari               |    Merged in:
        Authors:  Peter Bruin        |    Reviewers:  Frédéric Chapoton,
Report Upstream:  N/A                |  Jeroen Demeyer
         Branch:                     |  Work issues:
  public/ticket/18739                |       Commit:
   Dependencies:                     |  5373b8eeeeca1aef7816c85ccdf36d576850375d
                                     |     Stopgaps:
-------------------------------------+-------------------------------------

Comment (by pbruin):

 Replying to [comment:9 jdemeyer]:
 > Please replace
 > {{{
 > This method may raise errors or
 > return undefined results if called with invalid arguments.
 > }}}
 > by
 > {{{
 > This method may segfault or
 > return undefined results if called with invalid arguments.
 > }}}
 I thought about phrasing it like that, but to me "segfault" (besides
 perhaps being somewhat cryptic to many users) makes it sound like Sage
 crashes if this is called with the wrong arguments.  In fact, the
 segmentation fault is caught and a `SignalError` is raised.  Also, it is
 entirely possible that PARI will throw other kinds of errors for certain
 inputs.

--
Ticket URL: <http://trac.sagemath.org/ticket/18739#comment:10>
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 unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/d/optout.

Reply via email to