RFR: 8283013: Simplify Arguments::parse_argument() [v2]
David Holmes
dholmes at openjdk.java.net
Mon Mar 28 22:37:46 UTC 2022
On Mon, 28 Mar 2022 20:57:46 GMT, Ioi Lam <iklam at openjdk.org> wrote:
>> src/hotspot/share/runtime/arguments.cpp line 896:
>>
>>> 894:
>>> 895: static bool set_fp_numeric_flag(JVMFlag* flag, const char* value, JVMFlagOrigin origin) {
>>> 896: if (*value == '\0' || isspace(*value)) {
>>
>> Please preceded with comment
>>
>> // strtod allows leading whitespace, but our flag format does not.
>
> Fixed.
Sorry I meant for the comment to precede line 896.
-------------
PR: https://git.openjdk.java.net/jdk/pull/7916
More information about the hotspot-dev
mailing list