RFR(S): 8236111 : narrow allowSmartActionArgs disabling

David Holmes david.holmes at oracle.com
Mon Dec 23 21:33:16 UTC 2019


Hi Igor,

Hotspot changes seem fine. Can't comment on jdk tests.

Thanks,
David

On 24/12/2019 6:42 am, Igor Ignatyev wrote:
> ping?
> 
>> On Dec 17, 2019, at 11:30 AM, Igor Ignatyev <igor.ignatyev at oracle.com> wrote:
>>
>> http://cr.openjdk.java.net/~iignatyev/8236111/webrev.00/
>>> 31 lines changed: 20 ins; 11 del; 0 mod;
>>
>> Hi all,
>>
>> could you please review this small patch which enables allowSmartActionArgs in hotspot and jdk test suites and disables them in a small number of test directories? the patch also removes TEST.properties files which enabled allowSmartActionArgs as they aren't needed anymore.
>>
>> from JBS:
>>> currently, allowSmartActionArgs is disabled for the whole hotspot and jdk test suites and enabled just in few places. this makes it a bit harder for people to use smart action arguments in these test suites as they have to not to forget to enable them. and given in all the other test suites, smart action arguments are enabled, it can be confusing and frustrating.
>>
>>
>> testing: tier1-5
>> JBS: https://bugs.openjdk.java.net/browse/JDK-8236111
>> webrev: http://cr.openjdk.java.net/~iignatyev/8236111/webrev.00/
>>
>> Thanks,
>> -- Igor
> 


More information about the core-libs-dev mailing list