Hi Sergey, Christoph, thanks for the review!
Sure, here you go: Old output, unsorted: thomas@mainframe /shared/projects/openjdk/jdk-submit-hs/output-fastdebug $ ./images/jdk/bin/jcmd test3.Example2 help 24278: The following commands are available: VM.log VM.native_memory ManagementAgent.status ManagementAgent.stop ManagementAgent.start_local ManagementAgent.start Compiler.directives_clear Compiler.directives_remove Compiler.directives_add Compiler.directives_print Compiler.CodeHeap_Analytics VM.print_touched_methods Compiler.codecache Compiler.codelist Compiler.queue VM.classloader_stats Thread.print JVMTI.data_dump JVMTI.agent_load VM.metaspace VM.stringtable VM.symboltable VM.class_hierarchy VM.systemdictionary GC.class_stats GC.class_histogram GC.heap_dump GC.finalizer_info GC.heap_info GC.run_finalization GC.run VM.info VM.uptime VM.dynlibs VM.set_flag VM.flags VM.system_properties VM.command_line VM.version help New output, sorted: thomas@mainframe /shared/projects/openjdk/jdk-submit-hs/output-fastdebug $ ./images/jdk/bin/jcmd test3.Example2 help 30230: The following commands are available: Compiler.CodeHeap_Analytics Compiler.codecache Compiler.codelist Compiler.directives_add Compiler.directives_clear Compiler.directives_print Compiler.directives_remove Compiler.queue GC.class_histogram GC.class_stats GC.finalizer_info GC.heap_dump GC.heap_info GC.run GC.run_finalization JVMTI.agent_load JVMTI.data_dump ManagementAgent.start ManagementAgent.start_local ManagementAgent.status ManagementAgent.stop Thread.print VM.class_hierarchy VM.classloader_stats VM.command_line VM.dynlibs VM.flags VM.info VM.log VM.metaspace VM.native_memory VM.print_touched_methods VM.set_flag VM.stringtable VM.symboltable VM.system_properties VM.systemdictionary VM.uptime VM.version help I'm running submit tests now, if they pass I'll push. Best Regards, Thomas On Tue, Apr 3, 2018 at 3:52 AM, [email protected] < [email protected]> wrote: > Hi Thomas, > > Added the serviceability-dev mailing list as it is a Serviceability area. > > The fix looks good to me. > One question: > Could you, please, post the sorted help output? > It is interesting how does it look like when sorted. > > Thanks, > Serguei > > > > On 3/28/18 13:08, Thomas Stüfe wrote: > >> Hi all, >> >> may I get reviews for this tiny trivial change which causes jcmd help >> output (the command list) to be sorted? >> >> bug: https://bugs.openjdk.java.net/browse/JDK-8200384 >> webrev: >> http://cr.openjdk.java.net/~stuefe/webrevs/8200384-jcmd-help >> -sorted/webrev.00/webrev/ >> >> Thanks! >> >> Best Regards, Thomas >> > >
