On Thu, 26 Oct 2023 08:58:13 GMT, Thomas Stuefe <stu...@openjdk.org> wrote:
>> When using 'MemStat' CompileCommand, we accidentally register the command if >> an invalid suboption had been specified. Fixed, added regression test >> (verified). > > Thomas Stuefe has updated the pull request incrementally with two additional > commits since the last revision: > > - remove tab > - Make patch more palatable test/hotspot/jtreg/serviceability/dcmd/compiler/CompilerMemoryStatisticTest.java line 53: > 51: * @run main/othervm -XX:CompileCommand=memstat,*.*,collect > CompilerMemoryStatisticTest > 52: */ > 53: Reviewer hint: Removed because its redundant with the syntax check introduced in MemStatTest.java ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/16335#discussion_r1372819967
