On 2018-04-12 21:25, Roger Riggs wrote:
Looks good,  Thanks Sherman

On 4/12/2018 2:43 PM, Xueming Shen wrote:

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.


Reply via email to