On Thu, 3 Feb 2022 17:12:05 GMT, Weijun Wang <wei...@openjdk.org> wrote:

> Add the `-providerPath` option to jarsigner to be consistent with keytool.

Looks good to me except a minor comment.  It's fine to me if you don't want to 
take it.

src/jdk.jartool/share/classes/sun/security/tools/jarsigner/Main.java line 256:

> 254:                     String path = null;
> 255:                     path = PathList.appendPath(
> 256:                             path, System.getProperty("java.class.path"));

Is it more straightforward to use "String path = System.getProperty(...)", by 
combining line 254 and 255?

-------------

Marked as reviewed by xuelei (Reviewer).

PR: https://git.openjdk.java.net/jdk/pull/7338

Reply via email to