blender
/
blender
136
407
Fork 589
Code Issues 6.3k Pull Requests 777 Projects 19 Wiki Activity

Curves: Add simplify_curve_attribute function #118560

Merged
Falk David merged 17 commits from filedescriptor/blender:curves-resample-adaptive into main 2024-03-26 15:28:22 +01:00
Conversation 11 Commits 17 Files Changed 3 +1 -1
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit ad5ea59f47 - Show all commits

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

@ -58,7 +58,7 @@ static void ramer_douglas_peucker(const IndexRange range,
/* Mark all points to be kept. */
points_to_delete.slice(range).fill(false);
filedescriptor marked this conversation as resolved
Hans Goudey commented 2024-03-26 14:01:24 +01:00
Review
Copy Link

Maybe give this a slightly larger inline buffer to avoid allocations for every curve

Maybe give this a slightly larger inline buffer to avoid allocations for every curve
Stack<IndexRange> stack;
Stack<IndexRange, 32> stack;
stack.push(range);
while (!stack.is_empty()) {
const IndexRange sub_range = stack.pop();