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