Clippy lints / Restriction
Clippy lint · Restriction · checked with Clippy 0.1.99
clippy::float_cmp_const: strict comparison of `f32` or `f64` constant
What it meansChecks for (in-)equality comparisons on constant floating-point values (apart from zero), except in functions called *eq* (which probably implement equality for a type involving floats). It is in the restriction group and off by default.
What you see
warning: strict comparison of `f32` or `f64` constant --> main.rs:11:17 | 11 | let are_equal = x == Y; | ^^^^^^ help: consider comparing them within some margin of error: `(x - Y).abs() < error_margin` | = help: for further information visit https://rust-lang.github.io/rust-clippy/stable/index.html#float_cmp_const = note: requested on the command line with `-W clippy::float-cmp-const`
Real output of Clippy 0.1.99 (Rust 1.99) on the example below. Names and line numbers in yours will differ.
Does it matter?
Floating point calculations are usually imprecise, so asking if two values are exactly equal is asking for trouble because arriving at the same logical result via different routes (e.g. calculation versus constant) may yield different values.
Lints that forbid language features for specific codebases (no unwrap, no indexing...). Off by default; pick them one by one, never the whole group.
How to fix it
Clippy does not fix this one automatically (its suggestion may change behaviour, so it only shows it). Change the code by hand:
Flagged
let a: f64 = 1000.1;
let b: f64 = 0.2;
let x = a + b;
const Y: f64 = 1000.3; // Expected value.
// Actual value: 1000.3000000000001
println!("{x}");
let are_equal = x == Y;
println!("{are_equal}"); // false
Write this instead
let a: f64 = 1000.1;
let b: f64 = 0.2;
let x = a + b;
const Y: f64 = 1000.3; // Expected value.
const ALLOWED_ERROR_VEHICLE_LENGTH_CM: f64 = 0.1;
let within_tolerance = (x - Y).abs() < ALLOWED_ERROR_VEHICLE_LENGTH_CM;
println!("{within_tolerance}"); // true
Turn it off, or on
It is off by default. To use it, set it to warn or deny:
Whole crate (Cargo.toml)
[lints.clippy]
float_cmp_const = "warn"In code
#![warn(clippy::float_cmp_const)]One run
cargo clippy -- -W clippy::float-cmp-constThe [lints] table needs Rust 1.74 or newer. In a workspace, put it under [workspace.lints.clippy] and add [lints] workspace = true to each member.
Clippy's own documentation
What it does
Checks for (in-)equality comparisons on constant floating-point
values (apart from zero), except in functions called *eq* (which probably
implement equality for a type involving floats).
Why restrict this?
Floating point calculations are usually imprecise, so asking if two values are exactly equal is asking for trouble because arriving at the same logical result via different routes (e.g. calculation versus constant) may yield different values.
Example
let a: f64 = 1000.1;
let b: f64 = 0.2;
let x = a + b;
const Y: f64 = 1000.3; // Expected value.
// Actual value: 1000.3000000000001
println!("{x}");
let are_equal = x == Y;
println!("{are_equal}"); // false
The correct way to compare floating point numbers is to define an allowed error margin. This may be challenging if there is no “natural” error margin to permit. Broadly speaking, there are two cases:
- If your values are in a known range and you can define a threshold for “close enough to be equal”, it may be appropriate to define an absolute error margin. For example, if your data is “length of vehicle in centimeters”, you may consider 0.1 cm to be “close enough”.
- If your code is more general and you do not know the range of values, you should use a relative error margin, accepting e.g. 0.1% of error regardless of specific values.
For the scenario where you can define a meaningful absolute error margin, consider using:
let a: f64 = 1000.1;
let b: f64 = 0.2;
let x = a + b;
const Y: f64 = 1000.3; // Expected value.
const ALLOWED_ERROR_VEHICLE_LENGTH_CM: f64 = 0.1;
let within_tolerance = (x - Y).abs() < ALLOWED_ERROR_VEHICLE_LENGTH_CM;
println!("{within_tolerance}"); // true
NOTE: Do not use f64::EPSILON - while the error margin is often called “epsilon”, this is
a different use of the term that is not suitable for floating point equality comparison.
Indeed, for the example above using f64::EPSILON as the allowed error would return false.
For the scenario where no meaningful absolute error can be defined, refer to
the floating point guide
for a reference implementation of relative error based comparison of floating point values.
MIN_NORMAL in the reference implementation is equivalent to MIN_POSITIVE in Rust.
From the Clippy lint list (0.1.99), part of the Rust project, MIT or Apache-2.0. Official entry.
Related lints
- unwrap_usedChecks for .unwrap() or .unwrap_err() calls on Results and .unwrap() call on Options
- module_name_repetitionsDetects public item names that are prefixed or suffixed by the containing public module’s name
- expect_usedChecks for .expect() or .expect_err() calls on Results and .expect() call on Options
- field_scoped_visibility_modifiersscoped visibility modifier on a field
- filetype_is_fileChecks for FileType::is_file()
- float_arithmeticChecks for float arithmetic
- fn_to_numeric_cast_anycasting function pointer `fn1` to `usize`
- get_unwrapcalled `.get().unwrap()` on a Vec
- host_endian_bytesuse of `to_ne_bytes`
All 847 Clippy lints · Rust compiler errors · Cargo.toml checker
Up next
Related lintclippy::unwrap_usedChecks for .unwrap() or .unwrap_err() calls on Results and .unwrap() call on OptionsRead it next →Did this answer your question?