fix benchmarks

Author: sebcrozet

crates/parry2d/examples/solid_point_query2d.rs

@@ -1,4 +1,4 @@
-use parry2d::math::{Pose, Vector, Vector};
+use parry2d::math::{Pose, Vector};
 use parry2d::query::PointQuery;
 use parry2d::shape::Cuboid;
 
@@ -9,23 +9,23 @@ fn main() {
 
     // Solid projection.
     assert_eq!(
-        cuboid.distance_to_point(&Pose::identity(), &pt_inside, true),
+        cuboid.distance_to_point(&Pose::identity(), pt_inside, true),
         0.0
     );
 
     // Non-solid projection.
     assert_eq!(
-        cuboid.distance_to_point(&Pose::identity(), &pt_inside, false),
+        cuboid.distance_to_point(&Pose::identity(), pt_inside, false),
         -1.0
     );
 
     // The other point is outside of the cuboid so the `solid` flag has no effect.
     assert_eq!(
-        cuboid.distance_to_point(&Pose::identity(), &pt_outside, false),
+        cuboid.distance_to_point(&Pose::identity(), pt_outside, false),
         1.0
     );
     assert_eq!(
-        cuboid.distance_to_point(&Pose::identity(), &pt_outside, true),
+        cuboid.distance_to_point(&Pose::identity(), pt_outside, true),
         1.0
     );
 }

crates/parry2d/examples/solid_ray_cast2d.rs

@@ -1,4 +1,4 @@
-use parry2d::math::{Pose, Vector, Vector};
+use parry2d::math::{Pose, Vector};
 use parry2d::query::{Ray, RayCast};
 use parry2d::shape::Cuboid;
 

crates/parry2d/examples/time_of_impact_query2d.rs

@@ -20,30 +20,30 @@ fn main() {
 
     let toi_intersecting = query::cast_shapes(
         &ball_pos_intersecting,
-        &ball_vel1,
+        ball_vel1,
         &ball,
         &cuboid_pos,
-        &box_vel1,
+        box_vel1,
         &cuboid,
         ShapeCastOptions::default(),
     )
     .unwrap();
     let toi_will_touch = query::cast_shapes(
         &ball_pos_will_touch,
-        &ball_vel2,
+        ball_vel2,
         &ball,
         &cuboid_pos,
-        &box_vel2,
+        box_vel2,
         &cuboid,
         ShapeCastOptions::default(),
     )
     .unwrap();
     let toi_wont_touch = query::cast_shapes(
         &ball_pos_wont_touch,
-        &ball_vel1,
+        ball_vel1,
         &ball,
         &cuboid_pos,
-        &box_vel1,
+        box_vel1,
         &cuboid,
         ShapeCastOptions::default(),
     )

crates/parry2d/tests/geometry/aabb_scale.rs

@@ -1,13 +1,13 @@
 use parry2d::bounding_volume::Aabb;
-use parry2d::math::{Vector, Vector};
+use parry2d::math::Vector;
 
 #[test]
 fn test_aabb_scale_wrt_center() {
     let aabb = Aabb::from_half_extents(Vector::new(1.0, 2.0), Vector::new(4.0, 5.0));
     let scale = Vector::new(10.0, -20.0);
-    let scaled_aabb = aabb.scaled_wrt_center(&scale);
-    let scaled_aabb_neg = aabb.scaled_wrt_center(&-scale);
-    let scaled_aabb_abs = aabb.scaled_wrt_center(&scale.abs());
+    let scaled_aabb = aabb.scaled_wrt_center(scale);
+    let scaled_aabb_neg = aabb.scaled_wrt_center(-scale);
+    let scaled_aabb_abs = aabb.scaled_wrt_center(scale.abs());
 
     assert_eq!(&scaled_aabb, &scaled_aabb_neg);
     assert_eq!(&scaled_aabb, &scaled_aabb_abs);

crates/parry2d/tests/geometry/ball_ball_toi.rs

@@ -13,7 +13,7 @@ fn test_ball_ball_toi() {
     let v1 = Vector::new(0.0, 10.0);
     let v2 = Vector::ZERO;
 
-    let cast = query::cast_shapes(&m1, &v1, &b, &m2, &v2, &b, ShapeCastOptions::default()).unwrap();
+    let cast = query::cast_shapes(&m1, v1, &b, &m2, v2, &b, ShapeCastOptions::default()).unwrap();
 
     assert_eq!(cast.unwrap().time_of_impact, 0.9);
 }

crates/parry2d/tests/geometry/epa_convergence.rs

@@ -1,5 +1,5 @@
 use parry2d::{
-    math::{Pose, Real, Vector, Vector},
+    math::{Pose, Vector},
     query,
     shape::{Capsule, ConvexPolygon, SharedShape},
 };

crates/parry2d/tests/geometry/ray_cast.rs

@@ -1,4 +1,4 @@
-use parry2d::math::{Pose, Real, Vector, Vector};
+use parry2d::math::{Pose, Real, Vector};
 use parry2d::query::{Ray, RayCast};
 use parry2d::shape::{ConvexPolygon, Segment, Triangle};
 

crates/parry2d/tests/geometry/time_of_impact2.rs

@@ -1,4 +1,4 @@
-use parry2d::math::{Pose, Real, Vector, Vector};
+use parry2d::math::{Pose, Real, Vector};
 use parry2d::query;
 use parry2d::query::details::ShapeCastOptions;
 use parry2d::shape::{Ball, Cuboid, Polyline, Segment};
@@ -17,21 +17,21 @@ fn ball_ball_intersecting_toi() {
 
     let toi_separating = query::cast_shapes(
         &ball1_pos_intersecting,
-        &ball1_vel_separating,
+        ball1_vel_separating,
         &ball1,
         &ball2_pos,
-        &ball2_vel,
+        ball2_vel,
         &ball2,
         ShapeCastOptions::default(),
     )
     .unwrap();
 
     let toi_penetrating_ignore_pen = query::cast_shapes(
         &ball1_pos_intersecting,
-        &ball1_vel_penetrating,
+        ball1_vel_penetrating,
         &ball1,
         &ball2_pos,
-        &ball2_vel,
+        ball2_vel,
         &ball2,
         ShapeCastOptions {
             stop_at_penetration: false,
@@ -42,10 +42,10 @@ fn ball_ball_intersecting_toi() {
 
     let toi_separating_ignore_pen = query::cast_shapes(
         &ball1_pos_intersecting,
-        &ball1_vel_separating,
+        ball1_vel_separating,
         &ball1,
         &ball2_pos,
-        &ball2_vel,
+        ball2_vel,
         &ball2,
         ShapeCastOptions {
             stop_at_penetration: false,
@@ -85,30 +85,30 @@ fn ball_cuboid_toi() {
 
     let toi_intersecting = query::cast_shapes(
         &ball_pos_intersecting,
-        &ball_vel1,
+        ball_vel1,
         &ball,
         &cuboid_pos,
-        &cuboid_vel1,
+        cuboid_vel1,
         &cuboid,
         ShapeCastOptions::default(),
     )
     .unwrap();
     let toi_will_touch = query::cast_shapes(
         &ball_pos_will_touch,
-        &ball_vel2,
+        ball_vel2,
         &ball,
         &cuboid_pos,
-        &cuboid_vel2,
+        cuboid_vel2,
         &cuboid,
         ShapeCastOptions::default(),
     )
     .unwrap();
     let toi_wont_touch = query::cast_shapes(
         &ball_pos_wont_touch,
-        &ball_vel2,
+        ball_vel2,
         &ball,
         &cuboid_pos,
-        &cuboid_vel1,
+        cuboid_vel1,
         &cuboid,
         ShapeCastOptions::default(),
     )
@@ -135,10 +135,10 @@ fn cuboid_cuboid_toi_issue_214() {
 
     let hit = query::cast_shapes(
         &pos1,
-        &vel1,
+        vel1,
         &shape1,
         &pos2,
-        &vel2,
+        vel2,
         &shape2,
         ShapeCastOptions::default(),
     )
@@ -166,10 +166,10 @@ fn cast_shapes_should_return_toi_for_ball_and_rotated_polyline() {
 
     let hit = query::cast_shapes(
         &ball_isometry,
-        &ball_velocity,
+        ball_velocity,
         &ball,
         &polyline_isometry,
-        &polyline_velocity,
+        polyline_velocity,
         &polyline,
         ShapeCastOptions::with_max_time_of_impact(1.0),
     )
@@ -198,10 +198,10 @@ fn cast_shapes_should_return_toi_for_ball_and_rotated_segment() {
 
     let hit = query::cast_shapes(
         &ball_isometry,
-        &ball_velocity,
+        ball_velocity,
         &ball,
         &segment_isometry,
-        &segment_velocity,
+        segment_velocity,
         &segment,
         ShapeCastOptions::with_max_time_of_impact(1.0),
     )
@@ -230,10 +230,10 @@ fn cast_shapes_should_return_toi_for_rotated_segment_and_ball() {
 
     let hit = query::cast_shapes(
         &segment_isometry,
-        &segment_velocity,
+        segment_velocity,
         &segment,
         &ball_isometry,
-        &ball_velocity,
+        ball_velocity,
         &ball,
         ShapeCastOptions::with_max_time_of_impact(1.0),
     )

crates/parry2d/tests/query/point_composite_shape.rs

@@ -13,12 +13,12 @@ fn project_local_point_and_get_feature_gets_the_enclosing_triangle() {
     let mesh = TriMesh::new(vertices, vec![[0, 1, 2], [3, 0, 2]]).unwrap();
     let query_pt = Vector::new(0.6, 0.6); // Inside the top-right triangle (index 1)
 
-    let (proj, feat) = mesh.project_local_point_and_get_feature(&query_pt);
+    let (proj, feat) = mesh.project_local_point_and_get_feature(query_pt);
 
     let correct_tri_idx = 1;
     let correct_tri = mesh.triangle(correct_tri_idx);
 
-    let is_inside_correct = correct_tri.contains_local_point(&query_pt);
+    let is_inside_correct = correct_tri.contains_local_point(query_pt);
 
     assert!(is_inside_correct);
     assert_eq!(proj.is_inside, is_inside_correct);
@@ -39,12 +39,12 @@ fn project_local_point_and_get_feature_projects_correctly_from_outside() {
     {
         let query_pt = Vector::new(-1.0, 0.0); // Left from the bottom-left triangle (index 0)
 
-        let (proj, feat) = mesh.project_local_point_and_get_feature(&query_pt);
+        let (proj, feat) = mesh.project_local_point_and_get_feature(query_pt);
 
         let correct_tri_idx = 0;
         let correct_tri = mesh.triangle(correct_tri_idx);
 
-        let is_inside_correct = correct_tri.contains_local_point(&query_pt);
+        let is_inside_correct = correct_tri.contains_local_point(query_pt);
 
         assert_eq!(is_inside_correct, false);
         assert_eq!(proj.is_inside, is_inside_correct);
@@ -54,12 +54,12 @@ fn project_local_point_and_get_feature_projects_correctly_from_outside() {
     {
         let query_pt = Vector::new(0.5, 2.0); // Above the top-right triangle (index 1)
 
-        let (proj, feat) = mesh.project_local_point_and_get_feature(&query_pt);
+        let (proj, feat) = mesh.project_local_point_and_get_feature(query_pt);
 
         let correct_tri_idx = 1;
         let correct_tri = mesh.triangle(correct_tri_idx);
 
-        let is_inside_correct = correct_tri.contains_local_point(&query_pt);
+        let is_inside_correct = correct_tri.contains_local_point(query_pt);
 
         assert_eq!(is_inside_correct, false);
         assert_eq!(proj.is_inside, is_inside_correct);

crates/parry2d/tests/query/point_triangle.rs

@@ -12,8 +12,8 @@ fn project_local_point_point_on_ab() {
 
     let query_pt = Vector::new(1.4, 1.0);
 
-    let proj1 = tri1.project_local_point(&query_pt, false); // Used to fail on 0.14 and earlier
-    let proj2 = tri2.project_local_point(&query_pt, false);
+    let proj1 = tri1.project_local_point(query_pt, false); // Used to fail on 0.14 and earlier
+    let proj2 = tri2.project_local_point(query_pt, false);
 
     assert_eq!(proj1.point, proj2.point);
     assert_eq!(proj1.point, query_pt);