RFR(S): 8164738: Convert AltHashing_test to GTest

David Holmes david.holmes at oracle.com
Fri Aug 26 03:38:00 UTC 2016


On 25/08/2016 1:47 AM, Kirill Zhaldybin wrote:
> Dear all,
>
> Could you please review this fix for 8164738?

Seems okay.

> To convert the test I added new friend class to AltHashing class so we
> could access private member function static juint murmur3_32(const int*
> data, int len). There are also few formating fixes.

Any reason all the murmur functions shouldn't be public? I'm not a fan 
of friends. No big deal either way.

Thanks,
David

>
> WebRev: http://cr.openjdk.java.net/~kzhaldyb/webrevs/JDK-8164738/webrev.00/
> CR: https://bugs.openjdk.java.net/browse/JDK-8164738
>
> Regards, Kirill


More information about the hotspot-runtime-dev mailing list