Please review java.util.jar.pack.* exceptions

Alan Bateman Alan.Bateman at oracle.com
Wed Nov 3 11:50:03 UTC 2010


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.



More information about the core-libs-dev mailing list