On Thu, 24 Apr 2025 15:14:27 GMT, Henry Jen <[email protected]> wrote:
>> Jiangli Zhou has updated the pull request incrementally with two additional
>> commits since the last revision:
>>
>> - Merge branch 'JDK-8355080' of ssh://github.com/jianglizhou/jdk into
>> JDK-8355080
>> - Address henryjen@ comment:
>> - Remove '#include <jni.h>'.
>
> src/java.base/share/classes/jdk/internal/foreign/SystemLookup.java line 135:
>
>> 133: @SuppressWarnings("restricted")
>> 134: private static SymbolLookup sysLookup() {
>> 135: NativeLibraries libs = NativeLibraries.newInstance(null);
>
> I would consult @mcimadamore or @JornVernee look at this for the native
> function call permission requirements.
@mcimadamore or @JornVernee Can you help take a look of this? Thanks!
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/24801#discussion_r2058930806
