reverse_point_data
and reverse_points_of
are only called with IndexRange
. Using IndexMask
arguments here is unnecessary and confusing, since the indices would have to be offset by the firstr/last point.
Can just store a IndexRange
local variable and change it directly, no need to split into begin/size.
The entire conversion function is so convoluted, I believe it was partly due to not using static/local functions in the beginning and it grew this way. Maybe cleaning this up would also become a…
After taking a closer look at Bullet's damping implementation i'm pretty sure @Nektarios-Kalogridis is correct that this is a bug. Unfortunately it's a Bullet bug rather than a Blender bug. We…
I agree with @Sean-Kim, we should avoid making object_convert_exec
recursive, especially since it requires overriding the operator properties while the operator is being executed. It should be…