>>> Shouldn't you also include the FileOuputStream in try-with-resources?
>> 
>> It is. Do you need to refresh your browers?
> 
> If you read the code quickly, I missed it too, maybe it would be better to 
> put the opening brace on a separate line, i.e.:
> 
> try (ZipFile in = new ZipFile("multi-release.jar");
>     FileOutputStream os = new FileOutputStream("signed-multi-release.jar"))
> {
>    js.sign(in, os);
> }

Done.  See http://cr.openjdk.java.net/~sdrach/8147607/changeset 
<http://cr.openjdk.java.net/~sdrach/8147607/changeset>

Reply via email to