[Bf-blender-cvs] [153644cefff] gpencil-new-data-proposal: Use CurvesGeometry directly instead of GPStroke, as an example

[Hans Goudey]

Commit: 153644cefff5ab2853d6aa4cb6ec5751fac2184d
Author: Hans Goudey
Date:   Mon Dec 12 13:35:25 2022 -0600
Branches: gpencil-new-data-proposal
https://developer.blender.org/rB153644cefff5ab2853d6aa4cb6ec5751fac2184d

Use CurvesGeometry directly instead of GPStroke, as an example

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

M	source/blender/blenkernel/intern/gpencil_new_proposal_test.cc

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

diff --git a/source/blender/blenkernel/intern/gpencil_new_proposal_test.cc b/source/blender/blenkernel/intern/gpencil_new_proposal_test.cc
index c3bc8347693..befd94eacd0 100644
--- a/source/blender/blenkernel/intern/gpencil_new_proposal_test.cc
+++ b/source/blender/blenkernel/intern/gpencil_new_proposal_test.cc
@@ -419,14 +419,17 @@ TEST(gpencil_proposal, ChangeStrokePoints)
 
   const int frame_index = data.add_frame_on_layer(layer1_index, 0);
   EXPECT_NE(frame_index, -1);
-  GPStroke stroke = data.frames_for_write(frame_index).add_new_stroke(test_positions.size());
 
-  for (const int i : stroke.points_positions_for_write().index_range()) {
-    stroke.points_positions_for_write()[i] = test_positions[i];
-  }
+  CurvesGeometry &curves = data.frames_for_write(frame_index).strokes_as_curves();
+  curves.resize(curves.points_num() + test_positions.size(), curves.curves_num() + 1);
+  curves.offsets_for_write().last() = curves.offsets().last(1) + test_positions.size();
+
+  const IndexRange new_points = curves.points_for_curve(curves.curves_range().last());
+  MutableSpan<float3> new_positions = curves.positions_for_write().slice(new_points);
+  new_positions.copy_from(test_positions);
 
-  for (const int i : stroke.points_positions().index_range()) {
-    EXPECT_V3_NEAR(stroke.points_positions()[i], test_positions[i], 1e-5f);
+  for (const int i : curves.positions().index_range()) {
+    EXPECT_V3_NEAR(new_positions[i], test_positions[i], 1e-5f);
   }
 }