RFR: 8163476: java/lang/StackWalker/VerifyStackTrace.java fails after JDK-8163369
Ivan Gerasimov
ivan.gerasimov at oracle.com
Tue Aug 9 12:41:31 UTC 2016
Hi Claes!
Not a thorough review, but a minor comment.
209 .replaceAll("DMH.invoke", "DMH/xxxxxxxx.invoke")
The dot in the regexp needs to be escaped here.
With kind regards,
Ivan
On 09.08.2016 15:26, Claes Redestad wrote:
> <resending due to missing RFR in subject>
>
> On 2016-08-09 13:48, Claes Redestad wrote:
>> Hi,
>>
>> change to enable DMH pre-generation caused a subtle change in the
>> stacktrace, which caused this test to fail.
>>
>> Please review this patch which reverts the stackframe to look like
>> before for non-pregenerated DMHs, while also updating the test to
>> accept pregenerated DMHs.
>>
>> To verify this works I added invokeVirtual_LL_V (used in the test) to
>> the list of pre-generated DMHs in GenerateJLIClassesPlugin and
>> verified that the test pass both on a jlinked image and an exploded
>> image.
>>
>> Webrev: http://cr.openjdk.java.net/~redestad/8163476/webrev.01
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8163476
>>
>> Thanks!
>>
>> /Claes
>
>
More information about the core-libs-dev
mailing list