RFR: JDK-8256864: [windows] Improve tracing for mapping errors [v5]
Thomas Stuefe
stuefe at openjdk.java.net
Wed Dec 2 12:29:59 UTC 2020
On Wed, 2 Dec 2020 08:50:49 GMT, Richard Reingruber <rrich at openjdk.org> wrote:
>> Thomas Stuefe has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Feedback Ioi
>
> src/hotspot/os/windows/os_windows.cpp line 3218:
>
>> 3216: (attempt_reserve_memory_at(split_address, size - split) != NULL);
>> 3217: if (!rc) {
>> 3218: log_warning(os)("os::split_reserved_memory failed for " RANGE_FORMAT,
>
> I'm not very much in favor of the closing ")". It will confuse editors trying to match brackets/parenthesis when reading logs. Also I think it is redundant because an inclusive end does not make much sense without specifying how many bytes are included at the end address.
>
> I see there are already uses of that style so I won't veto if you prefer to keep it.
Okay, I agree. Good point with the editors. Exclusive end is the usual default anyway.
I'll rework this in a follow up if there is a follow up (there probably will be one).
-------------
PR: https://git.openjdk.java.net/jdk/pull/1390
More information about the hotspot-dev
mailing list