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

 I let the `Gap._start` method print a stack when there is a problem that
 leads to the "this is harmless" message.

 After finishing the tests, there is a `RuntimeError` "Unable to start
 gap", and the stack is
 {{{
 (<frame object at 0x117d940>,
 '/mnt/local/king/SAGE/prereleases/sage-5.3.beta2/local/lib/python/site-
 packages/sage/interfaces/gap.py', 964, '_start', ['            for bla in
 inspect.stack():\n'], 0)
 (<frame object at 0x117d6f0>,
 '/mnt/local/king/SAGE/prereleases/sage-5.3.beta2/local/lib/python/site-
 packages/sage/interfaces/gap.py', 536, '_eval_line', ['
 self._start()\n'], 0)
 (<frame object at 0x4a027e0>,
 '/mnt/local/king/SAGE/prereleases/sage-5.3.beta2/local/lib/python/site-
 packages/sage/interfaces/gap.py', 1092, 'set', ['        out =
 self._eval_line(cmd, allow_use_file=True)\n'], 0)
 (<frame object at 0x42d1d50>,
 '/mnt/local/king/SAGE/prereleases/sage-5.3.beta2/local/lib/python/site-
 packages/sage/interfaces/interface.py', 385, '_create', ['
 self.set(name, value)\n'], 0)
 (<frame object at 0x4a07970>,
 '/mnt/local/king/SAGE/prereleases/sage-5.3.beta2/local/lib/python/site-
 packages/sage/interfaces/expect.py', 1327, '__init__', ['
 self._name = parent._create(value, name=name)\n'], 0)
 (<frame object at 0x4d3b9e0>,
 '/mnt/local/king/SAGE/prereleases/sage-5.3.beta2/local/lib/python/site-
 packages/sage/interfaces/interface.py', 195, '__call__', ['
 return cls(self, x, name=name)\n'], 0)
 (<frame object at 0x49ec260>,
 '/mnt/local/king/SAGE/prereleases/sage-5.3.beta2/local/lib/python/site-
 packages/sage/groups/perm_gps/permgroup.py', 562, '__cmp__', ['
 gSelf = self._gap_()\n'], 0)
 (<frame object at 0x1408240>,
 
'/mnt/local/king/SAGE/prereleases/sage-5.3.beta2/local/lib/python2.7/weakref.py',
 51, 'remove', ['                del self.data[wr.key]\n'], 0)
 }}}

 So, apparently one has to look at the `__cmp__` method in permgroup.py and
 make sure that it works when gap can not start.

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