diff options
author | Campbell Barton <ideasman42@gmail.com> | 2012-08-27 12:23:51 +0400 |
---|---|---|
committer | Campbell Barton <ideasman42@gmail.com> | 2012-08-27 12:23:51 +0400 |
commit | 5d60dabef468a79c7d253f53dc19b89a0f36fe80 (patch) | |
tree | 300c6bbfa76f008d4ff04258809eb5611e5f0fac /source/blender/editors/animation | |
parent | ae6907a065da928043da35221ca879f1d58ecf0b (diff) |
remove incorrect comment, add in useful one.
Diffstat (limited to 'source/blender/editors/animation')
-rw-r--r-- | source/blender/editors/animation/keyframes_draw.c | 10 |
1 files changed, 7 insertions, 3 deletions
diff --git a/source/blender/editors/animation/keyframes_draw.c b/source/blender/editors/animation/keyframes_draw.c index 98ab6dc99a6..46537674e59 100644 --- a/source/blender/editors/animation/keyframes_draw.c +++ b/source/blender/editors/animation/keyframes_draw.c @@ -852,12 +852,17 @@ void summary_to_keylist(bAnimContext *ac, DLRBT_Tree *keys, DLRBT_Tree *blocks) int filter; /* get F-Curves to take keyframes from */ - filter = ANIMFILTER_DATA_VISIBLE; // curves only + filter = ANIMFILTER_DATA_VISIBLE; ANIM_animdata_filter(ac, &anim_data, filter, ac->data, ac->datatype); /* loop through each F-Curve, grabbing the keyframes */ for (ale = anim_data.first; ale; ale = ale->next) { + /* Why not use all #eAnim_KeyType here? + * All of the other key types are actually "summaries" themselves, and will just end up duplicating stuff + * that comes up through standard filtering of just F-Curves. + * Given the way that these work, there isn't really any benefit at all from including them. - Aligorith */ + switch (ale->datatype) { case ALE_FCURVE: fcurve_to_keylist(ale->adt, ale->data, keys, blocks); @@ -869,9 +874,8 @@ void summary_to_keylist(bAnimContext *ac, DLRBT_Tree *keys, DLRBT_Tree *blocks) gpl_to_keylist(ac->ads, ale->data, keys); break; default: - printf("%s: datatype %d unhandled\n", __func__, ale->datatype); + // printf("%s: datatype %d unhandled\n", __func__, ale->datatype); break; - } } |