> 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 four additional commits since 
the last revision:

 - 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/aa22301c..fabcaebe

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

  Stats: 198321 lines in 1421 files changed: 184773 ins; 6839 del; 6709 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

Reply via email to