From cc501e4f3b0fbdea9e80d54d595908c1129a7dcd Mon Sep 17 00:00:00 2001
From: stephengold <sgold@sonic.net>
Date: Tue, 2 Jul 2024 10:28:43 -0700
Subject: [PATCH] library:  simple refactoring

---
 .../java/com/jme3/bullet/CollisionSpace.java     |  3 +--
 .../main/java/com/jme3/bullet/PhysicsSpace.java  | 16 ++++++++--------
 .../jme3/bullet/control/RigidBodyControl.java    |  8 ++++----
 .../java/com/jme3/bullet/util/DebugMeshKey.java  |  4 ++--
 4 files changed, 15 insertions(+), 16 deletions(-)

diff --git a/MinieLibrary/src/main/java/com/jme3/bullet/CollisionSpace.java b/MinieLibrary/src/main/java/com/jme3/bullet/CollisionSpace.java
index 0d34c83b8..bf14c278f 100644
--- a/MinieLibrary/src/main/java/com/jme3/bullet/CollisionSpace.java
+++ b/MinieLibrary/src/main/java/com/jme3/bullet/CollisionSpace.java
@@ -673,8 +673,7 @@ public List<PhysicsRayTestResult> rayTestRaw(
     public void remove(Object object) {
         if (object == null) {
             return;
-        }
-        if (object instanceof PhysicsCollisionObject) {
+        } else if (object instanceof PhysicsCollisionObject) {
             removeCollisionObject((PhysicsCollisionObject) object);
         } else {
             String typeName = object.getClass().getCanonicalName();
diff --git a/MinieLibrary/src/main/java/com/jme3/bullet/PhysicsSpace.java b/MinieLibrary/src/main/java/com/jme3/bullet/PhysicsSpace.java
index 613e56411..a4567568c 100644
--- a/MinieLibrary/src/main/java/com/jme3/bullet/PhysicsSpace.java
+++ b/MinieLibrary/src/main/java/com/jme3/bullet/PhysicsSpace.java
@@ -1089,10 +1089,10 @@ public void add(Object object) {
         if (object instanceof PhysicsControl) {
             ((PhysicsControl) object).setPhysicsSpace(this);
         } else if (object instanceof Spatial) {
-            Spatial node = (Spatial) object;
-            for (int i = 0; i < node.getNumControls(); ++i) {
-                if (node.getControl(i) instanceof PhysicsControl) {
-                    add(node.getControl(i));
+            Spatial spatial = (Spatial) object;
+            for (int i = 0; i < spatial.getNumControls(); ++i) {
+                if (spatial.getControl(i) instanceof PhysicsControl) {
+                    add(spatial.getControl(i));
                 }
             }
         } else if (object instanceof PhysicsJoint) {
@@ -1225,10 +1225,10 @@ public void remove(Object object) {
         if (object instanceof PhysicsControl) {
             ((PhysicsControl) object).setPhysicsSpace(null);
         } else if (object instanceof Spatial) {
-            Spatial node = (Spatial) object;
-            for (int i = 0; i < node.getNumControls(); ++i) {
-                if (node.getControl(i) instanceof PhysicsControl) {
-                    remove((node.getControl(i)));
+            Spatial spatial = (Spatial) object;
+            for (int i = 0; i < spatial.getNumControls(); ++i) {
+                if (spatial.getControl(i) instanceof PhysicsControl) {
+                    remove((spatial.getControl(i)));
                 }
             }
         } else if (object instanceof PhysicsJoint) {
diff --git a/MinieLibrary/src/main/java/com/jme3/bullet/control/RigidBodyControl.java b/MinieLibrary/src/main/java/com/jme3/bullet/control/RigidBodyControl.java
index 5e7cd9568..967aca521 100644
--- a/MinieLibrary/src/main/java/com/jme3/bullet/control/RigidBodyControl.java
+++ b/MinieLibrary/src/main/java/com/jme3/bullet/control/RigidBodyControl.java
@@ -416,10 +416,10 @@ public void setSpatial(Spatial controlledSpatial) {
                 createCollisionShape();
                 rebuildRigidBody();
             }
-            Vector3f translation = getSpatialTranslation(); // alias
-            setPhysicsLocation(translation);
-            Quaternion rotation = getSpatialRotation(); // alias
-            setPhysicsRotation(rotation);
+            Vector3f location = getSpatialTranslation(); // alias
+            setPhysicsLocation(location);
+            Quaternion orientation = getSpatialRotation(); // alias
+            setPhysicsRotation(orientation);
         }
     }
 
diff --git a/MinieLibrary/src/main/java/com/jme3/bullet/util/DebugMeshKey.java b/MinieLibrary/src/main/java/com/jme3/bullet/util/DebugMeshKey.java
index 7faa5330a..b664fccb1 100644
--- a/MinieLibrary/src/main/java/com/jme3/bullet/util/DebugMeshKey.java
+++ b/MinieLibrary/src/main/java/com/jme3/bullet/util/DebugMeshKey.java
@@ -162,9 +162,9 @@ public int hashCode() {
      */
     @Override
     public String toString() {
-        String desc = String.format(
+        String result = String.format(
                 "shape=%x scale=%s margin=%f res=%d normals=%s",
                 shapeId, scale, margin, resolution, normals);
-        return desc;
+        return result;
     }
 }