forestia/bevy
2
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

replace std::f32::EPSILON with f32::EPSILON (#13267)

Browse Source 
# Objective
fixes clippy warning related to using a std::f32::EPSILON which is
planned to be depreciated for f32::EPSILON
This commit is contained in:
moonlightaria 2024-05-07 01:23:53 -04:00 committed by GitHub
parent 4f9f987099
commit 1126b5a3d6
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
2 changed files with 36 additions and 36 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

36
crates/bevy_math/src/bounding/bounded2d/mod.rs
View File

@ -271,12 +271,12 @@ mod aabb2d_tests {
min: Vec2::new(-0.5, -1.), min: Vec2::new(-0.5, -1.),
max: Vec2::new(1., 1.), max: Vec2::new(1., 1.),
}; };
assert!((aabb.center() - Vec2::new(0.25, 0.)).length() < std::f32::EPSILON); assert!((aabb.center() - Vec2::new(0.25, 0.)).length() < f32::EPSILON);
let aabb = Aabb2d { let aabb = Aabb2d {
min: Vec2::new(5., -10.), min: Vec2::new(5., -10.),
max: Vec2::new(10., -5.), max: Vec2::new(10., -5.),
}; };
assert!((aabb.center() - Vec2::new(7.5, -7.5)).length() < std::f32::EPSILON); assert!((aabb.center() - Vec2::new(7.5, -7.5)).length() < f32::EPSILON);
} }
#[test] #[test]
@ -286,7 +286,7 @@ mod aabb2d_tests {
max: Vec2::new(1., 1.), max: Vec2::new(1., 1.),
}; };
let half_size = aabb.half_size(); let half_size = aabb.half_size();
assert!((half_size - Vec2::new(0.75, 1.)).length() < std::f32::EPSILON); assert!((half_size - Vec2::new(0.75, 1.)).length() < f32::EPSILON);
} }
#[test] #[test]
@ -295,12 +295,12 @@ mod aabb2d_tests {
min: Vec2::new(-1., -1.), min: Vec2::new(-1., -1.),
max: Vec2::new(1., 1.), max: Vec2::new(1., 1.),
}; };
assert!((aabb.visible_area() - 4.).abs() < std::f32::EPSILON); assert!((aabb.visible_area() - 4.).abs() < f32::EPSILON);
let aabb = Aabb2d { let aabb = Aabb2d {
min: Vec2::new(0., 0.), min: Vec2::new(0., 0.),
max: Vec2::new(1., 0.5), max: Vec2::new(1., 0.5),
}; };
assert!((aabb.visible_area() - 0.5).abs() < std::f32::EPSILON); assert!((aabb.visible_area() - 0.5).abs() < f32::EPSILON);
} }
#[test] #[test]
@ -332,8 +332,8 @@ mod aabb2d_tests {
max: Vec2::new(0.75, 1.), max: Vec2::new(0.75, 1.),
}; };
let merged = a.merge(&b); let merged = a.merge(&b);
assert!((merged.min - Vec2::new(-2., -1.)).length() < std::f32::EPSILON); assert!((merged.min - Vec2::new(-2., -1.)).length() < f32::EPSILON);
assert!((merged.max - Vec2::new(1., 1.)).length() < std::f32::EPSILON); assert!((merged.max - Vec2::new(1., 1.)).length() < f32::EPSILON);
assert!(merged.contains(&a)); assert!(merged.contains(&a));
assert!(merged.contains(&b)); assert!(merged.contains(&b));
assert!(!a.contains(&merged)); assert!(!a.contains(&merged));
@ -347,8 +347,8 @@ mod aabb2d_tests {
max: Vec2::new(1., 1.), max: Vec2::new(1., 1.),
}; };
let padded = a.grow(Vec2::ONE); let padded = a.grow(Vec2::ONE);
assert!((padded.min - Vec2::new(-2., -2.)).length() < std::f32::EPSILON); assert!((padded.min - Vec2::new(-2., -2.)).length() < f32::EPSILON);
assert!((padded.max - Vec2::new(2., 2.)).length() < std::f32::EPSILON); assert!((padded.max - Vec2::new(2., 2.)).length() < f32::EPSILON);
assert!(padded.contains(&a)); assert!(padded.contains(&a));
assert!(!a.contains(&padded)); assert!(!a.contains(&padded));
} }
@ -360,8 +360,8 @@ mod aabb2d_tests {
max: Vec2::new(2., 2.), max: Vec2::new(2., 2.),
}; };
let shrunk = a.shrink(Vec2::ONE); let shrunk = a.shrink(Vec2::ONE);
assert!((shrunk.min - Vec2::new(-1., -1.)).length() < std::f32::EPSILON); assert!((shrunk.min - Vec2::new(-1., -1.)).length() < f32::EPSILON);
assert!((shrunk.max - Vec2::new(1., 1.)).length() < std::f32::EPSILON); assert!((shrunk.max - Vec2::new(1., 1.)).length() < f32::EPSILON);
assert!(a.contains(&shrunk)); assert!(a.contains(&shrunk));
assert!(!shrunk.contains(&a)); assert!(!shrunk.contains(&a));
} }
@ -373,8 +373,8 @@ mod aabb2d_tests {
max: Vec2::ONE, max: Vec2::ONE,
}; };
let scaled = a.scale_around_center(Vec2::splat(2.)); let scaled = a.scale_around_center(Vec2::splat(2.));
assert!((scaled.min - Vec2::splat(-2.)).length() < std::f32::EPSILON); assert!((scaled.min - Vec2::splat(-2.)).length() < f32::EPSILON);
assert!((scaled.max - Vec2::splat(2.)).length() < std::f32::EPSILON); assert!((scaled.max - Vec2::splat(2.)).length() < f32::EPSILON);
assert!(!a.contains(&scaled)); assert!(!a.contains(&scaled));
assert!(scaled.contains(&a)); assert!(scaled.contains(&a));
} }
@ -647,8 +647,8 @@ mod bounding_circle_tests {
let a = BoundingCircle::new(Vec2::ONE, 5.); let a = BoundingCircle::new(Vec2::ONE, 5.);
let b = BoundingCircle::new(Vec2::new(1., -4.), 1.); let b = BoundingCircle::new(Vec2::new(1., -4.), 1.);
let merged = a.merge(&b); let merged = a.merge(&b);
assert!((merged.center - Vec2::new(1., 0.5)).length() < std::f32::EPSILON); assert!((merged.center - Vec2::new(1., 0.5)).length() < f32::EPSILON);
assert!((merged.radius() - 5.5).abs() < std::f32::EPSILON); assert!((merged.radius() - 5.5).abs() < f32::EPSILON);
assert!(merged.contains(&a)); assert!(merged.contains(&a));
assert!(merged.contains(&b)); assert!(merged.contains(&b));
assert!(!a.contains(&merged)); assert!(!a.contains(&merged));
@ -680,7 +680,7 @@ mod bounding_circle_tests {
fn grow() { fn grow() {
let a = BoundingCircle::new(Vec2::ONE, 5.); let a = BoundingCircle::new(Vec2::ONE, 5.);
let padded = a.grow(1.25); let padded = a.grow(1.25);
assert!((padded.radius() - 6.25).abs() < std::f32::EPSILON); assert!((padded.radius() - 6.25).abs() < f32::EPSILON);
assert!(padded.contains(&a)); assert!(padded.contains(&a));
assert!(!a.contains(&padded)); assert!(!a.contains(&padded));
} }
@ -689,7 +689,7 @@ mod bounding_circle_tests {
fn shrink() { fn shrink() {
let a = BoundingCircle::new(Vec2::ONE, 5.); let a = BoundingCircle::new(Vec2::ONE, 5.);
let shrunk = a.shrink(0.5); let shrunk = a.shrink(0.5);
assert!((shrunk.radius() - 4.5).abs() < std::f32::EPSILON); assert!((shrunk.radius() - 4.5).abs() < f32::EPSILON);
assert!(a.contains(&shrunk)); assert!(a.contains(&shrunk));
assert!(!shrunk.contains(&a)); assert!(!shrunk.contains(&a));
} }
@ -698,7 +698,7 @@ mod bounding_circle_tests {
fn scale_around_center() { fn scale_around_center() {
let a = BoundingCircle::new(Vec2::ONE, 5.); let a = BoundingCircle::new(Vec2::ONE, 5.);
let scaled = a.scale_around_center(2.); let scaled = a.scale_around_center(2.);
assert!((scaled.radius() - 10.).abs() < std::f32::EPSILON); assert!((scaled.radius() - 10.).abs() < f32::EPSILON);
assert!(!a.contains(&scaled)); assert!(!a.contains(&scaled));
assert!(scaled.contains(&a)); assert!(scaled.contains(&a));
} }

36
crates/bevy_math/src/bounding/bounded3d/mod.rs
View File

@ -268,12 +268,12 @@ mod aabb3d_tests {
min: Vec3A::new(-0.5, -1., -0.5), min: Vec3A::new(-0.5, -1., -0.5),
max: Vec3A::new(1., 1., 2.), max: Vec3A::new(1., 1., 2.),
}; };
assert!((aabb.center() - Vec3A::new(0.25, 0., 0.75)).length() < std::f32::EPSILON); assert!((aabb.center() - Vec3A::new(0.25, 0., 0.75)).length() < f32::EPSILON);
let aabb = Aabb3d { let aabb = Aabb3d {
min: Vec3A::new(5., 5., -10.), min: Vec3A::new(5., 5., -10.),
max: Vec3A::new(10., 10., -5.), max: Vec3A::new(10., 10., -5.),
}; };
assert!((aabb.center() - Vec3A::new(7.5, 7.5, -7.5)).length() < std::f32::EPSILON); assert!((aabb.center() - Vec3A::new(7.5, 7.5, -7.5)).length() < f32::EPSILON);
} }
#[test] #[test]
@ -282,7 +282,7 @@ mod aabb3d_tests {
min: Vec3A::new(-0.5, -1., -0.5), min: Vec3A::new(-0.5, -1., -0.5),
max: Vec3A::new(1., 1., 2.), max: Vec3A::new(1., 1., 2.),
}; };
assert!((aabb.half_size() - Vec3A::new(0.75, 1., 1.25)).length() < std::f32::EPSILON); assert!((aabb.half_size() - Vec3A::new(0.75, 1., 1.25)).length() < f32::EPSILON);
} }
#[test] #[test]
@ -291,12 +291,12 @@ mod aabb3d_tests {
min: Vec3A::new(-1., -1., -1.), min: Vec3A::new(-1., -1., -1.),
max: Vec3A::new(1., 1., 1.), max: Vec3A::new(1., 1., 1.),
}; };
assert!((aabb.visible_area() - 12.).abs() < std::f32::EPSILON); assert!((aabb.visible_area() - 12.).abs() < f32::EPSILON);
let aabb = Aabb3d { let aabb = Aabb3d {
min: Vec3A::new(0., 0., 0.), min: Vec3A::new(0., 0., 0.),
max: Vec3A::new(1., 0.5, 0.25), max: Vec3A::new(1., 0.5, 0.25),
}; };
assert!((aabb.visible_area() - 0.875).abs() < std::f32::EPSILON); assert!((aabb.visible_area() - 0.875).abs() < f32::EPSILON);
} }
#[test] #[test]
@ -328,8 +328,8 @@ mod aabb3d_tests {
max: Vec3A::new(0.75, 1., 2.), max: Vec3A::new(0.75, 1., 2.),
}; };
let merged = a.merge(&b); let merged = a.merge(&b);
assert!((merged.min - Vec3A::new(-2., -1., -1.)).length() < std::f32::EPSILON); assert!((merged.min - Vec3A::new(-2., -1., -1.)).length() < f32::EPSILON);
assert!((merged.max - Vec3A::new(1., 1., 2.)).length() < std::f32::EPSILON); assert!((merged.max - Vec3A::new(1., 1., 2.)).length() < f32::EPSILON);
assert!(merged.contains(&a)); assert!(merged.contains(&a));
assert!(merged.contains(&b)); assert!(merged.contains(&b));
assert!(!a.contains(&merged)); assert!(!a.contains(&merged));
@ -343,8 +343,8 @@ mod aabb3d_tests {
max: Vec3A::new(1., 1., 1.), max: Vec3A::new(1., 1., 1.),
}; };
let padded = a.grow(Vec3A::ONE); let padded = a.grow(Vec3A::ONE);
assert!((padded.min - Vec3A::new(-2., -2., -2.)).length() < std::f32::EPSILON); assert!((padded.min - Vec3A::new(-2., -2., -2.)).length() < f32::EPSILON);
assert!((padded.max - Vec3A::new(2., 2., 2.)).length() < std::f32::EPSILON); assert!((padded.max - Vec3A::new(2., 2., 2.)).length() < f32::EPSILON);
assert!(padded.contains(&a)); assert!(padded.contains(&a));
assert!(!a.contains(&padded)); assert!(!a.contains(&padded));
} }
@ -356,8 +356,8 @@ mod aabb3d_tests {
max: Vec3A::new(2., 2., 2.), max: Vec3A::new(2., 2., 2.),
}; };
let shrunk = a.shrink(Vec3A::ONE); let shrunk = a.shrink(Vec3A::ONE);
assert!((shrunk.min - Vec3A::new(-1., -1., -1.)).length() < std::f32::EPSILON); assert!((shrunk.min - Vec3A::new(-1., -1., -1.)).length() < f32::EPSILON);
assert!((shrunk.max - Vec3A::new(1., 1., 1.)).length() < std::f32::EPSILON); assert!((shrunk.max - Vec3A::new(1., 1., 1.)).length() < f32::EPSILON);
assert!(a.contains(&shrunk)); assert!(a.contains(&shrunk));
assert!(!shrunk.contains(&a)); assert!(!shrunk.contains(&a));
} }
@ -369,8 +369,8 @@ mod aabb3d_tests {
max: Vec3A::ONE, max: Vec3A::ONE,
}; };
let scaled = a.scale_around_center(Vec3A::splat(2.)); let scaled = a.scale_around_center(Vec3A::splat(2.));
assert!((scaled.min - Vec3A::splat(-2.)).length() < std::f32::EPSILON); assert!((scaled.min - Vec3A::splat(-2.)).length() < f32::EPSILON);
assert!((scaled.max - Vec3A::splat(2.)).length() < std::f32::EPSILON); assert!((scaled.max - Vec3A::splat(2.)).length() < f32::EPSILON);
assert!(!a.contains(&scaled)); assert!(!a.contains(&scaled));
assert!(scaled.contains(&a)); assert!(scaled.contains(&a));
} }
@ -671,8 +671,8 @@ mod bounding_sphere_tests {
let a = BoundingSphere::new(Vec3::ONE, 5.); let a = BoundingSphere::new(Vec3::ONE, 5.);
let b = BoundingSphere::new(Vec3::new(1., 1., -4.), 1.); let b = BoundingSphere::new(Vec3::new(1., 1., -4.), 1.);
let merged = a.merge(&b); let merged = a.merge(&b);
assert!((merged.center - Vec3A::new(1., 1., 0.5)).length() < std::f32::EPSILON); assert!((merged.center - Vec3A::new(1., 1., 0.5)).length() < f32::EPSILON);
assert!((merged.radius() - 5.5).abs() < std::f32::EPSILON); assert!((merged.radius() - 5.5).abs() < f32::EPSILON);
assert!(merged.contains(&a)); assert!(merged.contains(&a));
assert!(merged.contains(&b)); assert!(merged.contains(&b));
assert!(!a.contains(&merged)); assert!(!a.contains(&merged));
@ -704,7 +704,7 @@ mod bounding_sphere_tests {
fn grow() { fn grow() {
let a = BoundingSphere::new(Vec3::ONE, 5.); let a = BoundingSphere::new(Vec3::ONE, 5.);
let padded = a.grow(1.25); let padded = a.grow(1.25);
assert!((padded.radius() - 6.25).abs() < std::f32::EPSILON); assert!((padded.radius() - 6.25).abs() < f32::EPSILON);
assert!(padded.contains(&a)); assert!(padded.contains(&a));
assert!(!a.contains(&padded)); assert!(!a.contains(&padded));
} }
@ -713,7 +713,7 @@ mod bounding_sphere_tests {
fn shrink() { fn shrink() {
let a = BoundingSphere::new(Vec3::ONE, 5.); let a = BoundingSphere::new(Vec3::ONE, 5.);
let shrunk = a.shrink(0.5); let shrunk = a.shrink(0.5);
assert!((shrunk.radius() - 4.5).abs() < std::f32::EPSILON); assert!((shrunk.radius() - 4.5).abs() < f32::EPSILON);
assert!(a.contains(&shrunk)); assert!(a.contains(&shrunk));
assert!(!shrunk.contains(&a)); assert!(!shrunk.contains(&a));
} }
@ -722,7 +722,7 @@ mod bounding_sphere_tests {
fn scale_around_center() { fn scale_around_center() {
let a = BoundingSphere::new(Vec3::ONE, 5.); let a = BoundingSphere::new(Vec3::ONE, 5.);
let scaled = a.scale_around_center(2.); let scaled = a.scale_around_center(2.);
assert!((scaled.radius() - 10.).abs() < std::f32::EPSILON); assert!((scaled.radius() - 10.).abs() < f32::EPSILON);
assert!(!a.contains(&scaled)); assert!(!a.contains(&scaled));
assert!(scaled.contains(&a)); assert!(scaled.contains(&a));
} }