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

[Kumar Srinivasan]

Thanks for all the reviews and suggestions!

the new version is at:
http://cr.openjdk.java.net/~ksrini/6985763/webrev.01

In this revision:
1. the input parameter is renamed to "in",
     btw. we call out throwing of  NPEs at the package level documentation
     http://download.oracle.com/javase/6/docs/api/java/util/jar/Pack200.html
     I copied the same verbiage to the interfaces sections as well.

2. moved the exception check into scanJar,  per Alan's suggestion where 
it is
     isolated to catch IllegalStateException and wraps it up into an IOE.


Thanks
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.