RFR 8242921: test/hotspot/jtreg/runtime/CompactStrings/TestMethodNames.java uses nashorn script engine

[Harold Seigel]

Thanks Coleen!

Harold

On 4/27/2020 9:21 AM, coleen.phillimore at oracle.com wrote:
> +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
>