[csharp] Port of commit 853b5ee. Fix NaN in IKConstraint due to fp pr…

…ecision. dd - r * r might result in a negative value, even if dd == r * r, depending on compiler optimizations and operation order.
EsotericSoftware · Aug 6, 2024 · b17595d · b17595d
1 parent 853b5ee
commit b17595d
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/spine-csharp/src/IkConstraint.cs b/spine-csharp/src/IkConstraint.cs
@@ -328,7 +328,7 @@ static public void Apply (Bone parent, Bone child, float targetX, float targetY,
 					q = -(c1 + q) * 0.5f;
 					float r0 = q / c2, r1 = c / q;
 					float r = Math.Abs(r0) < Math.Abs(r1) ? r0 : r1;
-					if (r * r <= dd) {
+					if (dd - r * r >= 0) {
 						y = (float)Math.Sqrt(dd - r * r) * bendDir;
 						a1 = ta - (float)Math.Atan2(y, r);
 						a2 = (float)Math.Atan2(y / psy, (r - l1) / psx);

diff --git a/spine-csharp/src/package.json b/spine-csharp/src/package.json
@@ -2,7 +2,7 @@
 	"name": "com.esotericsoftware.spine.spine-csharp",
 	"displayName": "spine-csharp Runtime",
 	"description": "This plugin provides the spine-csharp core runtime.",
-	"version": "4.2.28",
+	"version": "4.2.29",
 	"unity": "2018.3",
 	"author": {
 		"name": "Esoteric Software",