RFR: 8920687: Deflater.setLevel does not work as expected
Alan Bateman
Alan.Bateman at oracle.com
Sun Sep 15 19:45:35 UTC 2013
On 15/09/2013 19:30, Xueming Shen wrote:
> Thanks Alan. I dropped the "current" in setLevel. Yes, I would like to
> be explicit about the
> deflate invocation here, as the implementation clearly just calls the
> deflate() blindly with
> the old level/strategy after the level/strategy changed, regardless
> whether there is really
> anything in the buffer or not. So I think it might be good to set the
> expectation clear here.
>
> http://cr.openjdk.java.net/~sherman/8020687/webrev/
>
> -Sherman
>
This is okay, the only thing is the words "of deflate methods" is a bit
odd. Do you want to change this to simple "deflate"?
-Alan.
More information about the core-libs-dev
mailing list