RFR 8056174: New APIs for jar signing

Weijun Wang weijun.wang at oracle.com
Mon Sep 28 23:34:39 UTC 2015



On 9/29/2015 3:27 AM, Sean Mullan wrote:
> Looks good, just a couple of comments:
>
> AlgorithmId: can you use braces around the conditional statements on
> lines 1008-1017?

OK.

>
> Function: are you missing an @modules tag for the jarsigner module?

Which one? I thought @modules is only used if you want to call 
non-exported classes.

>
> Options.java: why not use the JarSigner API here instead of the
> jarsigner tool?

This test is to make sure jarsigner still behaves correctly after it is 
modified to be based on the JarSigner API. As for the API itself, I use 
Function.java to check its function and Spec.java to check it's spec.

I'll add some @summary.

Thanks
Max

>
> --Sean
>
> >    http://cr.openjdk.java.net/~weijun/8056174/webrev.05/
>>




More information about the security-dev mailing list