Changes between Initial Version and Version 1 of Ticket #14711, comment 61


Ignore:
Timestamp:
10/02/13 03:31:52 (6 years ago)
Author:
nbruin
Comment:

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #14711, comment 61

    initial v1  
    11Too bad. The idea as suggested doesn't actually solve the memory leak; it just makes it less severe (by a constant factor). The problem is: The weakened maps don't prevent their domain from being GCed, but after than happens they linger (now defunct) in `_coerce_from`. You'll see that even with your patch in, the example in the ticket description will still eat memory--just a little less quickly. You'll find that `CDF._coerce_from_hash` will contain a LOT of entries.
    22
    3 With the `_coerce_to` alternative also in place, this problem would not occur.
     3If we were to use a mix of `_coerce_from` and `_coerce_to` (somehow choosing which one to use) you wouldn't see this problem.
    44
    55If we really want/need to, we could probably salvage the "weakened map" solution: