RFR: 8247449: Revisit the argument processing logic for MetaspaceShared::disable_optimized_module_handling() [v2]

[Ioi Lam]

On Wed, 27 Mar 2024 02:52:33 GMT, Calvin Cheung <ccheung at openjdk.org> wrote:

>> The changes include:
>> 
>> - refactor` CDSConfig::check_system_property()` into `CDSConfig:check_internal_module_property()` and `CDSConfig::check_incompatible_property()`;
>> - `CDSConfig:check_internal_module_property()` will be called from `Arguments::create_module_property()` and `Arguments::create_numbered_module_property()`;
>> - `CDSConfig::check_incompatible_property()` will be called from `Arguments::add_property()` only for `ExternalProperty`.
>> 
>> Passed tiers 1 - 3 testing.
>
> Calvin Cheung has updated the pull request incrementally with one additional commit since the last revision:
> 
>   @iklam comments

LGTM

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

Marked as reviewed by iklam (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/18495#pullrequestreview-1962216678