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

[Alan Bateman]

Kumar Srinivasan wrote:
> 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
This looks much better. It may be slightly better to limit the handling 
of the IllegalStateException to just the call to the JarFile's entries 
method but I can't see anything else that might throw it so what you 
have is fine with me.

On the test, I notice you compile with -XDignore.symbol.file but I don't 
think that is needed. Also, it might be better to eliminate the 
reference to a JCK test.

-Alan.