Comment #2 on issue 74 by jfuerth: Memory leak with PSwingRepaintManager http://code.google.com/p/piccolo2d/issues/detail?id=74
I don't normally use weak references in my own programming, and I'm not terribly attached to the fix we've submitted. Here's what I can say in its defense: 1. It works :) 2. The swingWrappers list in the PSwingRepaintManager is basically a cache of live, reachable swing wrappers that have been encountered before. Because those semantics (live, reachable) match weak references exactly, no other solution will work better. A correctly implemented alternative solution could be just as effective, but it will involve adding more lines of code to PSwing. So, you'll get no killer arguments from me. As long as the fix is 100% effective, and released soon, I'll be happy! -- You received this message because you are listed in the owner or CC fields of this issue, or because you starred this issue. You may adjust your issue notification preferences at: http://code.google.com/hosting/settings --~--~---------~--~----~------------~-------~--~----~ Piccolo2D Developers Group: http://groups.google.com/group/piccolo2d-dev?hl=en -~----------~----~----~----~------~----~------~--~---
