#12313: Fix yet another memory leak caused by caching of coercion data
---------------------------------------------------------------------------+
Reporter: SimonKing |
Owner:
Type: defect |
Status: positive_review
Priority: major |
Milestone: sage-5.8
Component: memleak |
Resolution:
Keywords: coercion weak dictionary |
Work issues:
Report Upstream: N/A |
Reviewers: Simon King, Jean-Pierre Flori, John Perry, Nils Bruin
Authors: Simon King, Jean-Pierre Flori |
Merged in:
Dependencies: #715, #11521, #12215, #13746, #13378; merge with #13387 |
Stopgaps:
---------------------------------------------------------------------------+
Changes (by nbruin):
* status: needs_review => positive_review
Comment:
Why add that patch here when it belongs on #14159? This ticket already has
positive review. It's bad form to start changing the code unless
absolutely necessary and that new patch has nothing to do with this
ticket. I've uploaded your patch there (and changed the commit message to
reflect the appropriate ticket number). I hope that's OK with you. If so,
you can just remove the ticket here from the list and move the ticket back
to "Positive". We really want to get this merged, especially #13387 (which
depends on this ticket) because that fixes some issues with `TripleDict`
that could already bite its uses in 5.6 and 5.7.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/12313#comment:329>
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?hl=en.
For more options, visit https://groups.google.com/groups/opt_out.
