[Bf-blender-cvs] [6478f90cc23] master: curve_fit_nd: fix assert caused by too small epsilon

Campbell Barton noreply at git.blender.org
Thu Apr 11 17:07:15 CEST 2019


Commit: 6478f90cc2301ac86ed0fae19f626fd67ebf00cc
Author: Campbell Barton
Date:   Fri Apr 12 00:58:15 2019 +1000
Branches: master
https://developer.blender.org/rB6478f90cc2301ac86ed0fae19f626fd67ebf00cc

curve_fit_nd: fix assert caused by too small epsilon

Curve decimate would assert in some cases.

===================================================================

M	extern/curve_fit_nd/intern/curve_fit_cubic.c

===================================================================

diff --git a/extern/curve_fit_nd/intern/curve_fit_cubic.c b/extern/curve_fit_nd/intern/curve_fit_cubic.c
index ed855d34b08..0005cbe5a93 100644
--- a/extern/curve_fit_nd/intern/curve_fit_cubic.c
+++ b/extern/curve_fit_nd/intern/curve_fit_cubic.c
@@ -462,7 +462,7 @@ static double points_calc_circumference_factor(
 		 * We could try support this but will likely cause extreme >1 scales which could cause other issues. */
 		// assert(angle >= len_tangent);
 		double factor = (angle / len_tangent);
-		assert(factor < (M_PI / 2) + (DBL_EPSILON * 10));
+		assert(factor < (M_PI / 2) + 1e-6);
 		return factor;
 	}
 	else {



More information about the Bf-blender-cvs mailing list