RFR: 8361213: J2DAnalyzer should emit the score as a decimal [v2]

Eric Caspole ecaspole at openjdk.org
Wed Jul 2 18:56:34 UTC 2025


On Wed, 2 Jul 2025 16:10:10 GMT, Jonas Norlinder <duke at openjdk.org> wrote:

>> I am not sure what you mean, but with the 10, the scores line up with the rest of the minor stats when it is monospace font like this example -
>> 
>> 
>> Summary:
>>   vimg_text_noaa: 
>>     Number of tests:  2
>>     Overall average:  20002.7449
>>     Best spread:      1.3% variance
>>     Worst spread:     3.62% variance
>>     (Basis for results comparison)
>
> Thanks. I was thinking if the difference between the old and new behavior would result in any rendering differences. The example you showed is a special case where the amount of characters is => 10.
> 
> If `overallscore` is say `1.678` then it renders like this:
> 
> `System.out.printf("    Overall average:  %10.4f%n", overallscore);`
> 
> Summary:
>   vimg_text_noaa_1: 
>     Number of tests:  2
>     Overall average:      1,6780
>     Best spread:      2.14% variance
>     Worst spread:     2.57% variance
>     (Basis for results comparison)

The scores are chars/sec or pixels/sec so it is always a big number with (usually) many decimal places. 

    Overall average:  19046.2340
    Overall average:  20010.8816
    Overall average:  19829.0079
    Overall average:  16595601.1584
    Overall average:  61195068.8361

I added a - to the format string to always left align the result which should work for big or small numbers.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/26076#discussion_r2180771315


More information about the client-libs-dev mailing list