RFR: JDK-8148467: Consistent use of : in the logging

[Bengt Rutisson]

On 2016-01-28 15:38, Tom Benson wrote:
> Same here.

Thanks, Tom!

Bengt

> Tom
>
> On 1/28/2016 8:29 AM, Mikael Gerdin wrote:
>> Hi Bengt,
>>
>> On 2016-01-28 14:05, Bengt Rutisson wrote:
>>>
>>> Hi everyone,
>>>
>>> Could I have a couple of reviews for this simple change to make the
>>> logging a bit more consistent?
>>>
>>> http://cr.openjdk.java.net/~brutisso/8148467/webrev.00/
>>
>> Looks good.
>> /Mikael
>>
>>> https://bugs.openjdk.java.net/browse/JDK-8148467
>>>
>>> As described by Jenny in JDK-8065344 some of the log messages are
>>> missing a ":".
>>>
>>> Here's are the comments regarding ":" from JDK-8065344:
>>>
>>> Parallel Redirty>Parallel Redirty:
>>>   -Other phases has ':' as separator, except for this one.
>>>
>>> [34.589s][debug ][gc,ergo,cset] GC(6) Added expensive regions to CSet
>>> (old CSet region num not reached min).old 2940 regions, expensive: 2940
>>> regions, min 2940 regions, remaining time: 0.00ms
>>> ->[34.589s][debug ][gc,ergo,cset] GC(6) Added expensive regions to CSet
>>> (old CSet region num not reached min).old: 2940 regions, expensive: 
>>> 2940
>>> regions, min: 2940 regions, remaining time: 0.00ms
>>> By adding ":" after old, expensive, min to make the format the same.
>>>
>>> Finish choosing CSet. old 0 regions, predicted old region time: 0.00ms,
>>> time remaining: 0.00
>>> -> Finish choosing CSet. old: 0 regions, predicted old region time:
>>> 0.00ms, time remaining: 0.00
>>> The ':' after old makes the format the same
>>>
>>>
>>> Thanks,
>>> Bengt
>>
>