[JDK-6341887] RFR: Patch V3: java.util.zip: Add ByteBuffer methods to Inflater/Deflater
Xueming Shen
xueming.shen at oracle.com
Thu Mar 29 04:38:39 UTC 2018
On 3/28/18, 6:14 AM, David Lloyd wrote:
>
>>> The current wording (which pre-dates your changes of course) reads more like
>>> API advice. I think it's a bit confusing too as it doesn't define what a
>>> "flush marker" is. I think we will need to re-word that sentence to make it
>>> clearer.
>> OK. I am at a loss for a better way to explain it though; any suggestions?
> I've implemented all the other changes except for this one. Latest
> version is attached, online version is here [1].
>
>
"flush marker" was copy/pasted from the original zlib.h api doc when I
added the
support for different flush options. It might be better to put it is a
apiNote. A
"flush marker" in this case is a 5-byte empty block, which will be
output if the
deflater does not have enough space to output the bytes and it is in
full_flush or
sync_flush mode. I think you can leave it as is for now and I will try
to see if I
can move it around or add some understandable wording, if I can.
-Sherman
More information about the core-libs-dev
mailing list