|
From: | Bryce McKinlay |
Subject: | Re: Potential bug in Hashtable |
Date: | Tue, 06 Nov 2001 19:59:59 +1300 |
User-agent: | Mozilla/5.0 (X11; U; Linux ppc; en-US; rv:0.9.5) Gecko/20011016 |
John Leuner wrote:
I ran the latest classpath code against mauve, a hashtable test which checks the size() after a clone() failed: I think copy.size should be set to 0 in the clone method (before the putAll)?
I agree. HashMap probibly needs this fix also. Also I'm not sure Hashtable.clone() should call putAll, since this will be visible to subclasses. Maybe we should add the equivilent of the HashMap.putAllInternal.
regards Bryce.
[Prev in Thread] | Current Thread | [Next in Thread] |