#14894: Fix PARI error handling (needed for future upgrade to PARI 2.6)
--------------------------------------+-------------------------------
Reporter: vbraun | Owner: jdemeyer
Type: defect | Status: needs_review
Priority: major | Milestone: sage-5.12
Component: packages: standard | Resolution:
Keywords: pari error signal | Merged in:
Authors: Peter Bruin | Reviewers:
Report Upstream: N/A | Work issues:
Branch: | Dependencies: #12142, #14873
Stopgaps: |
--------------------------------------+-------------------------------
Comment (by pbruin):
Replying to [comment:11 jdemeyer]:
> One recomendation: could you use other names than `sage_pari_catch`,
something which clearly refers to `sig_on()`, since `sage_pari_catch()`
really is an extension of `sig_on()`.
As I understand it, the two purposes of `sage_pari_catch()` are equally
important, namely
- catching signals
- catching PARI errors.
It would be nice to have a name that mirror this. Maybe the prefix
`sage_` is somewhat redundant, as Volker Braun suggested, and we could
call them `pari_catch_on()` and `pari_catch_off()`?
> What about `pari_sig_on()` and `pari_sig_off()` which would mirror
`ecl_sig_on()` and `ecl_sig_off()` which we already have?
Do these have the same two purposes as above? In that case, I would find
the names `pari_sig_on()` and `pari_sig_off()` acceptable for the sake of
consistency, but in principle I dislike the fact that they don't hint at
the fact that these macros catch PARI errors and not just signals.
--
Ticket URL: <http://trac.sagemath.org/ticket/14894#comment:14>
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/groups/opt_out.
