[Bf-blender-cvs] SVN commit: /data/svn/bf-blender [19868] branches/blender2.5/blender/source /blender: Assorted 2.5 fixes:
Joshua Leung
aligorith at gmail.com
Wed Apr 22 12:07:12 CEST 2009
Revision: 19868
http://projects.blender.org/plugins/scmsvn/viewcvs.php?view=rev&root=bf-blender&revision=19868
Author: aligorith
Date: 2009-04-22 12:07:12 +0200 (Wed, 22 Apr 2009)
Log Message:
-----------
Assorted 2.5 fixes:
- compiling errors in outliner.c
- incorrect comments
Modified Paths:
--------------
branches/blender2.5/blender/source/blender/blenkernel/intern/fcurve.c
branches/blender2.5/blender/source/blender/editors/space_outliner/outliner.c
branches/blender2.5/blender/source/blender/python/intern/bpy_interface.c
Modified: branches/blender2.5/blender/source/blender/blenkernel/intern/fcurve.c
===================================================================
--- branches/blender2.5/blender/source/blender/blenkernel/intern/fcurve.c 2009-04-22 09:47:57 UTC (rev 19867)
+++ branches/blender2.5/blender/source/blender/blenkernel/intern/fcurve.c 2009-04-22 10:07:12 UTC (rev 19868)
@@ -1258,8 +1258,11 @@
/* Utilities For F-Curve Modifiers ---------------------- */
-/* Recalculate the F-Curve at evaltime, as modified by the given F-Curve */
-// TODO: this isn't really such an elegant solution for time-modifying F-Modifiers, but it gets too difficult otherwise for now...
+/* Recalculate the F-Curve at evaltime, as modified by the given F-Curve
+ *
+ * While this may sound wrong (and be potentially very slow), it is invalid for F-Curve modifiers to actually
+ * modify the evaltime in such a way that those after it will end up evaluating in the wrong time space.
+ */
static float fcm_reevaluate_fcurve (FCurve *fcu, FModifier *fcm, float cvalue, float evaltime)
{
ListBase modifiers = {NULL, NULL};
Modified: branches/blender2.5/blender/source/blender/editors/space_outliner/outliner.c
===================================================================
--- branches/blender2.5/blender/source/blender/editors/space_outliner/outliner.c 2009-04-22 09:47:57 UTC (rev 19867)
+++ branches/blender2.5/blender/source/blender/editors/space_outliner/outliner.c 2009-04-22 10:07:12 UTC (rev 19868)
@@ -100,6 +100,7 @@
#include "RNA_access.h"
#include "ED_armature.h"
+#include "ED_keyframing.h"
#include "ED_object.h"
#include "ED_screen.h"
@@ -3270,7 +3271,6 @@
static int outliner_drivers_addsel_exec(bContext *C, wmOperator *op)
{
SpaceOops *soutliner= (SpaceOops*)CTX_wm_space_data(C);
- Scene *scene= CTX_data_scene(C);
/* check for invalid states */
if (soutliner == NULL)
@@ -3306,7 +3306,6 @@
static int outliner_drivers_deletesel_exec(bContext *C, wmOperator *op)
{
SpaceOops *soutliner= (SpaceOops*)CTX_wm_space_data(C);
- Scene *scene= CTX_data_scene(C);
/* check for invalid states */
if (soutliner == NULL)
Modified: branches/blender2.5/blender/source/blender/python/intern/bpy_interface.c
===================================================================
--- branches/blender2.5/blender/source/blender/python/intern/bpy_interface.c 2009-04-22 09:47:57 UTC (rev 19867)
+++ branches/blender2.5/blender/source/blender/python/intern/bpy_interface.c 2009-04-22 10:07:12 UTC (rev 19868)
@@ -560,7 +560,7 @@
}
}
- /* add target values to a py dictionary that we add to the drivers dict as 'd' */
+ /* add target values to a dict that will be used as '__locals__' dict */
driver_vars = PyDict_New(); // XXX do we need to decref this?
for (dtar= driver->targets.first; dtar; dtar= dtar->next) {
PyObject *driver_arg = NULL;
@@ -569,11 +569,10 @@
/* try to get variable value */
tval= driver_get_target_value(driver, dtar);
driver_arg= PyFloat_FromDouble((double)tval);
- if (driver_arg == NULL) continue;
/* try to add to dictionary */
if (PyDict_SetItemString(driver_vars, dtar->name, driver_arg)) {
- /* this target failed */
+ /* this target failed - bad name */
if (targets_ok) {
/* first one - print some extra info for easier identification */
fprintf(stderr, "\nBPY_pydriver_eval() - Error while evaluating PyDriver:\n");
More information about the Bf-blender-cvs
mailing list