RFR(s): 8178364: Command-line flags of type double should accept integer values

Kim Barrett kim.barrett at oracle.com
Tue Apr 11 23:12:33 UTC 2017


> On Apr 10, 2017, at 7:03 AM, David Holmes <david.holmes at oracle.com> wrote:
> 
> On 10/04/2017 5:40 PM, Per Liden wrote:
>> With the way this code is structured we only end up in
>> set_fp_numeric_flag() if the string looks like a float to begin with,
>> i.e. has the format "<flag>=<num>.<num>". If the flag looks like
>> "<flag>=<num>" we end up in set_numeric_flag(). Have a look at the call
>> sites for set_fp_numeric_flag() and set_numeric_flag() and you'll see.
> 
> That's somewhat odd - shouldn't the type of the flag dictate how we try to parse it ??

Coming to this a little late, but I remember making that same point during the review
of the command line range checking stuff, and thought there was a bug filed for it.
But I can’t find any such bug.




More information about the hotspot-runtime-dev mailing list