#12313: Fix yet another memory leak caused by caching of coercion data
--------------------------------------------------+-------------------------
       Reporter:  SimonKing                       |         Owner:              
                             
           Type:  defect                          |        Status:  
needs_review                             
       Priority:  major                           |     Milestone:  sage-5.3    
                             
      Component:  memleak                         |    Resolution:              
                             
       Keywords:  coercion weak dictionary        |   Work issues:              
                             
Report Upstream:  N/A                             |     Reviewers:  Simon King, 
Jean-Pierre Flori, John Perry
        Authors:  Simon King, Jean-Pierre Flori   |     Merged in:              
                             
   Dependencies:  #11521, #11599, #12969, #12215  |      Stopgaps:              
                             
--------------------------------------------------+-------------------------

Comment (by john_perry):

 Replying to [comment:210 SimonKing]:
 > Replying to [comment:208 nbruin]:
 > > rings/polynomial/pbori.pyx:255:rings = weakref.WeakValueDictionary()
 >
 > But is that dictionary used? I can't find a spot in pbori.pyx where it
 is modified. Not in pbori itself, and also I couldn't find another module
 where it is imported.

 I ran a hypersearch, and don't see anywhere else that it appears, either.
 Have you tried deleting that line?

 (Believe it or not, I've been following this...)

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/12313#comment:212>
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 post to this group, send email to [email protected].
To unsubscribe from this group, send email to 
[email protected].
For more options, visit this group at 
http://groups.google.com/group/sage-trac?hl=en.

Reply via email to