On 09/22/2014 06:06 PM, Aleksey Shipilev wrote:
> Hi again,
>
> On 09/17/2014 06:28 PM, Aleksey Shipilev wrote:
>> Can I have a review and a sponsorship for this tiny readability cleanup
>> in String.hashCode()?
>> http://cr.openjdk.java.net/~shade/8058643/webrev.01/
>> https://bugs.openjdk.java.net/browse/JDK-8058643
>
> I think we have enough reviews? Here is a changeset:
> http://cr.openjdk.java.net/~shade/8058643/8058643.changeset
>
> Please sponsor!
Wait. Claes spotted an inconsistency:
Should be:
for (char v : value) {
Not:
for (int v : value) {
Let me also write a unit test for this :)
-Aleksey.
