- Amsterdam, Netherlands
- https://cessen.com
-
Animator, rigger, and software developer. Currently working at the Blender Institute as a developer on Blender's animation system.
Been using Blender since 1998, and worked on Big Buck Bunny and Sintel (two of Blender's open movie projects).
- Joined on
2003-03-21
My first reaction was that this call isn't right for layered actions. That function is supposed to be specific to legacy actions. But in practice it doesn't hurt anything right now, because it will just fail to find the fcurve in the legacy action fcurves and skip the legacy-specific stuff anyway.
Not for this PR, but just noting:
Looking at both this and #124598, it appears to me that we have a similar situation with keyframe deletion as we did (and still do in many areas) with keyframe…
I think that's still the case here: every call to fcurve_find
just does a linear scan of fcurves
.
However, the number of array property elements getting keyed can (I think?) effectively be…