RFR: 6983726: remove Proxy from MethodHandleProxies.asInterfaceInstance SAM conversion [v3]
Chen Liang
liach at openjdk.org
Wed Mar 29 01:06:36 UTC 2023
On Tue, 28 Mar 2023 10:57:28 GMT, RĂ©mi Forax <forax at openjdk.org> wrote:
>> Chen Liang has updated the pull request incrementally with one additional commit since the last revision:
>>
>> rethrow error
>
> src/java.base/share/classes/java/lang/invoke/MethodHandleProxies.java line 206:
>
>> 204: try {
>> 205: proxy = (Object) info.ctor.invokeExact(mhs); // non-varargs
>> 206: } catch (Throwable e) {
>
> At least Error should be directly propagated (especially OutOfMemoryError)
Done. I just noticed LambdaMetafactory has the same problem at no-arg constructor call site, might be worthy to create a bug as well.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/13197#discussion_r1151293383
More information about the core-libs-dev
mailing list