#18905: fix more leaks found in #18897
-------------------------------------+-------------------------------------
       Reporter:  dimpase            |        Owner:
           Type:  defect             |       Status:  needs_work
       Priority:  major              |    Milestone:  sage-6.8
      Component:  memleak            |   Resolution:
       Keywords:                     |    Merged in:
        Authors:  Simon King         |    Reviewers:
Report Upstream:  N/A                |  Work issues:
         Branch:                     |       Commit:
  u/SimonKing/memleak_for_integer_mod_rel_14058|  
3ada19d1acf58d1d085c1908b3058de8377901a4
   Dependencies:  #18897, #14058     |     Stopgaps:
-------------------------------------+-------------------------------------

Comment (by slabbe):

 I think the description of the ticket should be updated because I don't
 see those instances with multiple of one hundred anymore on sage-6.9.rc0.
 (Exactly what leaks are fixed in this ticket?)

 Is the following line needed?
 {{{
 +from sage.structure.parent import Parent
 }}}

 Other than that, I verified that the branches indeed fixes the problem
 mentioned at comment 33. When the two thing above are fixed, to me, it
 will be a positive review.

--
Ticket URL: <http://trac.sagemath.org/ticket/18905#comment:43>
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.

Reply via email to