bugfix: some ref targets are instantiated as owners

stephengold · Nov 5, 2024 · b30cf2d · b30cf2d
1 parent b68a0c1
commit b30cf2d
Show file tree

Hide file tree

Showing 6 changed files with 6 additions and 6 deletions.
diff --git a/src/main/java/com/github/stephengold/joltjni/MotorcycleControllerSettings.java b/src/main/java/com/github/stephengold/joltjni/MotorcycleControllerSettings.java
@@ -37,7 +37,7 @@ public class MotorcycleControllerSettings
     public MotorcycleControllerSettings() {
         super(false);
         long settingsVa = createDefault();
-        setVirtualAddress(settingsVa, true);
+        setVirtualAddress(settingsVa, false); // not owner due to ref counting
     }
     // *************************************************************************
     // native private methods

diff --git a/src/main/java/com/github/stephengold/joltjni/SoftBodySharedSettings.java b/src/main/java/com/github/stephengold/joltjni/SoftBodySharedSettings.java
@@ -42,7 +42,7 @@ public class SoftBodySharedSettings
      */
     public SoftBodySharedSettings() {
         long settingsVa = createDefault();
-        setVirtualAddress(settingsVa, () -> free(settingsVa));
+        setVirtualAddress(settingsVa, null); // not owner due to ref counting
     }
 
     /**

diff --git a/src/main/java/com/github/stephengold/joltjni/TrackedVehicleControllerSettings.java b/src/main/java/com/github/stephengold/joltjni/TrackedVehicleControllerSettings.java
@@ -36,7 +36,7 @@ public class TrackedVehicleControllerSettings
      */
     public TrackedVehicleControllerSettings() {
         long settingsVa = createDefault();
-        setVirtualAddress(settingsVa, true);
+        setVirtualAddress(settingsVa, false); // not owner due to ref counting
     }
 
     /**

diff --git a/src/main/java/com/github/stephengold/joltjni/WheelSettingsTv.java b/src/main/java/com/github/stephengold/joltjni/WheelSettingsTv.java
@@ -36,7 +36,7 @@ public class WheelSettingsTv extends WheelSettings {
     public WheelSettingsTv() {
         super(true);
         long settingsVa = createDefault();
-        setVirtualAddress(settingsVa, true);
+        setVirtualAddress(settingsVa, false); // not owner due to ref counting
     }
 
     /**

diff --git a/src/main/java/com/github/stephengold/joltjni/WheelSettingsWv.java b/src/main/java/com/github/stephengold/joltjni/WheelSettingsWv.java
@@ -36,7 +36,7 @@ public class WheelSettingsWv extends WheelSettings {
     public WheelSettingsWv() {
         super(true);
         long settingsVa = createDefault();
-        setVirtualAddress(settingsVa, true);
+        setVirtualAddress(settingsVa, false); // not owner due to ref counting
     }
 
     /**

diff --git a/src/main/java/com/github/stephengold/joltjni/WheeledVehicleControllerSettings.java b/src/main/java/com/github/stephengold/joltjni/WheeledVehicleControllerSettings.java
@@ -36,7 +36,7 @@ public class WheeledVehicleControllerSettings
      */
     public WheeledVehicleControllerSettings() {
         long settingsVa = createDefault();
-        setVirtualAddress(settingsVa, true);
+        setVirtualAddress(settingsVa, false); // not owner due to ref counting
     }
 
     /**