On Wed, Oct 21, 2009 at 1:02 PM,  <span dir="ltr"><<a href="mailto:xueming.shen@sun.com">xueming.shen@sun.com</a>></span> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
4206909: want java.util.zip to work for interactive use (Z_SYNC_FLUSH)<br>
Summary: Add sync_flush option into Deflater/DefalterOutputStream<br>
Reviewed-by: martin, alanb<br>
<br>
! src/share/classes/java/util/zip/Deflater.java<br>
! src/share/classes/java/util/zip/DeflaterOutputStream.java<br>
! src/share/native/java/util/zip/Deflater.c<br>
+ test/java/util/zip/InflateIn_DeflateOut.java<br></blockquote><div><br></div><div>Should we use an enum for the flush strategies? It would help separate them from the compression strategies, and it would make the new method easier to use because it would be obvious at compile time which values you can pass in (as opposed to allowing any random int).</div>
<div><br></div><div>Also, we could add a note about how alternate flush strategies can negatively affect compatibility with tools like WinZip.</div><div><br></div><div>Bob </div></div>