jmx-dev RFR: 8373441: Remove DCmdFactory::_enabled [v2]

Ioi Lam iklam at openjdk.org
Fri Dec 12 23:37:20 UTC 2025


> The `DCmdFactory::_enabled` is always set to `true` and there doesn't seem to be a reason to set it to `false`.
> 
> This PR removes this field and simplified the creation of `DCmdFactory` objects.
> 
> The related `_hidden` field is also currently not used, but may be used in the future when deprecating DCmds, so we leave it unchanged.
> 
> Note that now `jmm_GetDiagnosticCommandInfo()` always set `dcmdInfo::enabled` to `true` to be compatible with Java code.

Ioi Lam has updated the pull request incrementally with one additional commit since the last revision:

  @kevinjwalls comments

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/28794/files
  - new: https://git.openjdk.org/jdk/pull/28794/files/8a047713..98176aa7

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=28794&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=28794&range=00-01

  Stats: 2 lines in 1 file changed: 0 ins; 0 del; 2 mod
  Patch: https://git.openjdk.org/jdk/pull/28794.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/28794/head:pull/28794

PR: https://git.openjdk.org/jdk/pull/28794


More information about the jmx-dev mailing list