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

Kumar Srinivasan kumar.x.srinivasan at oracle.com
Wed Nov 3 13:28:47 UTC 2010


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.




More information about the core-libs-dev mailing list