#715: Parents probably not reclaimed due to too much caching
----------------------------------------------------+-----------------------
Reporter: robertwb | Owner: somebody
Type: defect | Status:
positive_review
Priority: major | Milestone: sage-5.4
Component: coercion | Resolution:
Keywords: weak cache coercion Cernay2012 | Work issues:
Report Upstream: N/A | Reviewers:
Jean-Pierre Flori, Simon King, Nils Bruin
Authors: Simon King, Jean-Pierre Flori | Merged in:
Dependencies: #13145, to be merged with #11521 | Stopgaps:
----------------------------------------------------+-----------------------
Changes (by nbruin):
* status: needs_review => positive_review
* dependencies: #13145, #13447, to be merged with #11521 => #13145, to
be merged with #11521
Comment:
Removing dependency #13447 again, because it looks like that ticket is not
close to resolution. In the mean time, leaving polynomial rings immortal
is not a regression compared to previous behaviour. Note that while the
issue was only diagnosed on OSX, deallocation of polynomial rings indeed
leads to potential write-after-free, so the osx64-dealloc patch should be
applied universally.
That means we're back at comment:317 and positive review. We should really
get this merged. At least the libsingular interface is not worse than it
was before. Proper coordination of libsingular and python memory
management shouldn't hold up reclaiming of other rings.
Apply trac_715_combined.patch trac_715_local_refcache.patch
trac_715_safer.patch trac_715_specification.patch
trac_715_osx64-dealloc.patch
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/715#comment:324>
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 post to this group, send email to [email protected].
To unsubscribe from this group, send email to
[email protected].
For more options, visit this group at
http://groups.google.com/group/sage-trac?hl=en.
