RFR: 8320935: Move CDS config initialization code to cdsConfig.cpp [v2]
Ioi Lam
iklam at openjdk.org
Sat Dec 2 00:38:58 UTC 2023
> 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
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/16868/files
- new: https://git.openjdk.org/jdk/pull/16868/files/72f3e44c..01dd47bc
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=16868&range=01
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=16868&range=00-01
Stats: 2 lines in 1 file changed: 0 ins; 0 del; 2 mod
Patch: https://git.openjdk.org/jdk/pull/16868.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/16868/head:pull/16868
PR: https://git.openjdk.org/jdk/pull/16868
More information about the hotspot-dev
mailing list