RFR: JDK-8318671: Potential uninitialized uintx value after JDK-8317683 [v3]

Thomas Stuefe stuefe at openjdk.org
Thu Oct 26 09:02:12 UTC 2023


On Thu, 26 Oct 2023 08:58:13 GMT, Thomas Stuefe <stuefe at 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


More information about the serviceability-dev mailing list