On Thu, 27 Apr 2023 06:42:08 GMT, Jaikiran Pai <j...@openjdk.org> wrote:
>> Amit Kumar has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> comments from Lance
>
> test/jdk/java/util/zip/DeInflate.java line 124:
>
>> 122: * This method checks if a given Deflater and Inflater pair can
>> correctly compress and decompress data.
>> 123: * checks were performed for this functionality using various input
>> scenarios and ByteBuffer instances.
>> 124: */
>
> Would you consider updating this comment to something like:
>
>
> /**
> * Uses the {@code def} deflater to deflate the input data {@code in} of
> length {@code len}.
> * A new {@link Inflater} is then created within this method to inflate the
> deflated data. The
> * inflated data is then compared with the {@code in} to assert that it
> matches the original
> * input data.
> * This method repeats these checks for the different overloaded methods of
> * {@code Deflater.deflate(...)} and {@code Inflater.inflate(...)}
> *
> * @param def the deflater to use for deflating the contents in {@code in}
> * @param in the input content
> * @param len the length of the input content to use
> * @param nowrap will be passed to the constructor of the {@code Inflater}
> used in this
> * method
> * @throws Throwable if any error occurs during the check
> */
fixed :-)
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/12283#discussion_r1178705442
