RFR: 8319873: Add windows implementation for jcmd System.map and System.dump_map [v10]
Simon Tooke
stooke at openjdk.org
Fri Sep 13 14:30:10 UTC 2024
On Wed, 11 Sep 2024 23:57:11 GMT, David Holmes <dholmes at openjdk.org> wrote:
>> Simon Tooke has updated the pull request incrementally with one additional commit since the last revision:
>>
>> changes per review
>
> src/hotspot/share/services/diagnosticCommand.cpp line 1196:
>
>> 1194: void SystemDumpMapDCmd::execute(DCmdSource source, TRAPS) {
>> 1195: const char* name = _filename.value();
>> 1196: if (name == NULL || name[0] == 0) {
>
> Suggestion:
>
> if (name == nullptr ...
done.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/20597#discussion_r1758973435
More information about the serviceability-dev
mailing list