On 09/12/2009 04:45 PM, Martin Buchholz wrote:
On Fri, Sep 11, 2009 at 16:13, Xueming Shen<xueming.s...@sun.com> wrote:
David wrote:
This is not unusual in the JDK or elsewhere - that a stream calls the 
underlying stream's flush() method I mean - and I think in the (unlikely) event 
that someone wants to flush the underlying stream only, they could simply call 
.flush() directly on that stream.

David: In this case the deflater is not an "underlying stream", and the 
proposal was to
have flush() change the nature of the bytes written, while normally 
OutputStream.flush merely causes
the same bytes to be written more promptly.

In this paragraph I was referring to the real underlying OutputStream though, not the Deflater (afaict the existing implementation just calls through to the underlying stream currently).

The second paragraph is where I talk about the Deflater. And anyway I change my vote from "yes-kinda-no-but-qualified-yes" in favor of the plan B patch which I like a lot.

- DML

Reply via email to