[Bf-blender-cvs] [7e55ff15b09] master: Fix: Incorrectly sized curves created in set conversion node

[Hans Goudey]

Commit: 7e55ff15b094079d8052c353fcc8aa7e1f6422c5
Author: Hans Goudey
Date:   Thu Jun 30 21:31:11 2022 -0500
Branches: master
https://developer.blender.org/rB7e55ff15b094079d8052c353fcc8aa7e1f6422c5

Fix: Incorrectly sized curves created in set conversion node

The number of points in the source curve was needed, but the offset
(just zero) was passed instead. It's unclear how this worked before.
A mistake in the recent commit 9e393fc2f12583d32dd.

Also use a common utility for retrieving the sizes of curves
in ranges instead of reimplementing it for this file.

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

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

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

diff --git a/source/blender/nodes/geometry/nodes/node_geo_curve_spline_type.cc b/source/blender/nodes/geometry/nodes/node_geo_curve_spline_type.cc
index 8479d61a890..f846a6be53b 100644
--- a/source/blender/nodes/geometry/nodes/node_geo_curve_spline_type.cc
+++ b/source/blender/nodes/geometry/nodes/node_geo_curve_spline_type.cc
@@ -303,15 +303,6 @@ static int to_nurbs_size(const CurveType src_type, const int src_size)
   }
 }
 
-static void retrieve_curve_sizes(const bke::CurvesGeometry &curves, MutableSpan<int> sizes)
-{
-  threading::parallel_for(curves.curves_range(), 4096, [&](IndexRange range) {
-    for (const int i : range) {
-      sizes[i] = curves.points_for_curve(i).size();
-    }
-  });
-}
-
 struct GenericAttributes : NonCopyable, NonMovable {
   Vector<GSpan> src;
   Vector<GMutableSpan> dst;
@@ -354,17 +345,22 @@ static void convert_to_bezier(const CurveComponent &src_component,
                               CurveComponent &dst_component,
                               bke::CurvesGeometry &dst_curves)
 {
+  const Vector<IndexRange> unselected_ranges = selection.extract_ranges_invert(
+      src_curves.curves_range());
+
   const VArray<int8_t> src_knot_modes = src_curves.nurbs_knots_modes();
   const VArray<int8_t> src_types = src_curves.curve_types();
   const VArray<bool> src_cyclic = src_curves.cyclic();
   const Span<float3> src_positions = src_curves.positions();
 
   MutableSpan<int> dst_offsets = dst_curves.offsets_for_write();
-  retrieve_curve_sizes(src_curves, dst_curves.offsets_for_write());
+  bke::curves::fill_curve_counts(src_curves, unselected_ranges, dst_curves.offsets_for_write());
   threading::parallel_for(selection.index_range(), 1024, [&](IndexRange range) {
     for (const int i : selection.slice(range)) {
-      dst_offsets[i] = to_bezier_size(
-          CurveType(src_types[i]), src_cyclic[i], KnotsMode(src_knot_modes[i]), dst_offsets[i]);
+      const CurveType type = CurveType(src_types[i]);
+      const KnotsMode knots_mode = KnotsMode(src_knot_modes[i]);
+      const IndexRange points = src_curves.points_for_curve(i);
+      dst_offsets[i] = to_bezier_size(type, src_cyclic[i], knots_mode, points.size());
     }
   });
   bke::curves::accumulate_counts_to_offsets(dst_offsets);
@@ -489,9 +485,6 @@ static void convert_to_bezier(const CurveComponent &src_component,
                                      bezier_to_bezier,
                                      nurbs_to_bezier);
 
-  const Vector<IndexRange> unselected_ranges = selection.extract_ranges_invert(
-      src_curves.curves_range());
-
   for (const int i : attributes.src.index_range()) {
     bke::curves::copy_point_data(
         src_curves, dst_curves, unselected_ranges, attributes.src[i], attributes.dst[i]);
@@ -508,15 +501,19 @@ static void convert_to_nurbs(const CurveComponent &src_component,
                              CurveComponent &dst_component,
                              bke::CurvesGeometry &dst_curves)
 {
+  const Vector<IndexRange> unselected_ranges = selection.extract_ranges_invert(
+      src_curves.curves_range());
+
   const VArray<int8_t> src_types = src_curves.curve_types();
   const VArray<bool> src_cyclic = src_curves.cyclic();
   const Span<float3> src_positions = src_curves.positions();
 
   MutableSpan<int> dst_offsets = dst_curves.offsets_for_write();
-  retrieve_curve_sizes(src_curves, dst_curves.offsets_for_write());
+  bke::curves::fill_curve_counts(src_curves, unselected_ranges, dst_curves.offsets_for_write());
   threading::parallel_for(selection.index_range(), 1024, [&](IndexRange range) {
     for (const int i : selection.slice(range)) {
-      dst_offsets[i] = to_nurbs_size(CurveType(src_types[i]), dst_offsets[i]);
+      const IndexRange points = src_curves.points_for_curve(i);
+      dst_offsets[i] = to_nurbs_size(CurveType(src_types[i]), points.size());
     }
   });
   bke::curves::accumulate_counts_to_offsets(dst_offsets);
@@ -649,9 +646,6 @@ static void convert_to_nurbs(const CurveComponent &src_component,
                                      bezier_to_nurbs,
                                      nurbs_to_nurbs);
 
-  const Vector<IndexRange> unselected_ranges = selection.extract_ranges_invert(
-      src_curves.curves_range());
-
   for (const int i : attributes.src.index_range()) {
     bke::curves::copy_point_data(
         src_curves, dst_curves, unselected_ranges, attributes.src[i], attributes.dst[i]);