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