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