#12215: Memleak in UniqueRepresentation, @cached_method
--------------------------------------------------------------+-------------
Reporter: vbraun | Owner:
Type: defect | Status:
positive_review
Priority: major | Milestone:
sage-5.7
Component: memleak | Resolution:
Keywords: UniqueRepresentation cached_method caching | Work issues:
Report Upstream: N/A | Reviewers:
Nils Bruin
Authors: Simon King | Merged in:
Dependencies: | Stopgaps:
--------------------------------------------------------------+-------------
Comment (by SimonKing):
Replying to [comment:145 jdemeyer]:
> Replying to [comment:144 nbruin]:
> > We only introduced a codependence on #12313 because of comment
[comment:133]. In view of comment [comment:141] I suspect the source of
the problem noted in comment [comment:131] was actually solved, rather
than hidden by merging tickets together.
> >
> > Hence, I propose that this ticket is considered without dependencies
and be considered for merging in sage-5.7 anyway.
>
> Just to have a second opinion: Simon, do you agree?
Yes. Note also comment:52: It used to be the case that both #12215 and
#12313 were fine ''separately'', but problems occurred when they were used
''together''. But in later patch versions or Sage versions, it was
observed that having them together makes the Heisenbug magically disappear
- and the suggestion to merge them together was based on this observation.
Now it very much seems that the Cython upgrade was enough to fix the
crashes. We should of course verify that no problems occur when #12215 is
merged without #12313, but I think there is no reason to merge #12215 and
#12313 together.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/12215#comment:146>
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.
