#14711: Weak references in the coercion graph
-------------------------------------+-------------------------------------
Reporter: jpflori | Owner: davidloeffler
Type: defect | Status: needs_review
Priority: critical | Milestone: sage-6.1
Component: number fields | Resolution:
Keywords: memleak, number | Merged in:
field, QuadraticField | Reviewers: Jean-Pierre Flori
Authors: Simon King | Work issues:
Report Upstream: N/A | Commit:
Branch: | 0af59ea93689cb6abb9d3fae0f1cf11f2aee5cca
u/jpflori/ticket/14711 | Stopgaps:
Dependencies: |
-------------------------------------+-------------------------------------
Comment (by jpflori):
Sorry about the blank lines, I think I've been overzealous and played too
much with vim.
I personally prefer not leaving superfluous blank lines and didn't get the
impression it was an informal convention but I don't really care what ends
up in the final branch.
I was thinking about the todo in the ticket description:
- Provide a documentation of the use of weak references in coercion, and
of
different ways of registering coercions, with their different impacts on
garbage collecion.
At the moment, I feel like the ticket could be merged.
There's Travis issue with perfectly make sense, but how harmful would it
be to leave that for another ticket?
Would it really involve more work and rebasing than doing it here?
Nils idea would do the trick and doesn't look that hard at first but I
feel it won't be that easy to ensure that each current incarnations of
coerce_map_from have been correctly left as is when outside of the
coercion framework but had an underscore added within the coercion
framework (or we'll jsut end up with random memleaks again :().
--
Ticket URL: <http://trac.sagemath.org/ticket/14711#comment:176>
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.
