RFR (XS) CR 8058643: (str) Re-examine hashCode implementation

Aleksey Shipilev aleksey.shipilev at oracle.com
Wed Sep 24 17:01:12 UTC 2014


Thanks Martin!

-Aleksey.

On 09/24/2014 08:56 PM, Martin Buchholz wrote:
> I pushed Aleksey's change to jdk9-dev.
> 
> If it was up to me, I would give Aleksey all the commit/review bits I had...
> 
> On Wed, Sep 24, 2014 at 4:09 AM, Aleksey Shipilev
> <aleksey.shipilev at oracle.com <mailto:aleksey.shipilev at oracle.com>> wrote:
> 
>     Hi again,
> 
>     Friendly reminder: please sponsor this trivial change. :)
> 
>     -Aleksey.
> 
>     On 09/22/2014 06:29 PM, Aleksey Shipilev wrote:
>     > Fixed:
>     >  http://cr.openjdk.java.net/~shade/8058643/webrev.02/
>     <http://cr.openjdk.java.net/%7Eshade/8058643/webrev.02/>
>     >  http://cr.openjdk.java.net/~shade/8058643/8058643.changeset
>     <http://cr.openjdk.java.net/%7Eshade/8058643/8058643.changeset>
>     >
>     > Since there is no performance impact for this touchup (char -> int
>     > promotion is happening anyway), and no functionality changes (char ->
>     > int promotion zero-extends), I haven't developed any new tests, or
>     > re-spinned any existing ones.
>     >
>     > Thanks,
>     > -Aleksey.
>     >
> 
> 
> 





More information about the core-libs-dev mailing list