RFR(XXXS): 7092245: G1: Wrong format specifier in G1PrintRegionLivenessInfo header output

[Ramki Ramakrishna]

On 9/21/2011 1:48 PM, Tony Printezis wrote:
> ...
> Well, depends what you mean by internal or general use consumption. Of 
> course, the parameter is available for everyone to use. But, 
> realistically, who else is going to use it?

Sorry i made a perhaps artificial and almost certainly unfortunate 
distinction between the cognoscenti and the hoi-polloi, here... the 
distinction
is hereby withdrawn :-)

>
> Anyway, IIRC the reason I had left out the units when I first 
> introduced the G1PrintRegionLivenessInfo was so that the columns do 
> not get too large. Would it work if we add a new header line with the 
> units, like this?
>
> ### PHASE Post-Sorting @ 5.303
> ### HEAP  committed: 0xf2400000-0xf6400000  reserved: 
> 0xf2400000-0xf6400000  region-size: 1048576
> ###
> ###   type         address-range       used  prev-live  
> next-live          gc-eff
> ###                                 (bytes)    (bytes)    (bytes)      
> (bytes/ms)
> ###   OLD  0xf4d00000-0xf4e00000    1048576        680          
> 0       1036338.1
> ###   OLD  0xf4800000-0xf4900000    1048576        824          
> 0       1033205.5
>

Yes, that would work great (and indeed is the standard in most 
scientific tabular presentations).

> Tony
>> But this of course does not have to be done now.

thanks!
-- ramki