diff options
author | Joshua Leung <aligorith@gmail.com> | 2011-08-28 15:39:18 +0400 |
---|---|---|
committer | Joshua Leung <aligorith@gmail.com> | 2011-08-28 15:39:18 +0400 |
commit | 8e12b7b054c3c4e95a23f26db232d99ff18e2b90 (patch) | |
tree | ff37993677c52a010cb29113a37e146065583914 /source/blender/blenkernel/intern/depsgraph.c | |
parent | 94b3e83b6c564a66788624e824a2548d6c8b8650 (diff) |
Assorted comment clarification in response to code review notes
Diffstat (limited to 'source/blender/blenkernel/intern/depsgraph.c')
-rw-r--r-- | source/blender/blenkernel/intern/depsgraph.c | 13 |
1 files changed, 12 insertions, 1 deletions
diff --git a/source/blender/blenkernel/intern/depsgraph.c b/source/blender/blenkernel/intern/depsgraph.c index 4802601307a..6f27a104144 100644 --- a/source/blender/blenkernel/intern/depsgraph.c +++ b/source/blender/blenkernel/intern/depsgraph.c @@ -2062,8 +2062,19 @@ static short animdata_use_time(AnimData *adt) return 1; } - /* experimental check: if we have drivers, more likely than not, on a frame change + /* If we have drivers, more likely than not, on a frame change * they'll need updating because their owner changed + * + * This is kindof a hack to get around a whole host of problems + * involving drivers using non-object datablock data (which the + * depsgraph currently has no way of representing let alone correctly + * dependency sort+tagging). By doing this, at least we ensure that + * some commonly attempted drivers (such as scene -> current frame; + * see "Driver updates fail" thread on Bf-committers dated July 2) + * will work correctly, and that other non-object datablocks will have + * their drivers update at least on frame change. + * + * -- Aligorith, July 4 2011 */ if (adt->drivers.first) return 1; |