[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [cp-patches] FYI: RepaintManager fix
From: |
Roman Kennke |
Subject: |
Re: [cp-patches] FYI: RepaintManager fix |
Date: |
Sat, 19 Nov 2005 21:39:23 +0100 |
Am Samstag, den 19.11.2005, 19:09 +0100 schrieb Mark Wielaard:
> 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.
Yeah that is also correct. I'll fix this.
/Roman
signature.asc
Description: Dies ist ein digital signierter Nachrichtenteil