RFR 8242921: test/hotspot/jtreg/runtime/CompactStrings/TestMethodNames.java uses nashorn script engine
coleen.phillimore at oracle.com
coleen.phillimore at oracle.com
Mon Apr 27 13:21:43 UTC 2020
+1. yes, looks good with Lois's comment.
Coleen
On 4/27/20 9:12 AM, Harold Seigel wrote:
> Thanks Lois.
>
> I'll add the comment before I push the change.
>
> Harold
>
> On 4/27/2020 9:11 AM, Lois Foltan wrote:
>> On 4/27/2020 8:50 AM, Harold Seigel wrote:
>>> Hi,
>>>
>>> Please review this small change to test TestMethodNames.java to no
>>> longer use nashorn to generate nonstandard method names. Instead,
>>> the names are hard-coded in a new .jcod file.
>>>
>>> Open Webrev: http://cr.openjdk.java.net/~hseigel/bug_8242921/webrev/
>>>
>>> JBS Bug: https://bugs.openjdk.java.net/browse/JDK-8242921
>>>
>>> The test was run using Mach5 tiers 1-3 to check that it passed on
>>> multiple platforms and with multiple flags.
>>>
>>> Thanks, Harold
>>>
>>
>> Looks good Harold! Minor nit, please consider putting a comment at
>> the top of the .jcod files containing the method names since the
>> calls to the eval method, which also have the side effect of showing
>> the names, are being removed.
>>
>> Thanks,
>> Lois
More information about the hotspot-runtime-dev
mailing list