RFR: 8282648: Problems due to conflicting specification of Inflater::inflate(..) and InflaterInputStream::read(..) [v4]

[Volker Simonis]

On Fri, 8 Apr 2022 13:11:23 GMT, Jaikiran Pai <jpai at openjdk.org> wrote:

>> Volker Simonis has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Added API-refinement to GZIPInputStream::read()/ZipInputStream::read() and an Implementation note to ZipFile::getInputStream()
>
> src/java.base/share/classes/java/util/zip/ZipFile.java line 346:
> 
>> 344:      * have been returned by invocations of this method.
>> 345:      *
>> 346:      * @implNote In the JDK implementation this method returns an
> 
> The "In the JDK implementation" sounds a bit odd to me. I looked around some other `@implNote`s and many of of them use `@implNote This implementation ....`, so perhaps we should change this to:
> ``` 
> @implNote This implementation returns an instance of {@link java.util.zip.InflaterInputStream}.

Thanks for the suggestion. I've updated to code accordingly.

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

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