Re: [cp-patches] FYI: RepaintManager fix

[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.

Cheers,

Mark

signature.asc
Description: This is a digitally signed message part