- 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
An approach that might(?) help simplify the code, or at least make it easier to follow, is to have a function that (given an ID) returns a priority-ordered list of other IDs to check for actions. …
Reading through the code, one thing that occurred to me is that I don't think this is keying-order invariant. In other words, the code as it is right now will result in different action arrangements (what does and doesn't share an action) depending on the order you key things in.
Can this ever be null? I might be misunderstanding the code, but it looks to me like this could be an assert instead.
Might be worth putting a comment at the top of each case explaining its intent (e.g. "search for an action already assigned to the object's data").
BKE_main_namemap_get_name
not always returning true when modifying given name.
Basically looks good from what I can see. Just a documentation copy-paste error that can be fixed while landing.
Copy-paste error: not "get this slot", but something like "get its slot connection cleared".