Appease clippy

clippy.toml

@@ -0,0 +1 @@
+msrv = "1.63.0"

examples/dodge-the-creeps/src/player.rs

@@ -82,8 +82,8 @@ impl Player {
         let change = velocity * delta;
         let position = owner.global_position() + change;
         let position = Vector2::new(
-            position.x.max(0.0).min(self.screen_size.x),
-            position.y.max(0.0).min(self.screen_size.y),
+            position.x.clamp(0.0, self.screen_size.x),
+            position.y.clamp(0.0, self.screen_size.y),
         );
         owner.set_global_position(position);
     }

examples/scene-create/src/lib.rs

@@ -59,7 +59,7 @@ impl SceneCreate {
             Ok(spatial) => {
                 // Here is how you rename the child...
                 let key_str = format!("child_{}", self.children_spawned);
-                spatial.set_name(&key_str);
+                spatial.set_name(key_str);
 
                 let x = (self.children_spawned % 10) as f32;
                 let z = (self.children_spawned / 10) as f32;

gdnative-core/src/core_types/geom/transform.rs

@@ -103,6 +103,12 @@ impl Transform {
     This method will be renamed to translated_local in gdnative 0.12."]
     #[inline]
     pub fn translated(&self, translation: Vector3) -> Self {
+        self.translated_local(translation)
+    }
+
+    /// Returns this transform, with its origin moved by a certain `translation`
+    #[inline]
+    pub fn translated_local(&self, translation: Vector3) -> Self {
         Self {
             basis: self.basis,
             origin: self.origin + translation,
@@ -365,7 +371,7 @@ mod tests {
     #[test]
     fn translation_is_sane() {
         let translation = Vector3::new(1.0, 2.0, 3.0);
-        let t = Transform::default().translated(translation);
+        let t = Transform::default().translated_local(translation);
         assert!(t.basis.elements[0] == Vector3::new(1.0, 0.0, 0.0));
         assert!(t.basis.elements[1] == Vector3::new(0.0, 1.0, 0.0));
         assert!(t.basis.elements[2] == Vector3::new(0.0, 0.0, 1.0));

gdnative-core/src/lib.rs

@@ -25,7 +25,10 @@
     clippy::missing_safety_doc,
     clippy::non_send_fields_in_send_ty
 )]
-#![cfg_attr(feature = "gd-test", allow(clippy::blacklisted_name))]
+#![cfg_attr(
+    any(test, feature = "gd-test"),
+    allow(clippy::excessive_precision, clippy::disallowed_names)
+)]
 
 #[doc(hidden)]
 pub extern crate gdnative_sys as sys;

test/src/lib.rs

@@ -1,4 +1,4 @@
-#![allow(clippy::blacklisted_name)]
+#![allow(clippy::disallowed_names)]
 #![allow(deprecated)]
 
 use gdnative::prelude::*;

test/src/test_async.rs

@@ -9,7 +9,7 @@ pub(crate) fn run_tests() -> bool {
 
 thread_local! {
     static EXECUTOR: &'static SharedLocalPool = {
-        Box::leak(Box::new(SharedLocalPool::default()))
+        Box::leak(Box::default())
     };
 }