Re: RFR: JDK-8286783: Expand use of @inheritDoc in InputStream and OutputStream subclasses [v4]

2022-05-16 Thread Joe Darcy
On Mon, 16 May 2022 17:53:17 GMT, Alan Bateman  wrote:

> thanks for the updates.

Updated the CSR to match this version.

-

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


Re: RFR: JDK-8286783: Expand use of @inheritDoc in InputStream and OutputStream subclasses [v4]

2022-05-16 Thread Alan Bateman
On Mon, 16 May 2022 17:29:16 GMT, Joe Darcy  wrote:

>> Make the javadoc in the InputStream and OutputStream subclasses in core libs 
>> DRY-er by use of inheritDoc. (Any analagous changes to AudioInputStream in 
>> client libs will be done another a separate bug.) When the time comes, will 
>> do any necessary merging for the changes in[JDK-8286200.
>> 
>> Please also review the CSR to cover the introduction of implspec and 
>> implNote tags: https://bugs.openjdk.java.net/browse/JDK-8286784
>
> Joe Darcy has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Add @Override's requested in review feedback.

thanks for the updates.

-

Marked as reviewed by alanb (Reviewer).

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


Re: RFR: JDK-8286783: Expand use of @inheritDoc in InputStream and OutputStream subclasses [v4]

2022-05-16 Thread Joe Darcy
> Make the javadoc in the InputStream and OutputStream subclasses in core libs 
> DRY-er by use of inheritDoc. (Any analagous changes to AudioInputStream in 
> client libs will be done another a separate bug.) When the time comes, will 
> do any necessary merging for the changes in[JDK-8286200.
> 
> Please also review the CSR to cover the introduction of implspec and implNote 
> tags: https://bugs.openjdk.java.net/browse/JDK-8286784

Joe Darcy has updated the pull request incrementally with one additional commit 
since the last revision:

  Add @Override's requested in review feedback.

-

Changes:
  - all: https://git.openjdk.java.net/jdk/pull/8717/files
  - new: https://git.openjdk.java.net/jdk/pull/8717/files/07e8a6a6..c31a408d

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=jdk=8717=03
 - incr: https://webrevs.openjdk.java.net/?repo=jdk=8717=02-03

  Stats: 11 lines in 5 files changed: 11 ins; 0 del; 0 mod
  Patch: https://git.openjdk.java.net/jdk/pull/8717.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/8717/head:pull/8717

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