RFR: 8329112: Clean up CDS checking of unsupported module options [v2]
Ioi Lam
iklam at openjdk.org
Wed Mar 27 16:54:47 UTC 2024
> - Renamed the functions to `check_unsupported_dumping_module_options()` and `has_unsupported_runtime_module_options()`, since their error messages are about VM options, not system properties.
> - Consolidated the search of unsupported properties into a helper function.
> - Removed obsolete code/comment in classLoader.cpp that suggests that `--patch-module` is allowed during dumping. Replace with an assertion that forbids patching when dumping.
Ioi Lam has updated the pull request incrementally with one additional commit since the last revision:
@calvinccheung review comments
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/18492/files
- new: https://git.openjdk.org/jdk/pull/18492/files/84f470d8..641763ac
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=18492&range=01
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=18492&range=00-01
Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod
Patch: https://git.openjdk.org/jdk/pull/18492.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/18492/head:pull/18492
PR: https://git.openjdk.org/jdk/pull/18492
More information about the hotspot-runtime-dev
mailing list