RFR: 8357086: os::xxx functions returning memory size should return size_t [v24]
Anton Artemov
duke at openjdk.org
Thu Aug 21 08:48:28 UTC 2025
On Wed, 20 Aug 2025 15:41:37 GMT, Joel Sikström <jsikstro at openjdk.org> wrote:
>> src/hotspot/share/runtime/os.hpp line 343:
>>
>>> 341:
>>> 342: static size_t physical_memory();
>>> 343: static bool has_allocatable_memory_limit(size_t* limit);
>>
>> Future cleanup: for consistency should this now also take a reference?
>
> This is no longer relevant as it has been re-worked in [JDK-8364248](https://bugs.openjdk.org/browse/JDK-8364248) to not have an out-parameter and renamed to `static size_t commit_memory_limit()`.
Removed in the latest commit.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/25450#discussion_r2290346842
More information about the hotspot-dev
mailing list