RFR(XXS) : 8208647 : switch jtreg to 4.2b13
David Holmes
david.holmes at oracle.com
Thu Aug 2 06:07:31 UTC 2018
On 2/08/2018 3:39 PM, Igor Ignatyev wrote:
>
>
>> On Aug 1, 2018, at 10:33 PM, David Holmes <david.holmes at oracle.com> wrote:
>>
>> Hi Igor,
>>
>> On 2/08/2018 5:42 AM, Igor Ignatyev wrote:
>>> http://cr.openjdk.java.net/~iignatyev/8208647/webrev.00/index.html
>>>> 5 lines changed: 0 ins; 0 del; 5 mod;
>>> Hi all,
>>> could you please review this small fix which switches jtreg to the latest available build?
>>
>> Okay.
>>
>>> due to changes in javatest API, failure_handler had to be slightly updated.
>>
>> Don't understand how anything in javatest affected the code you changed, but it's cleaner to specify its a Map<String, String> and avoid the casts.
>
> com.sun.javatest.InterviewParameters::save used to be 'save(Map)', and now it's 'save(Map<String, String>)', so w/o changing 'map' local, we would get compile-time error.
Ah! Of course.
Thanks,
David
> -- Igor
>> Thanks,
>> David
>>
>>> webrev: http://cr.openjdk.java.net/~iignatyev//8208647/webrev.00/index.html
>>> JBS: https://bugs.openjdk.java.net/browse/JDK-8208647
>>> testing: tier1-2
>>> Thanks,
>>> -- Igor
>
More information about the build-dev
mailing list