On Mon, 4 Mar 2024 13:52:13 GMT, Jan Lahoda <jlah...@openjdk.org> wrote:

> Currently, JDK modules load by the bootstrap and platform ClassLoaders are 
> automatically granted the native access. I am working on an upgrade of JLine 
> inside the `jdk.internal.le` module, and I would like to replace the current 
> native bindings with FFM-based bindings (which are now somewhat provided by 
> JLine). But, for that, native access is needed for the `jdk.internal.le` 
> module. We could possibly move the module to the platform ClassLoader, but it 
> seems it might be better to have more control over which modules have the 
> native access.
> 
> This patch introduces an explicit list of modules that will automatically be 
> granted the native access. Note this patch is not yet intended to change the 
> end behavior - the list of modules granted native access is supposed to be 
> the same as modules in the boot and platform ClassLoaders.

src/java.base/share/classes/java/lang/ModuleLayer.java line 885:

> 883: 
> 884:     /**
> 885:      * Returns the module with the given name in this later only.

Suggestion:

     * Returns the module with the given name in this layer only.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/18106#discussion_r1511542904

Reply via email to