> This patch overhauls the library loading mechanism used by the Foreign Linker > API. We realized that, while handy, the *default* lookup abstraction > (`LibraryLookup::ofDefault`) was behaving inconsistentlt across platforms. > > This patch replaces `LibraryLookup` with a simpler `SymbolLookup` > abstraction, a functional interface. Crucially, `SymbolLookup` does not > concern with library loading, only symbol lookup. For this reason, two > factories are added: > > * `SymbolLookup::loaderLookup` - which obtains a lookup that can be used to > lookup symbols in libraries loaded by current loader > * `CLinker::systemLookup` - a more stable replacement for the *default* > lookup, which looks for symbols in libc.so (or its equivalent in other > platforms). The contents of this lookup are unspecified. > > Both factories are *restricted*, so they can only be called when > `--enable-native-access` is set.
Maurizio Cimadamore has updated the pull request incrementally with 10 additional commits since the last revision: - Update test/jdk/java/foreign/TestIntrinsics.java Co-authored-by: Paul Sandoz <paul.d.san...@googlemail.com> - Update test/jdk/java/foreign/valist/VaListTest.java Co-authored-by: Paul Sandoz <paul.d.san...@googlemail.com> - Update test/jdk/java/foreign/TestVarArgs.java Co-authored-by: Paul Sandoz <paul.d.san...@googlemail.com> - Update test/jdk/java/foreign/TestUpcall.java Co-authored-by: Paul Sandoz <paul.d.san...@googlemail.com> - Update test/jdk/java/foreign/TestIllegalLink.java Co-authored-by: Paul Sandoz <paul.d.san...@googlemail.com> - Update test/jdk/java/foreign/TestSymbolLookup.java Co-authored-by: Paul Sandoz <paul.d.san...@googlemail.com> - Update test/jdk/java/foreign/TestDowncall.java Co-authored-by: Paul Sandoz <paul.d.san...@googlemail.com> - Update test/jdk/java/foreign/StdLibTest.java Co-authored-by: Paul Sandoz <paul.d.san...@googlemail.com> - Update test/jdk/java/foreign/SafeFunctionAccessTest.java Co-authored-by: Paul Sandoz <paul.d.san...@googlemail.com> - Update src/jdk.incubator.foreign/share/classes/jdk/internal/foreign/abi/SharedUtils.java Co-authored-by: Paul Sandoz <paul.d.san...@googlemail.com> ------------- Changes: - all: https://git.openjdk.java.net/jdk/pull/4316/files - new: https://git.openjdk.java.net/jdk/pull/4316/files/2b668f7c..41ab74b6 Webrevs: - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=4316&range=02 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=4316&range=01-02 Stats: 10 lines in 10 files changed: 0 ins; 0 del; 10 mod Patch: https://git.openjdk.java.net/jdk/pull/4316.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/4316/head:pull/4316 PR: https://git.openjdk.java.net/jdk/pull/4316