[Bf-blender-cvs] [521d4190a09] master: Fix T96146: Subdivide curve node uninitialized dangling handles

Hans Goudey noreply at git.blender.org
Tue Mar 8 22:45:59 CET 2022


Commit: 521d4190a09b1117f4f729991a0c636be4c2b8f3
Author: Hans Goudey
Date:   Tue Mar 8 15:45:17 2022 -0600
Branches: master
https://developer.blender.org/rB521d4190a09b1117f4f729991a0c636be4c2b8f3

Fix T96146: Subdivide curve node uninitialized dangling handles

Handles of non-cyclic bezier curves were not initialized.
Now properly copy the dangling handle positions and types
from the source curve.

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

M	source/blender/nodes/geometry/nodes/node_geo_curve_subdivide.cc

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

diff --git a/source/blender/nodes/geometry/nodes/node_geo_curve_subdivide.cc b/source/blender/nodes/geometry/nodes/node_geo_curve_subdivide.cc
index 6456af5f295..371556c04f1 100644
--- a/source/blender/nodes/geometry/nodes/node_geo_curve_subdivide.cc
+++ b/source/blender/nodes/geometry/nodes/node_geo_curve_subdivide.cc
@@ -206,6 +206,10 @@ static void subdivide_bezier_spline(const BezierSpline &src,
   }
   else {
     dst_positions.last() = src_positions.last();
+    dst_type_left.first() = src.handle_types_left().first();
+    dst_type_right.last() = src.handle_types_right().last();
+    dst_handles_left.first() = src_handles_left.first();
+    dst_handles_right.last() = src_handles_right.last();
   }
 }



More information about the Bf-blender-cvs mailing list