#11521: Memleak when resolving the action of Integers on an Elliptic Curve
------------------------------------------------------------------+---------
   Reporter:  jpflori                                             |          
Owner:  robertwb  
       Type:  defect                                              |         
Status:  needs_work
   Priority:  major                                               |      
Milestone:  sage-4.8  
  Component:  coercion                                            |       
Keywords:  sd35      
Work_issues:  Understand why a weak key dictionary is not enough  |       
Upstream:  N/A       
   Reviewer:                                                      |         
Author:  Simon King
     Merged:                                                      |   
Dependencies:  #11900    
------------------------------------------------------------------+---------
Changes (by SimonKing):

  * work_issues:  => Understand why a weak key dictionary is not enough


Comment:

 I have attached a new patch version. It fixes the segfault I mentioned.
 However, it also does ''not'' fix the memory leak.

 The difference between the two versions is: The new patch still uses weak
 references to the key of the cache, but a strong reference to the value
 (i.e., the homset).

 The homset has a reference to domain and codomain, which constitute the
 cache key. Thus, I expected that it does not make any difference whether
 one has a strong or a weak reference to the homset. But I stand corrected.
 That needs to be investigated more deeply.

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