#14711: Weak references in the coercion graph
-------------------------------------+-------------------------------------
Reporter: jpflori | Owner: davidloeffler
Type: defect | Status: needs_review
Priority: critical | Milestone: sage-6.2
Component: number fields | Resolution:
Keywords: memleak, number | Merged in:
field, QuadraticField | Reviewers: Nils Bruin, Jean-
Authors: Simon King, | Pierre Flori
Travis Scrimshaw, Jean-Pierre | Work issues:
Flori | Commit:
Report Upstream: N/A | f68550d17989e50adecc9c1d6634109095174003
Branch: | Stopgaps:
public/ticket/14711 |
Dependencies: |
-------------------------------------+-------------------------------------
Comment (by tscrim):
Rebased on 6.2.beta5.
I'm not in favor of having any warning message displayed as part of an
error message. Here's another option then (a variant on the optional arg):
Define `_repr_no_warning()` which does the main bulk of the string
construction and then `_repr_` just adds the warning. Then for things
which want to include the a possible internal map, they just call
`_repr_no_warning()`.
Variant on the above: instead of `_repr_no_warning()`, we override
`__str__()` and have `TriangularModuleMorphism` call `str(self)`.
One more option: we change the warning message in
`TriangularModuleMorphism` to not call `repr(self)` and instead spell some
things out.
----
At some point we will have to do something (one of us) considers evil. If
people en masse think leaving an error message in there is the best course
of action, I'll accept it.
Thoughts?
--
Ticket URL: <http://trac.sagemath.org/ticket/14711#comment:249>
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.
