[Bf-blender-cvs] [4f920371580] master: Fix T61536: can't snap vertex to another vertex in edit mode using curves

Sebastian Parborg noreply at git.blender.org
Thu Feb 14 14:54:37 CET 2019


Commit: 4f920371580b07b9b5cbaeef2f3edb46592bd155
Author: Sebastian Parborg
Date:   Thu Feb 14 14:49:47 2019 +0100
Branches: master
https://developer.blender.org/rB4f920371580b07b9b5cbaeef2f3edb46592bd155

Fix T61536: can't snap vertex to another vertex in edit mode using curves

Previously, the curve self snapping would only snap to points that were
earlier in the curve structure. This was because of a simple coding
snafu of using break when meaning to use continue.

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

M	source/blender/editors/transform/transform_snap_object.c

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

diff --git a/source/blender/editors/transform/transform_snap_object.c b/source/blender/editors/transform/transform_snap_object.c
index 101c0f0356c..9916aa846d6 100644
--- a/source/blender/editors/transform/transform_snap_object.c
+++ b/source/blender/editors/transform/transform_snap_object.c
@@ -1537,7 +1537,7 @@ static short snapCurve(
 					if (nu->bezt) {
 						/* don't snap to selected (moving) or hidden */
 						if (nu->bezt[u].f2 & SELECT || nu->bezt[u].hide != 0) {
-							break;
+							continue;
 						}
 						has_snap |= test_projected_vert_dist(
 						        &neasrest_precalc,
@@ -1567,7 +1567,7 @@ static short snapCurve(
 					else {
 						/* don't snap to selected (moving) or hidden */
 						if (nu->bp[u].f1 & SELECT || nu->bp[u].hide != 0) {
-							break;
+							continue;
 						}
 						has_snap |= test_projected_vert_dist(
 						        &neasrest_precalc,



More information about the Bf-blender-cvs mailing list