> DiagnosticCommandImpl should only publish parameter types in a known standard > set, and use "STRING" on anything else. > e.g. We can say "FILE" in the help output for jcmd, as that's for humans, but > the MBean parameter info should contain "STRING".
Kevin Walls 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 five additional commits since the last revision: - Merge remote-tracking branch 'upstream/master' into 8338603_dcmd_mbean_types - Merge remote-tracking branch 'upstream/master' into 8338603_dcmd_mbean_types - Update comment about type name knowledge - Test update - 8338603: DiagnosticCommandMBean operations should standardize types for parameters. ------------- Changes: - all: https://git.openjdk.org/jdk/pull/21040/files - new: https://git.openjdk.org/jdk/pull/21040/files/fabcaebe..70312190 Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk&pr=21040&range=02 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=21040&range=01-02 Stats: 14306 lines in 284 files changed: 12064 ins; 1035 del; 1207 mod Patch: https://git.openjdk.org/jdk/pull/21040.diff Fetch: git fetch https://git.openjdk.org/jdk.git pull/21040/head:pull/21040 PR: https://git.openjdk.org/jdk/pull/21040