#13447: Make libsingular multivariate polynomial rings collectable
-------------------------------------+-------------------------------------
       Reporter:  nbruin             |        Owner:
           Type:  defect             |       Status:  needs_work
       Priority:  major              |    Milestone:  sage-6.4
      Component:  memleak            |   Resolution:
       Keywords:                     |    Merged in:
        Authors:  Nils Bruin, Simon  |    Reviewers:  Simon King
  King                               |  Work issues:  Understand why
Report Upstream:  None of the above  |  sometimes `new_RingWrap` needs an
  - read trac for reasoning.         |  incref and sometimes not
         Branch:                     |       Commit:
   Dependencies:  #11521             |     Stopgaps:
-------------------------------------+-------------------------------------

Comment (by SimonKing):

 Replying to [comment:114 SimonKing]:
 > Yes, that shortcut still is reasonable and shouldn't be changed.
 However, there should be no need to take the time and increase the
 reference count.

 ... because we also have the polynomial ring that keeps a (counted)
 reference to the libsingular ring.

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