On Tue, 13 Oct 2020 18:46:59 GMT, Alan Bateman <al...@openjdk.org> wrote:

>> Volker Simonis has updated the pull request with a new target base due to a 
>> merge or a rebase. The pull request now
>> contains one commit:
>>   8253952: Refine ZipOutputStream.putNextEntry() to recalculate ZipEntry's 
>> compressed size
>
> src/java.base/share/classes/java/util/jar/JarOutputStream.java line 85:
> 
>> 83:      * ZipEntry#setCompressedSize(long)} method, then the compressed
>> 84:      * size will be set to the actual compressed size after deflation.
>> 85:      * <p>
> 
> I'm happy with the wording. What you think about put a p tag before "The 
> default compression method ..." so that it
> goes into its own paragraph? I'm asking because it's inconsistent to have the 
> first paragraph include the details on
> the compression method and the details on the current time pushed into a 
> second paragraph - if you generate the javadoc
> then you'll see what I mean.

I think that is a good idea to at the paragraph tag as you suggest

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

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

Reply via email to