#14894: Fix PARI error handling
-------------------------------------+-------------------------------------
       Reporter:  vbraun             |        Owner:  jdemeyer
           Type:  defect             |       Status:  needs_review
       Priority:  major              |    Milestone:  sage-6.5
      Component:  packages:          |   Resolution:
  standard                           |    Merged in:
       Keywords:  pari error signal  |    Reviewers:
        Authors:  Peter Bruin,       |  Work issues:
  Jeroen Demeyer                     |       Commit:
Report Upstream:  N/A                |  0602271c328f6d0851af946671194786c8c78556
         Branch:                     |     Stopgaps:
  u/jdemeyer/ticket/14894            |
   Dependencies:  #12142, #14873     |
-------------------------------------+-------------------------------------
Changes (by {'newvalue': u'Peter Bruin, Jeroen Demeyer', 'oldvalue': u'Peter 
Bruin'}):

 * status:  needs_info => needs_review
 * reviewer:  Jeroen Demeyer =>
 * author:  Peter Bruin => Peter Bruin, Jeroen Demeyer


Old description:

> Instead of catching whatever PARI writes to `pariErr`, we should use a
> callback function `cb_pari_prehandle_exception`. That way, we don't need
> special tricks for warnings anymore.

New description:

 Instead of catching whatever PARI writes to `pariErr`, we should use a
 callback function `cb_pari_err_handle`. That way, we don't need special
 tricks for warnings anymore.

 Moreover, we also get access to the "error data" (a `t_ERROR` object)
 which we store in the exception.

--

--
Ticket URL: <http://trac.sagemath.org/ticket/14894#comment:57>
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