Request for review: 8010518 Move deprecating CMSIncrementalMode from Arguments::check_deprecated_gcs() to Arguments::check_deprecated_gc_flags()

[Tao Mao]

Thank you for pointing it out, Bengt. A new webrev is updated.
http://cr.openjdk.java.net/~tamao/8010518/webrev.02/

Please see inline.
Tao

On 3/24/13 1:16 PM, Bengt Rutisson wrote:
>
> Hi Tao,
>
> On 3/23/13 4:51 AM, Tao Mao wrote:
>> Thank you for review and suggestion. A new webrev is updated.
>> http://cr.openjdk.java.net/~tamao/8010518/webrev.01/
>
> I like Jon's suggestion about removing the word "likely" but that 
> means that you need to update these tests:
>
> test/gc/startup_warnings/TestCMSIncrementalMode.java
> test/gc/startup_warnings/TestIncGC.java
Test files modified.
>
> Also, would it make sense to remove the word "likely" from the warning 
> messages in Arguments::check_deprecated_gcs() too? In that case you 
> need to update these tests as well:
>
> test/gc/startup_warnings/TestDefNewCMS.java
> test/gc/startup_warnings/TestParNewSerialOld.java
Have we made a decision to certainly remove these gc comb's in future? 
If so, it's OK to state so. Anyway, it would be better to resolve it 
with a separate CR.
>
> Bengt
>
>>
>> Tao
>>
>> On 3/22/13 1:56 PM, Jon Masamitsu wrote:
>>> Tao,
>>>
>>> Changes look fine.  I would remove the "likely" so that messages 
>>> read like
>>>
>>> "and will be removed in a future release"
>>>
>>> Fewer words are better and the intent is still clear.
>>>
>>> Jon
>>>
>>>
>>> On 3/21/2013 4:23 PM, Tao Mao wrote:
>>>> A simple changeset. Need a reviewer!
>>>>
>>>> 8010518 Move deprecating CMSIncrementalMode from 
>>>> Arguments::check_deprecated_gcs() to 
>>>> Arguments::check_deprecated_gc_flags()
>>>> https://jbs.oracle.com/bugs/browse/JDK-8010518
>>>>
>>>> webrev:
>>>> http://cr.openjdk.java.net/~tamao/8010518/webrev.00/
>>>>
>>>> changeset:
>>>> Cleanup suggested by Bengt.
>>>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.openjdk.org/pipermail/hotspot-gc-dev/attachments/20130325/e50b87c3/attachment.htm>