Changes look fine.

Are you going to backport the fix to earlier JDKs? Otherwise, you can simplify

   cons = clazz.getConstructor(new Class[] {String.class});
   Object obj = cons.newInstance(new Object[] {configFile});

to

   cons = clazz.getConstructor(String.class);
   cons.newInstance(configFile);

Thanks
Max

On 04/15/2011 07:32 AM, Valerie (Yu-Ching) Peng wrote:
Max,

Do you have cycles to review this following trivial test bug?
7035115: sun/security/pkcs11/Provider/ConfigShortPath.java compilation
failed

Webrev at:
http://cr.openjdk.java.net/~valeriep/7035115/webrev.00/

This only occurs on windows 64-bit machines where SunPKCS11 provider
isn't built.
This wasn't caught in my previous JPRT job since pkcs11 regression tests
aren't run w/ default JPRT settings.

Thanks,
Valerie

Reply via email to