RFR(S): 8164737 - Remove Unsafe dependency from ProcessTools
George Triantafillou
george.triantafillou at oracle.com
Wed Aug 24 18:43:38 UTC 2016
Hi Christian,
Looks good.
-George
On 8/24/2016 2:31 PM, Christian Tornqvist wrote:
> Hi everyone,
>
>
>
> Please review this small change that removes the unnecessary dependency on
> Unsafe from ProcessTools. Jdk.test.lib.ProcessTools use some methods from
> jdk.test.lib.Utils which has the getUnsafe() method, all tests that use
> ProcessTools needed to have the @modules java.base/jdk.internal.misc even if
> they don't use Unsafe.
>
>
>
> This change moves getUnsafe() into a new class:
> jdk.test.lib.unsafe.UnsafeHelper and updates the few existing tests that use
> Unsafe to use this class instead.
>
>
>
> Webrev:
>
> http://cr.openjdk.java.net/~ctornqvi/webrev/8164737/webrev.00/
>
>
>
> Bug:
>
> https://bugs.openjdk.java.net/browse/JDK-8164737
>
>
>
> Thanks,
>
> Christian
>
>
>
More information about the hotspot-dev
mailing list