archive/blender-archive
Archived
1
1
Fork 0
Code Pull Requests Packages Activity

Deleting keyframes usability tweak:

Browse Source 
There won't be dangling "empty" actions left behind anymore in the
DopeSheet channel list after you've deleted all their keyframes (and
don't want to add keyframes to them anymore).

Of course, this poses problems with more actions getting created if
you then go and keyframe those objects again. If this does turn out to
be an equally bad problem, then another approach from the channel
filtering code side (probably aided by the restructed code) will help
(though doesn't solve the problem where people complain of having
heaps of "empty" actions dangling from objects they no longer want
animated).
This commit is contained in:
Joshua Leung
2011-06-02 12:21:55 +00:00
parent 072d350aab
commit 02c6cb8039
2 changed files with 29 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

32
source/blender/editors/animation/anim_channels_edit.c
View File

@@ -515,12 +515,34 @@ void ANIM_fcurve_delete_from_animdata (bAnimContext *ac, AnimData *adt, FCurve *
* - Drivers
* - TODO... some others?
*/
if (fcu->grp)
action_groups_remove_channel(adt->action, fcu);
else if ((ac) && (ac->datatype == ANIMCONT_DRIVERS))
if ((ac) && (ac->datatype == ANIMCONT_DRIVERS)) {
/* driver F-Curve */
BLI_remlink(&adt->drivers, fcu);
else if (adt->action)
BLI_remlink(&adt->action->curves, fcu);
}
else if (adt->action) {
/* remove from group or action, whichever one "owns" the F-Curve */
if (fcu->grp)
action_groups_remove_channel(adt->action, fcu);
else
BLI_remlink(&adt->action->curves, fcu);
/* if action has no more F-Curves as a result of this, unlink it from
* AnimData if it did not come from a NLA Strip being tweaked.
*
* This is done so that we don't have dangling Object+Action entries in
* channel list that are empty, and linger around long after the data they
* are for has disappeared (and probably won't come back).
*/
// XXX: does everybody always want this?
/* XXX: there's a problem where many actions could build up in the file if multiple
* full add/delete cycles are performed on the same objects, but assume that this is rare
*/
if ((adt->action->curves.first == NULL) && (adt->flag & ADT_NLA_EDIT_ON)==0)
{
id_us_min(&adt->action->id);
adt->action = NULL;
}
}
/* free the F-Curve itself */
free_fcurve(fcu);

4
source/blender/editors/animation/keyframes_general.c
View File

@@ -111,7 +111,7 @@ void delete_fcurve_keys(FCurve *fcu)
{
int i;
if(fcu->bezt==NULL) /* ignore baked curves */
if (fcu->bezt==NULL) /* ignore baked curves */
return;
/* Delete selected BezTriples */
@@ -124,7 +124,7 @@ void delete_fcurve_keys(FCurve *fcu)
}
/* Free the array of BezTriples if there are not keyframes */
if(fcu->totvert == 0)
if (fcu->totvert == 0)
clear_fcurve_keys(fcu);
}