Re: [9] RFR (S): 8036117: MethodHandles.catchException doesn't handle VarargsCollector right (8034120 failed)

2014-03-10 Thread Vladimir Ivanov
Chris, thanks for the review. John suggested an elegant way to fix the problem - use asFixedArity. Updated fix: http://cr.openjdk.java.net/~vlivanov/8036117/webrev.01/ Best regards, Vladimir Ivanov On 3/8/14 4:51 AM, Christian Thalinger wrote: Seems good to me. I’d like to have another name

Re: [9] RFR (S): 8036117: MethodHandles.catchException doesn't handle VarargsCollector right (8034120 failed)

2014-03-10 Thread John Rose
Reviewed. – John > On Mar 10, 2014, at 3:21 PM, Vladimir Ivanov > wrote: > > Chris, thanks for the review. > > John suggested an elegant way to fix the problem - use asFixedArity. > > Updated fix: > http://cr.openjdk.java.net/~vlivanov/8036117/webrev.01/ > > Best regards, > Vladimir Ivanov

Re: [9] RFR (S): 8036117: MethodHandles.catchException doesn't handle VarargsCollector right (8034120 failed)

2014-03-10 Thread Christian Thalinger
Even better. On Mar 10, 2014, at 3:21 PM, Vladimir Ivanov wrote: > Chris, thanks for the review. > > John suggested an elegant way to fix the problem - use asFixedArity. > > Updated fix: > http://cr.openjdk.java.net/~vlivanov/8036117/webrev.01/ > > Best regards, > Vladimir Ivanov > > On 3/8

Re: [9] RFR (S): 8036117: MethodHandles.catchException doesn't handle VarargsCollector right (8034120 failed)

2014-03-10 Thread Vladimir Ivanov
John, Chris, thanks! Best regards, Vladimir Ivanov On 3/11/14 3:08 AM, Christian Thalinger wrote: Even better. On Mar 10, 2014, at 3:21 PM, Vladimir Ivanov wrote: Chris, thanks for the review. John suggested an elegant way to fix the problem - use asFixedArity. Updated fix: http://cr.ope