- 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 -
-
Discussed in person, and we're leaving the return type as bool for now.
Discussed in person, and we're leaving the return type of fcurve_assign_to_channel_group() as bool for now.
As discussed in person, I've gone ahead and made the wrapper class in this PR.
For now I've just made this a free function, since it was a quicker change and still puts the functionality in one place. I've left a TODO to move these to a wrapper class in the future. I can do that immediately in a separate PR, or we can wait so I can get back to the strip data storage PR.
Rather than making a method that calls the other two, I decided to just merge them into one. The rationale is that at least at the moment they shouldn't be called separately anyway, and having them available as separate functions gives the impression that that's appropriate. Additionally, they're trivial to split out again if needed in the future.