RFR: 8356888: (fs) FileSystems.newFileSystem that take an env must specify IllegalArgumentException
Jaikiran Pai
jpai at openjdk.org
Thu May 22 12:18:52 UTC 2025
On Thu, 22 May 2025 00:13:57 GMT, Brian Burkhalter <bpb at openjdk.org> wrote:
> Add a throws clause for `IllegalArgumentException` to the `java.nio.file.FileSystems` methods `newFileSystem(Path, Map)` and `newFileSystem(Path, Map, ClassLoader)`. Also update the equivalent IAE verbiage for the analogous methods which accept a `URI` parameter instead of a `Path`.
The change looks OK to me. While at it, should we add a regression test to verify that IllegalArgumentException gets thrown for some the filesystem implementations that we ship in the JDK?
-------------
Marked as reviewed by jpai (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/25376#pullrequestreview-2860997111
More information about the nio-dev
mailing list