[cp-patches] [PATCH] Bump up HashMap default initial capacity

2011-02-22 Thread Pekka Enberg
While debugging Jython bootstrap issues with GNU Classpath, I noticed that HashMap.DEFAULT_CAPACITY is set to 11 although 1.4 Javadocs, for example, define it to be 16. Signed-off-by: Pekka Enberg penb...@kernel.org --- ChangeLog |6 ++ java/util/HashMap.java |5 ++---

[cp-patches] [PATCH] Fix HashMap.put() to check for hashCode equality before equals()

2011-02-22 Thread Pekka Enberg
This patch is needed to run Jython 2.5.2 RC 4 under JamVM and GNU Classpath CVS HEAD. It turns out Jythin bootstrap is bit hairy and assumes HashMap.put() checks for hashCode equality before invoking Object.equals(). Signed-off-by: Pekka Enberg penb...@kernel.org --- ChangeLog |

Re: [cp-patches] [PATCH] Bump up HashMap default initial capacity

2011-02-22 Thread Mark Wielaard
On Tue, 2011-02-22 at 16:58 +0200, Pekka Enberg wrote: While debugging Jython bootstrap issues with GNU Classpath, I noticed that HashMap.DEFAULT_CAPACITY is set to 11 although 1.4 Javadocs, for example, define it to be 16. Signed-off-by: Pekka Enberg penb...@kernel.org /** - *

Re: [cp-patches] [PATCH] Fix HashMap.put() to check for hashCode equality before equals()

2011-02-22 Thread Mark Wielaard
On Tue, 2011-02-22 at 17:02 +0200, Pekka Enberg wrote: This patch is needed to run Jython 2.5.2 RC 4 under JamVM and GNU Classpath CVS HEAD. It turns out Jythin bootstrap is bit hairy and assumes HashMap.put() checks for hashCode equality before invoking Object.equals(). [...] 2011-02-22

Re: [cp-patches] [PATCH] Bump up HashMap default initial capacity

2011-02-22 Thread Dr Andrew John Hughes
On 16:07 Tue 22 Feb , Mark Wielaard wrote: On Tue, 2011-02-22 at 16:58 +0200, Pekka Enberg wrote: While debugging Jython bootstrap issues with GNU Classpath, I noticed that HashMap.DEFAULT_CAPACITY is set to 11 although 1.4 Javadocs, for example, define it to be 16. Signed-off-by: