RFR: JDK-8295993 : use K instead of 1024 in a couple of hotspot outputs

Lutz Schmidt lucy at openjdk.org
Fri Oct 28 13:47:27 UTC 2022


On Fri, 28 Oct 2022 12:32:40 GMT, Matthias Baesken <mbaesken at openjdk.org> wrote:

> There are a couple of outputs of values in k  (e.g. in os_posix.cpp and os_windows.cpp) where the K constant could be used nicely instead of 1024 (this is done already at a number of places but not everywhere).
> 
> (This is NOT about replacing all 1024 in the codebase by K because I think this would not be a good idea.)

LGTM. And it looks trivial.
I really prefer the use of a global symbol over writing the literal constant everywhere.

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

Marked as reviewed by lucy (Reviewer).

PR: https://git.openjdk.org/jdk/pull/10902


More information about the hotspot-runtime-dev mailing list