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

Alan Bateman alanb at openjdk.org
Mon Dec 4 16:00:39 UTC 2023


On Mon, 4 Dec 2023 15:00:56 GMT, Jorn Vernee <jvernee at openjdk.org> 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:      *         is called from a thread {@code T}, other than the arena's owner thread
>> 287:      * @throws IllegalArgumentException if {@code path} is not in the default file system
>> 288:      *         or if {@code path} does not point to a valid library
> 
> Please split this into separate `@throws IllegalArgumentException` tags, as that is what we do everywhere else in the FFM API.

It depends on the wording. If changed to something like "not a path to a valid library in the default file system" then it avoid describing it as two cases.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/16944#discussion_r1414118698


More information about the core-libs-dev mailing list