#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):

 If I understood correctly, singular functions are supposed to leave the
 .ref-field alone, unless a ring in the interpreter is concerned. However,
 I found that using the `singular_function` "ringlist" does decrease the
 reference count for the current ring. That's annoying.

 Could it in fact be an artefact of how `singular_function` is implemented?
 If I recall correctly `ringlist(R)` is a bit tricky: It should return a
 list defined in the *current* ring, but with data defining R. So, perhaps
 we are somehow messing up `currRing` versus the ring that is given as
 argument.

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