http://cr.openjdk.java.net/~vlivanov/8060483/webrev.00/
https://bugs.openjdk.java.net/browse/JDK-8060483
Recent changes broke MHs.explicitCastArguments for null values when
unboxing taking place.
The fix corrects the equivalence check for MHs.eCA & MH.asType() in
MT.explicitCastEquivalentToAs
On Oct 14, 2014, at 8:54 PM, Vladimir Ivanov
wrote:
> Paul,
>
> Thanks for the feedback!
>
> Updated version:
> http://cr.openjdk.java.net/~vlivanov/8059877/webrev.01/
>
>>> http://cr.openjdk.java.net/~vlivanov/8059877/webrev.00/
>>> https://bugs.openjdk.java.net/browse/JDK-8059877
>>>
>>
>
+1 for me. I assume that Wrapper.java is only reformatting. It looks like it.
Regards
Marcus
On 15 Oct 2014, at 14:12, Vladimir Ivanov wrote:
> http://cr.openjdk.java.net/~vlivanov/8060483/webrev.00/
> https://bugs.openjdk.java.net/browse/JDK-8060483
>
> Recent changes broke MHs.explicitCastAr
+1
On Wed, Oct 15, 2014 at 2:12 PM, Vladimir Ivanov <
[email protected]> wrote:
> http://cr.openjdk.java.net/~vlivanov/8060483/webrev.00/
> https://bugs.openjdk.java.net/browse/JDK-8060483
>
> Recent changes broke MHs.explicitCastArguments for null values when
> unboxing taking place.
Marcus, Attila, thanks for the prompt review!
Yes, I only reformatted Wrapper a bit to make it more readable.
I should've mentioned that in original request.
Best regards,
Vladimir Ivanov
On 10/15/14, 6:34 PM, Marcus Lagergren wrote:
+1 for me. I assume that Wrapper.java is only reformatting.
FTR filed request for additional test development [1]
Best regards,
Vladimir Ivanov
[1] https://bugs.openjdk.java.net/browse/JDK-8060717
On 10/15/14, 6:49 PM, Vladimir Ivanov wrote:
Marcus, Attila, thanks for the prompt review!
Yes, I only reformatted Wrapper a bit to make it more readable.
I
Thanks, Paul!
Updated version:
http://cr.openjdk.java.net/~vlivanov/8059877/webrev.01/
...
This is a good example to add to our "stomping on a final field" discussion, i
think here it is definitely a very special case with a careful dance between updating and
inlining (updateForm is also
Hi Remi,
I did some brief evaluation of this area.
MethodHandleProxies.asInterfaceInstance currently does not support proxying to
default methods. An InternalError will be thrown if a default method is
invoked. It should be possible to fix using a trusted internal IMPL_LOOKUP to
look up MHs th
