blender
/
blender
136
404
Fork 586
Code Issues 6.2k Pull Requests 754 Projects 19 Wiki Activity

Fix: Assert in subdivide curves node after span slicing change 

a5e7657cee missed this call where clamped slicing is necessary.
The subdivision of a segment purposefully modifies the handle types of
the other side of the following control point, but that didn't work for
the final cyclic segment.
This commit is contained in:
Hans Goudey 2022-11-30 21:21:44 -06:00
parent 4aac5b92c1
commit 3cebc58936
2 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
release/scripts/addons

@ -1 +1 @@
Subproject commit 0b0052bd53ad8249ed07dfb87705c338af698bde
Subproject commit fdfd24de034d4bba4fb67731d0aae81dc4940239

2
source/blender/geometry/intern/subdivide_curves.cc
View File

@ -172,7 +172,7 @@ static void subdivide_bezier_segment(const float3 &position_prev,
auto fill_segment_handle_types = [&](const HandleType type) {
/* Also change the left handle of the control point following the segment's points. And don't
* change the left handle of the first point, since that is part of the previous segment. */
dst_types_l.slice(segment_points.shift(1)).fill(type);
dst_types_l.slice_safe(segment_points.shift(1)).fill(type);
dst_types_r.slice(segment_points).fill(type);
};