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