I will look into incorporating all the comments, watch out for the next version.

Kumar

Kumar Srinivasan wrote:
Hi,

These are simple changes to the java.util.jar.pack:
  * fixes JCK  failures, specifically throwing unexpected exceptions
  * minor fixes to the ClassFormatException
  * added a new test to catch the JCK type failures up-front, which is
    bulk of the code changes.

http://cr.openjdk.java.net/~ksrini/6985763/webrev.00/

Thanks
Kumar

Kumar - would it be better if scanJar just handled the closed JAR case by catching the IllegalStateException and re-throwing it as an IOException? I assume other runtime exceptions are other bugs that you won't have translated into I/O exceptions.

-Alan.

Reply via email to