RFR(S): 8166156: Convert test_semaphore to GTest
Stefan Karlsson
stefan.karlsson at oracle.com
Mon Nov 28 13:01:10 UTC 2016
Hi Kirill,
On 2016-11-25 20:55, Kirill Zhaldybin wrote:
> Dear all,
>
> Could you please review this fix for 8166156?
> Pretty straight-forward conversion.
>
> WebRev: http://cr.openjdk.java.net/~kzhaldyb/webrevs/JDK-8166156/webrev.00/
> CR: https://bugs.openjdk.java.net/browse/JDK-8166156
Thanks for porting this test.
I think it would be preferable if this test was using TEST instead of
TEST_VM. I don't see anything in the test that requires the VM to run.
Thanks,
StefanK
>
> Thank you.
>
> Regards, Kirill
More information about the hotspot-runtime-dev
mailing list