+1
/Claes
On 2018-04-12 21:25, Roger Riggs wrote:
Looks good, Thanks Sherman
On 4/12/2018 2:43 PM, Xueming Shen wrote:
Hi,
Please help review the change for JDK-8201443
issue: https://bugs.openjdk.java.net/browse/JDK-8201443
webrev: http://cr.openjdk.java.net/~sherman/8201443/webrev
(1) To use SharedSecret/JavaLangAccess/Class.getDeclaredPublicMethods
to avoid
NoSuchMethodExcetpion throwing.
(2) Given the fact that our JarFile does not override
ZipFile.close(), to add the fast
check for JarFile to avoid unnecessary class/method check.
(3) No new regression test provided. To use the existing unit tests
for regression test.
Manually verified the provided test/scenario is passed.
Thanks,
Sherman