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
