blender
/
blender
129
398
Fork 572
Code Issues 6.2k Pull Requests 746 Projects 19 Wiki Activity

Cleanup: Typo, improve variable names 

`accumulate_counts_to_offsets` wasn't just used for the point domain.
This commit is contained in:
Hans Goudey 2022-03-23 23:50:10 -05:00
parent 7ef3a1a6e6
commit 6d61cf4e80
2 changed files with 6 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
source/blender/blenkernel/BKE_attribute_math.hh
View File

@ -22,7 +22,7 @@ inline void convert_to_static_type(const CPPType &cpp_type, const Func &func)
[&](auto type_tag) {
using T = typename decltype(type_tag)::type;
if constexpr (std::is_same_v<T, void>) {
/* It's expected that the given cpp type is one of the supported once. */
/* It's expected that the given cpp type is one of the supported ones. */
BLI_assert_unreachable();
}
else {

10
source/blender/nodes/geometry/nodes/node_geo_duplicate_elements.cc
View File

@ -83,12 +83,12 @@ static Array<int> accumulate_counts_to_offsets(const IndexMask selection,
const VArray<int> &counts)
{
Array<int> offsets(selection.size() + 1);
int dst_points_size = 0;
for (const int i_point : selection.index_range()) {
offsets[i_point] = dst_points_size;
dst_points_size += std::max(counts[selection[i_point]], 0);
int total = 0;
for (const int i : selection.index_range()) {
offsets[i] = total;
total += std::max(counts[selection[i]], 0);
}
offsets.last() = dst_points_size;
offsets.last() = total;
return offsets;
}