[Bf-blender-cvs] [f209529] master: Fix T47910: Knife project fails

Campbell Barton noreply at git.blender.org
Thu Mar 24 08:29:13 CET 2016


Commit: f209529c3ddedf03199072c72dbcebfaa1ac52cc
Author: Campbell Barton
Date:   Thu Mar 24 18:27:39 2016 +1100
Branches: master
https://developer.blender.org/rBf209529c3ddedf03199072c72dbcebfaa1ac52cc

Fix T47910: Knife project fails

Regression in fix for T43896, using screen-space precision here is very problematic,
using lower precision here works for both reports.

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

M	source/blender/editors/mesh/editmesh_knife.c

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

diff --git a/source/blender/editors/mesh/editmesh_knife.c b/source/blender/editors/mesh/editmesh_knife.c
index b606eb0..59967ff 100644
--- a/source/blender/editors/mesh/editmesh_knife.c
+++ b/source/blender/editors/mesh/editmesh_knife.c
@@ -1625,7 +1625,8 @@ static void knife_find_line_hits(KnifeTool_OpData *kcd)
 		face_tol = KNIFE_FLT_EPS_PX_FACE;
 	}
 	else {
-		vert_tol = line_tol = face_tol = 0.001f;
+		/* use 1/100th of a pixel, see T43896 (too big), T47910 (too small). */
+		vert_tol = line_tol = face_tol = 0.01f;
 	}
 
 	vert_tol_sq = vert_tol * vert_tol;




More information about the Bf-blender-cvs mailing list