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

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

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

2023-10-26 Thread Thomas Stuefe
On Wed, 25 Oct 2023 22:16:03 GMT, David Holmes wrote: >>> If "omitting value means 'collect'" then why do we not simply set value = >>> (uintx)MemStatAction::collect ? Otherwise what is that message supposed to >>> mean? >> >> We only enter this function if a value for this command had been gi

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

2023-10-26 Thread Thomas Stuefe
> 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 - Ma