RFR: 8253005: Add `@throws IOException` in javadoc for `HttpEchange.sendResponseHeaders` [v5]

Patrick Concannon pconcannon at openjdk.java.net
Wed Nov 4 20:44:10 UTC 2020


> Hi,
> 
> Could someone please review my fix for JDK-8253005: 'Add `@throws IOException` in javadoc for `HttpEchange.sendResponseHeaders`' ?
> 
> The method `HttpEchange.sendResponseHeaders` throws an `IOException` but is unspecified in its javadoc. This fix adds an `@throws IOException` to its specification and a description of the conditions under which the exception is thrown.
> 
> Kind regards,
> Patrick

Patrick Concannon 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 eight additional commits since the last revision:

 - 8253005: Added test to verify `IOException` thrown when headers already sent
 - Merge remote-tracking branch 'origin/master' into JDK-8253005
 - Merge remote-tracking branch 'origin/master' into JDK-8253005
 - Merge remote-tracking branch 'origin/master' into JDK-8253005
 - Merge remote-tracking branch 'origin/master' into JDK-8253005
 - Merge remote-tracking branch 'origin/master' into JDK-8253005
 - Merge remote-tracking branch 'origin/master' into JDK-8253005
 - 8253005: Add `@throws IOException` in javadoc for `HttpEchange.sendResponseHeaders`

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

Changes:
  - all: https://git.openjdk.java.net/jdk/pull/1014/files
  - new: https://git.openjdk.java.net/jdk/pull/1014/files/20cca899..0c5c7744

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=1014&range=04
 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=1014&range=03-04

  Stats: 1914 lines in 271 files changed: 839 ins; 713 del; 362 mod
  Patch: https://git.openjdk.java.net/jdk/pull/1014.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/1014/head:pull/1014

PR: https://git.openjdk.java.net/jdk/pull/1014


More information about the net-dev mailing list