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

David Holmes dholmes at openjdk.org
Mon Oct 31 06:45:56 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.)

Seems quite reasonable. The size_t type doesn't seem to cause any issues in those expressions.

Thanks.

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

Marked as reviewed by dholmes (Reviewer).

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


More information about the hotspot-runtime-dev mailing list