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

Volker Simonis simonis at openjdk.java.net
Tue Apr 12 14:45:34 UTC 2022


On Mon, 11 Apr 2022 16:07:15 GMT, Alan Bateman <alanb at openjdk.org> wrote:

>> Volker Simonis has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Adapted wording and copyrights based on @jaikiran review
>
> src/java.base/share/classes/java/util/zip/GZIPInputStream.java line 103:
> 
>> 101:      * elements {@code buf[off+}<i>n</i>{@code ]} through {@code buf[off+}<i>len</i>{@code -1]}
>> 102:      * are undefined and an implementation is free to change them during the inflate
>> 103:      * operation. If the return value is -1 or an exception is thrown the whole
> 
> The sentence is very long. How about putting "an implementation is free ..." in parentheses after "undefined".

Done (also in `ZipInputStream` and `GZipInputStream` which both have the same sentence).

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

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


More information about the core-libs-dev mailing list