> 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

Reply via email to