RFR: 8320935: Move CDS config initialization code to cdsConfig.cpp [v2]

Calvin Cheung ccheung at openjdk.org
Sat Dec 2 03:38:43 UTC 2023


On Sat, 2 Dec 2023 00:38:58 GMT, Ioi Lam <iklam at openjdk.org> wrote:

>> This is a simple clean up that moves the code for initializing the CDS config states from arguments.cpp to cdsConfig.cpp
>> 
>> I renamed a few functions, but otherwise the code is unchanged.
>> 
>> - `get_default_shared_archive_path()` -> `default_archive_path()`
>> - `GetSharedArchivePath()` -> `static_archive_path()`
>> - `GetSharedDynamicArchivePath()` -> `dynamic_archive_path()`
>> 
>> There's also less `#if INCLUDE_CDS` since the entire cdsConfig.cpp file is compiled only if CDS is enabled.
>
> Ioi Lam has updated the pull request incrementally with one additional commit since the last revision:
> 
>   fixed indentation

Marked as reviewed by ccheung (Reviewer).

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

PR Review: https://git.openjdk.org/jdk/pull/16868#pullrequestreview-1760771168


More information about the serviceability-dev mailing list