#14711: Memleak when creating QuadraticField
-------------------------------------------------+-------------------------
       Reporter:  jpflori                        |        Owner:
           Type:  defect                         |  davidloeffler
       Priority:  critical                       |       Status:  new
      Component:  number fields                  |    Milestone:  sage-5.12
       Keywords:  memleak, number field,         |   Resolution:
  QuadraticField                                 |    Merged in:
        Authors:                                 |    Reviewers:
Report Upstream:  N/A                            |  Work issues:
         Branch:                                 |       Commit:
   Dependencies:                                 |     Stopgaps:
-------------------------------------------------+-------------------------

Comment (by SimonKing):

 Replying to [comment:17 jpflori]:
 > > However, I don't see why one shouldn't use a `WeakValueDictionary`
 instead.
 > Me neither, unless someone has the habit to play frequently with the
 same number field but without keeping any of its elements alive between
 different uses, personally I don't and would not really see the point.
 >
 > If we switch to a weak value dict, we could also get rid of the cache
 argument of the constructor then which won't be that useful/sensible
 anymore.

 I think it isn't sensible anyway. But that's not the point. We first need
 to find out what keeps the fields alive, when using the branch that I am
 now about to push. Wait a minute...

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

Reply via email to