[Bf-blender-cvs] [c2a68c066b5] blender-v3.4-release: Fix T95335 Bevel operator Loop Slide overshoot.

Howard Trickey noreply at git.blender.org
Wed Nov 16 14:24:10 CET 2022


Commit: c2a68c066b59c87375e3169f50cfd5ac8f18465d
Author: Howard Trickey
Date:   Wed Nov 16 08:23:29 2022 -0500
Branches: blender-v3.4-release
https://developer.blender.org/rBc2a68c066b59c87375e3169f50cfd5ac8f18465d

Fix T95335 Bevel operator Loop Slide overshoot.

If the edge you are going to slide along is very close to in line
with the adjacent beveled edge, then there will be sharp overshoots.
There is an epsilon comparison to just abandon loop slide if this
situation is happening. That epsilon used to be 0.25 radians, but
bug T86768 complained that that value was too high, so it was changed
to .0001 radians (5 millidegrees). Now this current bug shows that
that was too aggressively small, so this change ups it by a factor
of 10, to .001 radians (5 centidegrees). All previous bug reports
remained fixed.

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

M	source/blender/bmesh/tools/bmesh_bevel.c

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

diff --git a/source/blender/bmesh/tools/bmesh_bevel.c b/source/blender/bmesh/tools/bmesh_bevel.c
index c45f9dbe49c..acbdab55444 100644
--- a/source/blender/bmesh/tools/bmesh_bevel.c
+++ b/source/blender/bmesh/tools/bmesh_bevel.c
@@ -1502,8 +1502,10 @@ static void offset_meet(BevelParams *bp,
 }
 
 /* This was changed from 0.25f to fix bug T86768.
- * Original bug T44961 remains fixed with this value. */
-#define BEVEL_GOOD_ANGLE 0.0001f
+ * Original bug T44961 remains fixed with this value.
+ * Update : changed again from 0.0001f to fix bug T95335.*Original two bugs remained fixed.
+ */
+#define BEVEL_GOOD_ANGLE 0.001f
 
 /**
  * Calculate the meeting point between e1 and e2 (one of which should have zero offsets),



More information about the Bf-blender-cvs mailing list