RFR: 8006593: Performance and compatibility improvements to hash based Map implementations (original) (raw)

Mike Duigou mike.duigou at oracle.com
Wed Mar 6 17:30:23 UTC 2013

On Mar 6 2013, at 09:19 , Alan Bateman wrote:

On 05/03/2013 22:46, Mike Duigou wrote:

I have updated the webrev to remove the useAltHashing boolean. http://cr.openjdk.java.net/~mduigou/JDK-8006593/5/webrev/ Mike Peter's suggestion to remove the useAltHashing field is a great idea. I've looked at the webrev and it looks good to me. The logical xor in initHashSeedAsNeeded is subtle. As this is for jdk7u only then I assume you'll get approval on jdk7u-dev once you are done here.

Yes. Per the original review request:

Once review is completed here this patch will be proposed to JDK7u-dev for integration into the next 7u performance/feature release. I am going to leave this review open for another 24 hours before requesting approval from JDK7u-dev.

Mike

More information about the core-libs-dev mailing list