RFR (M): 8142510: rev2: -XX:+PrintFlagsRanges should print default range value for those flags that have constraint and an implicit range.
Gerard Ziemski
gerard.ziemski at oracle.com
Thu Mar 17 13:52:57 UTC 2016
Thank you very much Dmitry!
> On Mar 17, 2016, at 8:29 AM, Dmitry Dmitriev <dmitry.dmitriev at oracle.com> wrote:
>
> Hi Gerard,
>
> Looks good, except copyrights years in the following modules(forgot about that in the first round):
> commandLineFlagConstraintList.cpp
> commandLineFlagConstraintList.hpp
> commandLineFlagRangeList.hpp
>
> Not need a new webrev for that.
>
> Thanks,
> Dmitry
>
> On 16.03.2016 19:55, Gerard Ziemski wrote:
>> hi all,
>>
>> This is rev2 of the fix incorporating feedback from Dmitry:
>>
>> - re-use "CommandLineFlagConstraintList::find()"
>>
>> Please review this enhancement to Command Line Options Validation JEP-245, which prints default ranges for those flags, that only have constraints (ie. no range, but a constraint, implies default range)
>>
>> With this fix we’ll be able to include more flags in test/runtime/CommandLine/OptionsValidation test.
>>
>>
>> bug https://bugs.openjdk.java.net/browse/JDK-8142510
>> webrev http://cr.openjdk.java.net/~gziemski/8142510_rev2
>>
>> tested with JPRT hotspot, RBT hotspot/test/runtime and local test/runtime/CommandLine/OptionsValidation
>>
>>
>> cheers
>
More information about the hotspot-runtime-dev
mailing list