Sorry all, it looks like GMail doesn't know how to keep replies with the thread when you change the subject line. The follow-up to this thread is http://mail.openjdk.java.net/pipermail/core-libs-dev/2018-March/051960.html with only a few small changes as discussed above.
On Fri, Mar 2, 2018 at 2:36 PM, David Lloyd <david.ll...@redhat.com> wrote: > On Fri, Mar 2, 2018 at 2:34 PM, David Lloyd <david.ll...@redhat.com> wrote: >> On Fri, Mar 2, 2018 at 12:49 PM, Xueming Shen <xueming.s...@oracle.com> >> wrote: >>> Hi David, >>> >>> (1) Deflater.deflate(Bytebuffer) >>> the api doc regarding "no_flush" appears to be the copy/paste of the >>> byte[] version >>> without being updated to the corresponding ByteBuffer? >> >> You're right, I missed that one. I've incorporated this fix locally: > > Oops, this should have been: > > --- 8< --- cut here --- 8< --- > > diff --git a/src/java.base/share/classes/java/util/zip/Deflater.java > b/src/java.base/share/classes/java/util/zip/Deflater.java > index 524125787a8..40f0d9736e2 100644 > --- a/src/java.base/share/classes/java/util/zip/Deflater.java > +++ b/src/java.base/share/classes/java/util/zip/Deflater.java > @@ -481,9 +481,9 @@ public class Deflater { > * in order to determine if more input data is required. > * > * <p>This method uses {@link #NO_FLUSH} as its compression flush mode. > - * An invocation of this method of the form {@code deflater.deflate(b)} > + * An invocation of this method of the form {@code > deflater.deflate(output)} > * yields the same result as the invocation of > - * {@code deflater.deflate(b, 0, b.length, Deflater.NO_FLUSH)}. > + * {@code deflater.deflate(output, Deflater.NO_FLUSH)}. > * > * @param output the buffer for the compressed data > * @return the actual number of bytes of compressed data written to the > > --- 8< --- cut here --- 8< --- > > -- > - DML -- - DML