On 02/07/2019 18:20, Mandy Chung wrote:
:
I've taken several passes over the changes and I don't see anything
obviously wrong. One minor nit is that the @throws
IllegalAccessException in accessClass(Class) needs to be updated to
allow for it to thrown when not accessible from the previous lookup
class. Also VerifyAccess::isModuleAccessible shouldn't need to if the
package is exported unconditionally as it is covered by
isExported(pn, m1) && isExported(pn, m2).
Good point. Fixed.
Webrev updated:
http://cr.openjdk.java.net/~mchung/jdk14/8173978/webrev.01/
The updated @throws IAE in accessClass and the updated
VerifyAccess::isModuleAccessible looks good.
-Alan.
