[Bf-blender-cvs] [74a5fb734ab] master: Fix: Spline parameter node broken for Catmull Rom curves

Hans Goudey noreply at git.blender.org
Wed May 11 11:33:53 CEST 2022


Commit: 74a5fb734ab5ddfa1c4dfbc50e5daf2190ad40a1
Author: Hans Goudey
Date:   Wed May 11 11:33:47 2022 +0200
Branches: master
https://developer.blender.org/rB74a5fb734ab5ddfa1c4dfbc50e5daf2190ad40a1

Fix: Spline parameter node broken for Catmull Rom curves

Subtracting one from the evaluated index could make the index -1.
That was only necessary for Bezier curves due to the specifics of
the "bezier_evaluated_offsets".

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

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

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

diff --git a/source/blender/nodes/geometry/nodes/node_geo_curve_spline_parameter.cc b/source/blender/nodes/geometry/nodes/node_geo_curve_spline_parameter.cc
index a548becf24e..ae36248b573 100644
--- a/source/blender/nodes/geometry/nodes/node_geo_curve_spline_parameter.cc
+++ b/source/blender/nodes/geometry/nodes/node_geo_curve_spline_parameter.cc
@@ -75,7 +75,7 @@ static Array<float> curve_length_point_domain(const bke::CurvesGeometry &curves)
         case CURVE_TYPE_CATMULL_ROM: {
           const int resolution = resolutions[i_curve];
           for (const int i : IndexRange(points.size()).drop_back(1)) {
-            lengths[i + 1] = evaluated_lengths[resolution * i - 1];
+            lengths[i + 1] = evaluated_lengths[resolution * i];
           }
           break;
         }



More information about the Bf-blender-cvs mailing list