<br><br><div class="gmail_quote">On Thu, Jun 25, 2009 at 13:04, Xueming Shen <span dir="ltr"><<a href="mailto:Xueming.Shen@sun.com">Xueming.Shen@sun.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"><br>
</blockquote>
yes, it now solved the puzzle:-)<br>
<br>
(1)closeEntry() is not strictly necessary, either putNextEntry or close() will close the previous entry<br>
(2)you might want to go a little further to "encapsulate" the updateEntry into crc32os, pass the entry<br>
into the constructor and setup the value with a crc32os.close()...bad idea?<br>
</blockquote><div><br>It's somewhat magical to have close to this, <br>and close is not currently called automatically anywhere.<br>explicit updateEntry is at least easy to understand.<br><br> Martin</div></div><br>