RFR: 8920687: Deflater.setLevel does not work as expected

Xueming Shen xueming.shen at oracle.com
Sun Sep 15 20:22:02 UTC 2013


Updated accordingly.

http://cr.openjdk.java.net/~sherman/8020687/webrev/

Thanks!
-Sherman

On 9/15/13 12:45 PM, Alan Bateman wrote:
> 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