Re: RFR: 8143142: AssertionError in MethodHandleImpl

2015-11-17 Thread Vladimir Ivanov
Reviewed. Best regards, Vladimir Ivanov On 11/17/15 5:21 PM, Claes Redestad wrote: Hi, it seems I've been testing assertions with the wrong flag. 8142334 added the same assertion check to MethodHandleImpl that was there for DirectMethodHandle, which due to some bootstrapping dependency fails

RFR: 8143142: AssertionError in MethodHandleImpl

2015-11-17 Thread Claes Redestad
Hi, it seems I've been testing assertions with the wrong flag. 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://c

Re: RFR: 8143142: AssertionError in MethodHandleImpl

2015-11-17 Thread 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 wrote: Hi, it seems I've been testing assertions with the wrong flag. To share some knowledge: when running jtreg locally it is important

Re: RFR: 8143142: AssertionError in MethodHandleImpl

2015-11-17 Thread Paul Sandoz
> On 17 Nov 2015, at 15:21, Claes Redestad 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 knowled