[Bf-blender-cvs] [4e73f60a15b] gpencil-new-data-proposal: Tests: Avoid unnecessary work when transforming curve points

Hans Goudey noreply at git.blender.org
Mon Dec 12 20:35:48 CET 2022


Commit: 4e73f60a15b4c4b83567799e5aee88c5da4bd2a5
Author: Hans Goudey
Date:   Mon Dec 12 13:22:48 2022 -0600
Branches: gpencil-new-data-proposal
https://developer.blender.org/rB4e73f60a15b4c4b83567799e5aee88c5da4bd2a5

Tests: Avoid unnecessary work when transforming curve points

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

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 3b1c56baa01..c3bc8347693 100644
--- a/source/blender/blenkernel/intern/gpencil_new_proposal_test.cc
+++ b/source/blender/blenkernel/intern/gpencil_new_proposal_test.cc
@@ -514,13 +514,8 @@ TEST(gpencil_proposal, TimeMultiFrameTransformStrokes)
     SCOPED_TIMER("TimeMultiFrameTransformStrokes");
     for (const int i : data.frames_on_active_layer()) {
       GPFrame &gpf = data.frames_for_write(i);
-      Vector<GPStroke> gpf_strokes = gpf.strokes_for_write();
-      MutableSpan<GPStroke> strokes_span = gpf_strokes.as_mutable_span();
-      threading::parallel_for(strokes_span.index_range(), 256, [&](const IndexRange range) {
-        for (GPStroke &stroke : strokes_span.slice(range)) {
-          stroke.transform(translate_mat);
-        }
-      });
+      CurvesGeometry &curves = gpf.strokes_as_curves();
+      curves.transform(translate_mat);
     }
   }



More information about the Bf-blender-cvs mailing list