[Bf-blender-cvs] [a704a66914f] master: Adding some verbose documentation

Joshua Leung noreply at git.blender.org
Tue Sep 12 14:55:28 CEST 2017


Commit: a704a66914faa7e82bb5403f898d4a2c11f678a9
Author: Joshua Leung
Date:   Mon Sep 11 23:42:31 2017 +1200
Branches: master
https://developer.blender.org/rBa704a66914faa7e82bb5403f898d4a2c11f678a9

Adding some verbose documentation

I thought this was already clear enough, but apparently not.

===================================================================

M	source/blender/editors/include/ED_anim_api.h

===================================================================

diff --git a/source/blender/editors/include/ED_anim_api.h b/source/blender/editors/include/ED_anim_api.h
index 4668d5d3d50..68a5dd8a94e 100644
--- a/source/blender/editors/include/ED_anim_api.h
+++ b/source/blender/editors/include/ED_anim_api.h
@@ -124,6 +124,15 @@ typedef struct bAnimListElem {
 	void   *key_data;       /* motion data - mostly F-Curves, but can be other types too */
 	
 	
+	/* NOTE: id here is the "IdAdtTemplate"-style datablock (e.g. Object, Material, Texture, NodeTree)
+	 *       from which evaluation of the RNA-paths takes place. It's used to figure out how deep
+	 *       channels should be nested (e.g. for Textures/NodeTrees) in the tree, and allows property
+	 *       lookups (e.g. for sliders and for inserting keyframes) to work. If we had instead used
+	 *       bAction or something similar, none of this would be possible: although it's trivial
+	 *       to use an IdAdtTemplate type to find the source action a channel (e.g. F-Curve) comes from
+	 *       (i.e. in the AnimEditors, it *must* be the active action, as only that can be edited),
+	 *       it's impossible to go the other way (i.e. one action may be used in multiple places).
+	 */
 	struct ID *id;          /* ID block that channel is attached to */
 	struct AnimData *adt;   /* source of the animation data attached to ID block (for convenience) */



More information about the Bf-blender-cvs mailing list