RFR: 8320935: Move CDS config initialization code to cdsConfig.cpp [v3]
Ioi Lam
iklam at openjdk.org
Tue Dec 5 23:13:11 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 with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains seven additional commits since the last revision:
- Merge branch 'master' into 8320935-move-cds-config-code-from-arguments-cpp
- fixed indentation
- code alignment
- step4
- step3
- step2
- step1
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/16868/files
- new: https://git.openjdk.org/jdk/pull/16868/files/01dd47bc..a080edeb
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=16868&range=02
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=16868&range=01-02
Stats: 84382 lines in 1756 files changed: 39063 ins; 38780 del; 6539 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 serviceability-dev
mailing list