RFR: 8294948: Document IllegalArgumentException and NullPointerException thrown by URLStreamHandler::parseURL and URLStreamHandler::setURL [v10]

Daniel Fuchs dfuchs at openjdk.org
Wed Oct 19 15:24:11 UTC 2022


> During the review of JDK-8293590, it was noted that some methods in `URLStreamHandler` were missing an `@throws IllegalArgumentException` clause in their API documentation. 
> 
> This change adds the requested information, and also clarifies throwing `MalformedURLException` from URL constructors.
> 
> The CSR can be reviewed here: https://bugs.openjdk.org/browse/JDK-8294984

Daniel Fuchs has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains 12 additional commits since the last revision:

 - Merge branch 'master' into url-iae-8294948
 - More review feedback
 - More review comments
 - Taking into account review feedback
 - Merge branch 'master' into url-iae-8294948
 - Merge branch 'master' into url-iae-8294948
 - Minor wording updates - unmmapable single quote character
 - Minor wording updates
 - Update Copyright Years
 - Adding missing @throws NPE
 - ... and 2 more: https://git.openjdk.org/jdk/compare/bcd47739...f062dc68

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/10609/files
  - new: https://git.openjdk.org/jdk/pull/10609/files/5b0692b7..f062dc68

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=10609&range=09
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=10609&range=08-09

  Stats: 7191 lines in 688 files changed: 4189 ins; 1226 del; 1776 mod
  Patch: https://git.openjdk.org/jdk/pull/10609.diff
  Fetch: git fetch https://git.openjdk.org/jdk pull/10609/head:pull/10609

PR: https://git.openjdk.org/jdk/pull/10609


More information about the net-dev mailing list