RFR: JDK-8058845 : Update JCE environment for build improvements
mandy.chung at oracle.com
Tue Sep 23 16:31:20 UTC 2014
On 9/21/14 3:51 PM, Bradford Wetmore wrote:
> Hi Sean/Mandy/Erik/Magnus/Alan/David/others,
> Please review:
> JDK-8058845 : Update JCE environment for build improvements
The change looks fine in general. Some minor comments:
line 264: this is not part of your change but caught my attention.
I wonder if you want to include some descriptive message in NPE
to help diagnosing the problem especially in jdk9 where future
changes will be made to support modules.
line 79: this could be (PrivilegedExceptionAction<Void>) as the
return value is ignored
It may be better to rename URLVerifier to ProviderVerifier as it verifies
the security provider of the given codebase. URLVerifier might give
an interpretation of verifying the given URL. Similarly, the verifyProviderJar
method can be renamed to verifyProvider.
line 117: should it be pae.getCause()?
More information about the build-dev