- 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
I think this has quadratic performance in the general case. If you imagine all the keys except one getting deleted, then this call to BKE_fcurve_delete_key()
shifts every remaining element in the array over by one for each visited element.
Gah! Sorry, I didn't refresh the page between Bastien's approval and merging, and missed your comment. Thanks for letting me know. I'll fix it.
action.frame_range
span always >=1 when there's only 1 key frame in it.
If they represent a half-open interval [start, end) then the current behaviour is actually correct, as it just means "from 6 up to but not including frame 7". If they represent a closed interval…