RFR: 8147607: Remove test library dependency on sun.security.tools.jarsigner.Main

Steve Drach steve.drach at oracle.com
Thu Jan 28 17:51:52 UTC 2016


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




More information about the core-libs-dev mailing list