[JDK-6341887] RFR: Patch V3: java.util.zip: Add ByteBuffer methods to Inflater/Deflater

[David Lloyd]

On Fri, Mar 30, 2018 at 12:05 PM, Xueming Shen <xueming.shen at oracle.com> wrote:
>
> Hi David,
>
> (1) Deflater.setDictionary(ByteBuffer) missed a "@since 11"

Oops, easy fix.

> (2) infalte(...)  may not need to catch DFE twice, better (?) to just update
> the inputPos/input
>      at outsider catch as
>
>      if (input == null)
>          this.inputPos = inputPos + inputConsumed;
>      else
>          input.position(inputPos + inputConsumed;
>
>     the if/else might be repetitive, but better than two layers of same DFE
> catch?

I'm okay with anything at this point, but this does introduce a
"inputPos possibly not initialized" problem which doesn't seem to have
a clean solution.  WDYT?

-- 
- DML