RFR: 8310920: Fix -Wconversion warnings in command line flags [v2]
David Holmes
dholmes at openjdk.org
Tue Jun 27 23:30:05 UTC 2023
On Tue, 27 Jun 2023 12:51:16 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:
>> This is broken out from but tested with the PR from https://github.com/openjdk/jdk/pull/14659
>
> Coleen Phillimore has updated the pull request incrementally with one additional commit since the last revision:
>
> Make phase parameter JVMFlagConstraintPhase and fix another double/float conversion.
Phase changes look good - thanks!
One minor nit below but approved.
src/hotspot/share/runtime/flags/jvmFlag.cpp line 582:
> 580:
> 581: JVMFlag* JVMFlag::fuzzy_match(const char* name, size_t length, bool allow_locked) {
> 582: double VMOptionsFuzzyMatchSimilarity = 0.7f;
You don't want the 'f' on `0.7f` now
-------------
Marked as reviewed by dholmes (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/14665#pullrequestreview-1501992492
PR Review Comment: https://git.openjdk.org/jdk/pull/14665#discussion_r1244475966
More information about the hotspot-runtime-dev
mailing list