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

Comment (by SimonKing):

 I just learnt that the codomain of an action coincides with the set that
 is acted upon. But here, we have a
 `sage.categories.action.PrecomposedAction`. So, it composes maps phi from
 left and psi from right domain with an action alpha that knows about the
 codomains of phi and psi only. And thus perhaps we have again the problem
 of keeping "the middle parent" alive.

 Namely, if the underlying set S of alpha is the codomain of psi, but psi
 is weak, then neither psi nor alpha will keep S alive. But S is sometimes
 not used as cache key for the precomposed action: Only the domains of psi
 and phi appear in the key. Hence, I think I just need to add a strong
 reference to the underlying set of alpha.

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