RFR 8010325 : Remove hash32() method and hash32 int field from java.lang.String
martinrb at google.com
Wed Jun 12 17:45:03 UTC 2013
Thanks for doing this.
Your webrev does not include mercurial changeset information, which I think
is supported by recent webrevs.
I assume you've been working with Doug Lea on this. If not, the two of you
BTW, it's sad not to see things like murmur hash be public supported
algorithms - but I think guava provides it.
On Tue, Jun 11, 2013 at 1:32 PM, Brent Christian <brent.christian at oracle.com
> Please review my fix for 8010325. Some background:
> The alternative string-hashing implementation added to java.lang.String
> a new public method, hash32(), as well as a new private int field to cache
> the alternate hash value.
> Following 8005698  and 8005704 , the new method and field in String
> are no longer used.
> By updating a couple lines in Hashtable/HashMap/WeakHashMap, the internal
> sun.misc.Hashing class can also be removed.
> The webrev is here:
> 1. http://hg.openjdk.java.net/**jdk8/jdk8/jdk/rev/43bd5ee0205e<http://hg.openjdk.java.net/jdk8/jdk8/jdk/rev/43bd5ee0205e>
> 2. http://bugs.sun.com/view_bug.**do?bug_id=8005698<http://bugs.sun.com/view_bug.do?bug_id=8005698>
> 3. http://bugs.sun.com/view_bug.**do?bug_id=8005704<http://bugs.sun.com/view_bug.do?bug_id=8005704>
More information about the core-libs-dev