#14058: Weakly reference binary operation codomains
-------------------------------------+-------------------------------------
       Reporter:  robertwb           |        Owner:  rlm
           Type:  enhancement        |       Status:  positive_review
       Priority:  major              |    Milestone:  sage-6.4
      Component:  memleak            |   Resolution:
       Keywords:                     |    Merged in:
        Authors:  Robert Bradshaw,   |    Reviewers:  Simon King, Frédéric
  Nils Bruin                         |  Chapoton, Jean-Pierre Flori
Report Upstream:  N/A                |  Work issues:
         Branch:                     |       Commit:
  public/ticket/14058                |  8713960717f916d459e0b6a0dcef391dc992d9b2
   Dependencies:  #12313             |     Stopgaps:
-------------------------------------+-------------------------------------
Changes (by jpflori):

 * status:  needs_review => positive_review
 * reviewer:  Simon King => Simon King, Frédéric Chapoton, Jean-Pierre Flori


Old description:

> R(1) + S(1) caches a strong reference to both R and S, which we'd like to
> avoid.
>
> See discussion at https://groups.google.com/forum/?fromgroups=#!topic
> /sage-devel/ozhIHIwQ4WA
>
> __APPLY__
>
> * [attachment:14058-weak-binop-morphisms.patch]
> * [attachment:trac_14058-doctest.patch]

New description:

 R(1) + S(1) caches a strong reference to both R and S, which we'd like to
 avoid.

 See discussion at https://groups.google.com/forum/?fromgroups=#!topic
 /sage-devel/ozhIHIwQ4WA

--

Comment:

 Your change is small and makes sense.

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