Re: Review Request: JDK-8173978: Lookup.in should allow teleporting from a lookup class in a named module without dropping all access

[Mandy Chung]

(Coming back to this patch and ready to push this change later today)

Here is the updated webrev:
   http://cr.openjdk.java.net/~mchung/jdk14/8173978/webrev.02/

This includes the change that Peter suggested accepting m2 == null.


Mandy

On 7/5/19 12:29 PM, Peter Levart wrote:

Hi Mandy,

Yes, either way will avoid double canRead/isExported calls against the same module. Although perhaps, an API with null prevModule would be more consistent with other API methods that use null prevLookupClass etc...

Regards, Peter