[9] RFR (XS): 8060483: NPE with explicitCastArguments unboxing null

2014-10-15 Thread Vladimir Ivanov
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

Re: [9] RFR (XS): 8060483: NPE with explicitCastArguments unboxing null

2014-10-15 Thread Marcus Lagergren
+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

Re: [9] RFR (XS): 8060483: NPE with explicitCastArguments unboxing null

2014-10-15 Thread Vladimir Ivanov
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.

Re: [9] RFR (XS): 8060483: NPE with explicitCastArguments unboxing null

2014-10-15 Thread Vladimir Ivanov
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

Re: [9] RFR (XS): 8060483: NPE with explicitCastArguments unboxing null

2014-10-15 Thread Attila Szegedi
+1 On Wed, Oct 15, 2014 at 2:12 PM, Vladimir Ivanov < vladimir.x.iva...@oracle.com> 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.

Re: [9] RFR (XS): 8060483: NPE with explicitCastArguments unboxing null

2014-11-02 Thread Attila Szegedi
Thanks for fixing this. Can we expect a backport to 8u-dev soon? On Oct 15, 2014, at 4:46 PM, Attila Szegedi wrote: > +1 > > On Wed, Oct 15, 2014 at 2:12 PM, Vladimir Ivanov < > vladimir.x.iva...@oracle.com> wrote: > >> http://cr.openjdk.java.net/~vlivanov/8060483/webrev.00/ >> https://bugs.op