RFR (M) 8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options

[coleen.phillimore at oracle.com]

Okay, somehow the columns of numbers didn't look very useful on my 
screen to me, and I wanted to convert this to UL (and straighten out the 
logic), so that's why I made this change.   I asked around internally to 
see which people would care about the format change and didn't find 
anyone specific.  Now I know!

Let me rework this to use UL but keep the table.

I'll withdraw this change for now.

Thank you for the quick feedback.
Coleen

On 7/9/18 1:58 PM, Aleksey Shipilev wrote:
> On 07/09/2018 07:48 PM, coleen.phillimore at oracle.com wrote:
>> Summary: Convert PrintSafepointStatistics to UL
>>
>> open webrev at http://cr.openjdk.java.net/~coleenp/8198720.01/webrev
>> bug link https://bugs.openjdk.java.net/browse/JDK-8198720
> The synopsis is misleading: it is not only obsoleting PrintSafepoint* options, it also reformats the
> output!
>
> We did JDK-8180482 not that long ago, and the reason was that both people and machine tools are
> accustomed to the particular non-noisy format for that table. I am not at all convinced that
> proposed format [2] is better than current version [3]. Can we keep (at least some resemblance of)
> the old format, please?
>
> -Aleksey
>
> [1] https://bugs.openjdk.java.net/browse/JDK-8180482
> [2] https://bugs.openjdk.java.net/secure/attachment/75330/out.safepoint-logging
> [3] http://cr.openjdk.java.net/~shade/8180482/after.txt
>