Re: RFR: 6983726: Reimplement MethodHandleProxies.asInterfaceInstance [v24]

2023-07-11 Thread Chen Liang
On Tue, 11 Jul 2023 17:36:33 GMT, Mandy Chung wrote: >> Chen Liang has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Fix the lazy test, thanks Jorn Vernee! > > test/jdk/java/lang/invoke/MethodHandleProxies/ProxiesImplementationTest.java >

Re: RFR: 6983726: Reimplement MethodHandleProxies.asInterfaceInstance [v24]

2023-07-11 Thread Mandy Chung
On Tue, 11 Jul 2023 15:15:51 GMT, Chen Liang wrote: >> As John Rose has pointed out in this issue, the current j.l.r.Proxy based >> implementation of MethodHandleProxies.asInterface has a few issues: >> 1. Exposes too much information via Proxy supertype (and WrapperInstance >> interface) >> 2.

Re: RFR: 6983726: Reimplement MethodHandleProxies.asInterfaceInstance [v24]

2023-07-11 Thread Mandy Chung
On Tue, 11 Jul 2023 15:15:51 GMT, Chen Liang wrote: >> As John Rose has pointed out in this issue, the current j.l.r.Proxy based >> implementation of MethodHandleProxies.asInterface has a few issues: >> 1. Exposes too much information via Proxy supertype (and WrapperInstance >> interface) >> 2.

Re: RFR: 6983726: Reimplement MethodHandleProxies.asInterfaceInstance [v24]

2023-07-11 Thread Mandy Chung
On Tue, 11 Jul 2023 17:38:42 GMT, Mandy Chung wrote: >> Chen Liang has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Fix the lazy test, thanks Jorn Vernee! > > test/jdk/java/lang/invoke/MethodHandleProxies/ProxiesImplementationTest.java >

Re: RFR: 6983726: Reimplement MethodHandleProxies.asInterfaceInstance [v24]

2023-07-11 Thread Mandy Chung
On Tue, 11 Jul 2023 15:15:51 GMT, Chen Liang wrote: >> As John Rose has pointed out in this issue, the current j.l.r.Proxy based >> implementation of MethodHandleProxies.asInterface has a few issues: >> 1. Exposes too much information via Proxy supertype (and WrapperInstance >> interface) >> 2.

Re: RFR: 6983726: Reimplement MethodHandleProxies.asInterfaceInstance [v24]

2023-07-11 Thread Chen Liang
> As John Rose has pointed out in this issue, the current j.l.r.Proxy based > implementation of MethodHandleProxies.asInterface has a few issues: > 1. Exposes too much information via Proxy supertype (and WrapperInstance > interface) > 2. Does not allow future expansion to support SAM[^1] abstrac