RFR: 8143142: AssertionError in MethodHandleImpl

[Claes Redestad]

Vladimir, Paul,

thanks for looking at this so quickly!

Pushed.

/Claes

On 2015-11-17 15:30, Paul Sandoz wrote:
>> On 17 Nov 2015, at 15:21, Claes Redestad <claes.redestad at oracle.com> wrote:
>>
>> Hi,
>>
>> it seems I've been testing assertions with the wrong flag.
>>
> To share some knowledge: when running jtreg locally it is important to use the same parameters that are used when running the automated tests. To the best of my knowledge those parameters are:
>
>    -ea -esa -avm
>
>
>> 8142334 added the same assertion check to MethodHandleImpl that was there for DirectMethodHandle, which
>> due to some bootstrapping dependency fails for the MethodHandleImpl case. This patch removes that assertion:
>>
>> webrev: http://cr.openjdk.java.net/~redestad/8143142/webrev.01/
>> bug: https://bugs.openjdk.java.net/browse/JDK-8143142
>>
> Looks ok to me (but i am getting a little lost in the maze of improvements and various iterations :-) ).
>
> Paul.