RFR: 8265612: revise the help info for jmap histo command [v2]

Chris Plummer cjplummer at openjdk.java.net
Wed Apr 21 18:46:45 UTC 2021


On Wed, 21 Apr 2021 05:47:35 GMT, Lin Zang <lzang at openjdk.org> wrote:

>> This PR  revise the help message of the `parallel=[N]` option of command `jmap -histo`. It mainly comes from the discussion at  https://github.com/openjdk/jdk/pull/2379#issuecomment-782609582  and https://github.com/openjdk/jdk/pull/2261#issuecomment-823623186.
>
> Lin Zang has updated the pull request incrementally with one additional commit since the last revision:
> 
>   fix identation issue

src/jdk.jcmd/share/classes/sun/tools/jmap/JMap.java line 232:

> 230:                if (compress_level.length() == 0) {
> 231:                    System.err.println("Fail: no number provided in option: '" + subopt + "'");
> 232:                    usage(1);

Actually lines 231 and 232 have the correct indentation. It's 229, 230, and 233 that are wrong. They need to be indented one more space.

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

PR: https://git.openjdk.java.net/jdk/pull/3598


More information about the serviceability-dev mailing list