RFR: 8024009 : Remove jdk.map.useRandomSeed system property
Please review my fix to remove the jdk.map.useRandomSeed system property added earlier in jdk8. Some history is in the bug report, http://bugs.sun.com/view_bug.do?bug_id=8024009 HashMap and LinkedHashMap stopped using the random hash seed as of 8023463. This change removes the code to read the jdk.map.useRandomSeed property and setup the hashSeed in Hashtable and WeakHashMap. Hashtable got a couple extra things cleaned up: * hash() calls were converted back to key.hashCode(), returning the code to its state prior to 7126277 (alternative String hashing). * With the hashSeed gone, the 8000955 change [1] should no longer be necessary, and we can go back to using the key's hash value stored in Entry.hash instead of re-calculating it. I also removed tests (or test @runs) which are now obsolete. Automated build and test runs look good. Webrev: http://cr.openjdk.java.net/~bchristi/8024009/webrev.00/ Thanks, -Brent [1] http://hg.openjdk.java.net/jdk8/jdk8/jdk/rev/5eed4a92ca8c
Re: RFR: 8024009 : Remove jdk.map.useRandomSeed system property
Hi Brent; Thanks for cleaning this up. The changes look fine. (A simplifying change in Collections, how unexpected!) Mike On Sep 11 2013, at 17:13 , Brent Christian wrote: > Please review my fix to remove the jdk.map.useRandomSeed system property > added earlier in jdk8. > > Some history is in the bug report, > http://bugs.sun.com/view_bug.do?bug_id=8024009 > > HashMap and LinkedHashMap stopped using the random hash seed as of > 8023463. This change removes the code to read the jdk.map.useRandomSeed > property and setup the hashSeed in Hashtable and WeakHashMap. > > Hashtable got a couple extra things cleaned up: > * hash() calls were converted back to key.hashCode(), returning the code to > its state prior to 7126277 (alternative String hashing). > > * With the hashSeed gone, the 8000955 change [1] should no longer be > necessary, and we can go back to using the key's hash value stored in > Entry.hash instead of re-calculating it. > > I also removed tests (or test @runs) which are now obsolete. > > Automated build and test runs look good. > > > Webrev: > http://cr.openjdk.java.net/~bchristi/8024009/webrev.00/ > > Thanks, > -Brent > > [1] http://hg.openjdk.java.net/jdk8/jdk8/jdk/rev/5eed4a92ca8c
Re: RFR: 8024009 : Remove jdk.map.useRandomSeed system property
On 12/09/2013 01:13, Brent Christian wrote: Please review my fix to remove the jdk.map.useRandomSeed system property added earlier in jdk8. It's good to see this going away, the changes (and clean-up) look good to me too. -Alan
Re: RFR: 8024009 : Remove jdk.map.useRandomSeed system property
On 9/12/13 1:09 AM, Alan Bateman wrote: On 12/09/2013 01:13, Brent Christian wrote: Please review my fix to remove the jdk.map.useRandomSeed system property added earlier in jdk8. It's good to see this going away, the changes (and clean-up) look good to me too. Thanks guys. Can someone push this for me? I can send you a changeset... -Brent