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