From: Jochen H. <Joc...@In...> - 2001-10-18 09:18:31
|
On Oct 9, Jörg Prante wrote: > Hi! > OK it's me again. Please apologize but my fix for ZipFile.java was not > correct! > > It fails with the trivial case, an empty zip file. Empty zip files are rare > and useless, of course, but an EOFException in that case is really not > required at all. Attached you find a correct ZipFIle.java that works now. I have committed a similar fix to the classpath repository. Note that this now contains the official code. > Another issue was that DeflaterEngine does not correctly deflate multiple zip > entries. It fails at a subsequent zip entry to deflate with an > ArrayIndexOutOfBoundException. I guess this has been reported as a bug. I > fixed this, too. You find a modified DeflaterEngine.java attached. The > reset() method has been extended by two more lines to reset some fields. That bug was already fixed in the classpath repository. There was another bug that could cause a ArrayIndexOutOfBoundException, which was fixed there, too. BTW, we should make a script that copies the classpath java.util.zip sources into a net.sf.jazzlib package which can then be used for inclusion in applet code. This package could be distributed from the SourceForge site. John what do you think? Jochen |