Hi Roman, On Mon, 2005-11-14 at 12:52 +0000, Roman Kennke wrote: > + /** > + * The current repaint managers, indexed by their ThreadGroups. > + */ > + static HashMap currentRepaintManagers;
Isn't this a potential memory leak? Maybe there are not many ThreadGroups in an application ever. But it might be wise to use a WeakHashMap here so that the entry disappears when the ThreadGroup is garbage collected. Cheers, Mark
signature.asc
Description: This is a digitally signed message part
_______________________________________________ Classpath-patches mailing list [email protected] http://lists.gnu.org/mailman/listinfo/classpath-patches
