Re: RFR: 8321159: SymbolLookup.libraryLookup(Path, Arena) Assumes default Filesystem [v5]

2023-12-05 Thread Maurizio Cimadamore
On Tue, 5 Dec 2023 12:02:59 GMT, Per Minborg wrote: >> This PR proposes to reject paths provided to the >> `SymbolLookup.libraryLookup(Path path, Arena arena)` method if a path is not >> in the default file system. > > Per Minborg has updated the pull request incrementally with one additional

Re: RFR: 8321159: SymbolLookup.libraryLookup(Path, Arena) Assumes default Filesystem [v5]

2023-12-05 Thread Per Minborg
> This PR proposes to reject paths provided to the > `SymbolLookup.libraryLookup(Path path, Arena arena)` method if a path is not > in the default file system. Per Minborg has updated the pull request incrementally with one additional commit since the last revision: Fix double negation

Re: RFR: 8321159: SymbolLookup.libraryLookup(Path, Arena) Assumes default Filesystem [v4]

2023-12-05 Thread Maurizio Cimadamore
On Tue, 5 Dec 2023 07:16:49 GMT, Per Minborg wrote: >> This PR proposes to reject paths provided to the >> `SymbolLookup.libraryLookup(Path path, Arena arena)` method if a path is not >> in the default file system. > > Per Minborg has updated the pull request incrementally with one additional

Re: RFR: 8321159: SymbolLookup.libraryLookup(Path, Arena) Assumes default Filesystem [v4]

2023-12-05 Thread Maurizio Cimadamore
On Tue, 5 Dec 2023 07:16:49 GMT, Per Minborg wrote: >> This PR proposes to reject paths provided to the >> `SymbolLookup.libraryLookup(Path path, Arena arena)` method if a path is not >> in the default file system. > > Per Minborg has updated the pull request incrementally with one additional

Re: RFR: 8321159: SymbolLookup.libraryLookup(Path, Arena) Assumes default Filesystem [v4]

2023-12-04 Thread Per Minborg
> This PR proposes to reject paths provided to the > `SymbolLookup.libraryLookup(Path path, Arena arena)` method if a path is not > in the default file system. Per Minborg has updated the pull request incrementally with one additional commit since the last revision: Change @throws

Re: RFR: 8321159: SymbolLookup.libraryLookup(Path, Arena) Assumes default Filesystem [v3]

2023-12-04 Thread Alan Bateman
On Mon, 4 Dec 2023 15:00:56 GMT, Jorn Vernee wrote: >> Per Minborg has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Add fail statement > > src/java.base/share/classes/java/lang/foreign/SymbolLookup.java line 288: > >> 286: *

Re: RFR: 8321159: SymbolLookup.libraryLookup(Path, Arena) Assumes default Filesystem [v3]

2023-12-04 Thread Jorn Vernee
On Mon, 4 Dec 2023 09:10:07 GMT, Per Minborg wrote: >> This PR proposes to reject paths provided to the >> `SymbolLookup.libraryLookup(Path path, Arena arena)` method if a path is not >> in the default file system. > > Per Minborg has updated the pull request incrementally with one additional

Re: RFR: 8321159: SymbolLookup.libraryLookup(Path, Arena) Assumes default Filesystem [v3]

2023-12-04 Thread Alan Bateman
On Mon, 4 Dec 2023 09:10:07 GMT, Per Minborg wrote: >> This PR proposes to reject paths provided to the >> `SymbolLookup.libraryLookup(Path path, Arena arena)` method if a path is not >> in the default file system. > > Per Minborg has updated the pull request incrementally with one additional

Re: RFR: 8321159: SymbolLookup.libraryLookup(Path, Arena) Assumes default Filesystem [v3]

2023-12-04 Thread Per Minborg
> This PR proposes to reject paths provided to the > `SymbolLookup.libraryLookup(Path path, Arena arena)` method if a path is not > in the default file system. Per Minborg has updated the pull request incrementally with one additional commit since the last revision: Add fail statement

Re: RFR: 8321159: SymbolLookup.libraryLookup(Path, Arena) Assumes default Filesystem [v2]

2023-12-04 Thread Per Minborg
> This PR proposes to reject paths provided to the > `SymbolLookup.libraryLookup(Path path, Arena arena)` method if a path is not > in the default file system. Per Minborg has updated the pull request incrementally with one additional commit since the last revision: Improve after comments

Re: RFR: 8321159: SymbolLookup.libraryLookup(Path, Arena) Assumes default Filesystem

2023-12-03 Thread Alan Bateman
On Mon, 4 Dec 2023 07:29:37 GMT, Per Minborg wrote: > This PR proposes to reject paths provided to the > `SymbolLookup.libraryLookup(Path path, Arena arena)` method if a path is not > in the default file system. src/java.base/share/classes/java/lang/foreign/SymbolLookup.java line 297: > 295:

RFR: 8321159: SymbolLookup.libraryLookup(Path, Arena) Assumes default Filesystem

2023-12-03 Thread Per Minborg
This PR proposes to reject paths provided to the `SymbolLookup.libraryLookup(Path path, Arena arena)` method if a path is not in the default file system. - Commit messages: - Fix logic - Reject paths not in the default file system Changes: