On Wed, 13 Nov 2024 13:48:14 GMT, Per Minborg <pminb...@openjdk.org> wrote:

> This PR proposes to remove the security manager dependencies in the FFM 
> implementing classes.

I didn't get a chance to review until now, had one small comment.

src/java.base/share/classes/jdk/internal/foreign/abi/fallback/LibFallback.java 
line 39:

> 37:     static final boolean SUPPORTED = tryLoadLibrary();
> 38: 
> 39:     @SuppressWarnings({"restricted"})

Nit: you no longer need the braces around "restricted".

-------------

PR Review: https://git.openjdk.org/jdk/pull/22071#pullrequestreview-2433981549
PR Review Comment: https://git.openjdk.org/jdk/pull/22071#discussion_r1840891458

Reply via email to