[Bf-blender-cvs] SVN commit: /data/svn/bf-blender [41511] trunk/blender/source/blender/ blenkernel/intern/anim.c: Bugfix [#28515] Dupliframes start/ end animation broken since 2.56A

[Joshua Leung]

Revision: 41511
          http://projects.blender.org/scm/viewvc.php?view=rev&root=bf-blender&revision=41511
Author:   aligorith
Date:     2011-11-04 08:06:46 +0000 (Fri, 04 Nov 2011)
Log Message:
-----------
Bugfix [#28515] Dupliframes start/end animation broken since 2.56A

Due to changes with the way that dupliframes were being evaluated (one
of those being that the animation for the object being animated would
get reevaluated for every frame sample for dupliframes) to get them
working properly again sometime during the 2.5 series, the dupliframe-
sampling loop would be prematurely terminated on the first frame in
this example.

That is because after the first step of the loop, the ob->dupend
property would have changed its value as it was being animated,
leading to the loop terminating, and only the a single dupli getting
added.

Modified Paths:
--------------
    trunk/blender/source/blender/blenkernel/intern/anim.c

Modified: trunk/blender/source/blender/blenkernel/intern/anim.c
===================================================================
--- trunk/blender/source/blender/blenkernel/intern/anim.c	2011-11-04 04:36:14 UTC (rev 41510)
+++ trunk/blender/source/blender/blenkernel/intern/anim.c	2011-11-04 08:06:46 UTC (rev 41511)
@@ -753,6 +753,7 @@
 	extern int enable_cu_speed;	/* object.c */
 	Object copyob = {{NULL}};
 	int cfrao = scene->r.cfra;
+	int dupend = ob->dupend;
 	
 	/* simple prevention of too deep nested groups */
 	if (level > MAX_DUPLI_RECUR) return;
@@ -773,7 +774,7 @@
 	/* duplicate over the required range */
 	if (ob->transflag & OB_DUPLINOSPEED) enable_cu_speed= 0;
 	
-	for (scene->r.cfra= ob->dupsta; scene->r.cfra<=ob->dupend; scene->r.cfra++) {
+	for (scene->r.cfra= ob->dupsta; scene->r.cfra<=dupend; scene->r.cfra++) {
 		short ok= 1;
 		
 		/* - dupoff = how often a frames within the range shouldn't be made into duplis