remove incorrect comment, add in useful one.

This commit is contained in:
Campbell Barton
2012-08-27 08:23:51 +00:00
parent ae6907a065
commit 5d60dabef4

View File

@@ -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;
}
}