Welcome to mirror list, hosted at ThFree Co, Russian Federation.

git.blender.org/blender.git - Unnamed repository; edit this file 'description' to name the repository.
summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorHoward Trickey <howard.trickey@gmail.com>2022-11-13 22:09:27 +0300
committerHoward Trickey <howard.trickey@gmail.com>2022-11-13 22:09:27 +0300
commitd17f5bcd8f26d32b26bc945658e6d308316cd3fe (patch)
tree9ccc74579d2cd5add0857ddeaadef1e05080735c
parent7419e291e8b81b9dd562a05909bbf30cdd77077e (diff)
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.
-rw-r--r--source/blender/bmesh/tools/bmesh_bevel.c7
1 files changed, 5 insertions, 2 deletions
diff --git a/source/blender/bmesh/tools/bmesh_bevel.c b/source/blender/bmesh/tools/bmesh_bevel.c
index c45f9dbe49c..fbc47505924 100644
--- a/source/blender/bmesh/tools/bmesh_bevel.c
+++ b/source/blender/bmesh/tools/bmesh_bevel.c
@@ -1502,8 +1502,11 @@ 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),