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