Py-Driver: add 'self' option
[blender.git] / source / blender / makesrna / intern / rna_fcurve.c
index d5fa9ca9a47c48f31ac2c96f363242ecd0f78b5a..08c0f98e45b184e09b2d8f89478ba0b489594e77 100644 (file)
  *  \ingroup RNA
  */
 
-
 #include <stdlib.h>
 
-#include "RNA_access.h"
-#include "RNA_define.h"
-#include "RNA_enum_types.h"
-
-#include "rna_internal.h"
-
 #include "DNA_anim_types.h"
+#include "DNA_curve_types.h"
 #include "DNA_object_types.h"
 #include "DNA_scene_types.h"
 
 
 #include "BKE_action.h"
 
+#include "RNA_access.h"
+#include "RNA_define.h"
+#include "RNA_enum_types.h"
+
+#include "rna_internal.h"
+
 #include "WM_types.h"
 
 #include "ED_keyframing.h"
 #include "ED_keyframes_edit.h"
 
-EnumPropertyItem fmodifier_type_items[] = {
+EnumPropertyItem rna_enum_fmodifier_type_items[] = {
        {FMODIFIER_TYPE_NULL, "NULL", 0, "Invalid", ""},
-       {FMODIFIER_TYPE_GENERATOR, "GENERATOR", 0, "Generator", ""},
-       {FMODIFIER_TYPE_FN_GENERATOR, "FNGENERATOR", 0, "Built-In Function", ""},
-       {FMODIFIER_TYPE_ENVELOPE, "ENVELOPE", 0, "Envelope", ""},
-       {FMODIFIER_TYPE_CYCLES, "CYCLES", 0, "Cycles", ""},
-       {FMODIFIER_TYPE_NOISE, "NOISE", 0, "Noise", ""},
+       {FMODIFIER_TYPE_GENERATOR, "GENERATOR", 0, "Generator",
+                                  "Generate a curve using a factorized or expanded polynomial"},
+       {FMODIFIER_TYPE_FN_GENERATOR, "FNGENERATOR", 0, "Built-In Function",
+                                     "Generate a curve using standard math functions such as sin and cos"},
+       {FMODIFIER_TYPE_ENVELOPE, "ENVELOPE", 0, "Envelope",
+                               "Reshape F-Curve values - e.g. change amplitude of movements"},
+       {FMODIFIER_TYPE_CYCLES, "CYCLES", 0, "Cycles",
+                               "Cyclic extend/repeat keyframe sequence"},
+       {FMODIFIER_TYPE_NOISE, "NOISE", 0, "Noise",
+                              "Add pseudo-random noise on top of F-Curves"},
        /*{FMODIFIER_TYPE_FILTER, "FILTER", 0, "Filter", ""},*/ /* FIXME: not implemented yet! */
-       /*{FMODIFIER_TYPE_PYTHON, "PYTHON", 0, "Python", ""},    *//* FIXME: not implemented yet! */
-       {FMODIFIER_TYPE_LIMITS, "LIMITS", 0, "Limits", ""},
-       {FMODIFIER_TYPE_STEPPED, "STEPPED", 0, "Stepped Interpolation", ""},
+       /*{FMODIFIER_TYPE_PYTHON, "PYTHON", 0, "Python", ""},*/ /* FIXME: not implemented yet! */
+       {FMODIFIER_TYPE_LIMITS, "LIMITS", 0, "Limits",
+                               "Restrict maximum and minimum values of F-Curve"},
+       {FMODIFIER_TYPE_STEPPED, "STEPPED", 0, "Stepped Interpolation",
+                                "Snap values to nearest grid-step - e.g. for a stop-motion look"},
        {0, NULL, 0, NULL, NULL}
 };
 
-EnumPropertyItem beztriple_keyframe_type_items[] = {
-       {BEZT_KEYTYPE_KEYFRAME, "KEYFRAME", 0, "Keyframe", ""},
-       {BEZT_KEYTYPE_BREAKDOWN, "BREAKDOWN", 0, "Breakdown", ""},
-       {BEZT_KEYTYPE_EXTREME, "EXTREME", 0, "Extreme", ""},
-       {BEZT_KEYTYPE_JITTER, "JITTER", 0, "Jitter", ""},
+EnumPropertyItem rna_enum_beztriple_keyframe_type_items[] = {
+       {BEZT_KEYTYPE_KEYFRAME, "KEYFRAME", VICO_KEYTYPE_KEYFRAME_VEC, "Keyframe", "Normal keyframe - e.g. for key poses"},
+       {BEZT_KEYTYPE_BREAKDOWN, "BREAKDOWN", VICO_KEYTYPE_BREAKDOWN_VEC, "Breakdown", "A breakdown pose - e.g. for transitions between key poses"},
+       {BEZT_KEYTYPE_MOVEHOLD, "MOVING_HOLD", VICO_KEYTYPE_MOVING_HOLD_VEC, "Moving Hold", "A keyframe that is part of a moving hold"},
+       {BEZT_KEYTYPE_EXTREME, "EXTREME", VICO_KEYTYPE_EXTREME_VEC, "Extreme", "An 'extreme' pose, or some other purpose as needed"},
+       {BEZT_KEYTYPE_JITTER, "JITTER", VICO_KEYTYPE_JITTER_VEC, "Jitter", "A filler or baked keyframe for keying on ones, or some other purpose as needed"},
+       {0, NULL, 0, NULL, NULL}
+};
+
+EnumPropertyItem rna_enum_beztriple_interpolation_easing_items[] =  {
+       /* XXX: auto-easing is currently using a placeholder icon... */
+       {BEZT_IPO_EASE_AUTO, "AUTO", ICON_IPO_EASE_IN_OUT, "Automatic Easing",
+                            "Easing type is chosen automatically based on what the type of interpolation used "
+                            "(e.g. 'Ease In' for transitional types, and 'Ease Out' for dynamic effects)"},
+                                                
+       {BEZT_IPO_EASE_IN, "EASE_IN", ICON_IPO_EASE_IN, "Ease In", "Only on the end closest to the next keyframe"},
+       {BEZT_IPO_EASE_OUT, "EASE_OUT", ICON_IPO_EASE_OUT, "Ease Out", "Only on the end closest to the first keyframe"},
+       {BEZT_IPO_EASE_IN_OUT, "EASE_IN_OUT", ICON_IPO_EASE_IN_OUT, "Ease In and Out", "Segment between both keyframes"},
        {0, NULL, 0, NULL, NULL}
 };
 
@@ -116,7 +136,7 @@ static void rna_ChannelDriver_update_data(Main *bmain, Scene *scene, PointerRNA
        driver->flag &= ~DRIVER_FLAG_INVALID;
        
        /* TODO: this really needs an update guard... */
-       DAG_scene_sort(bmain, scene);
+       DAG_relations_tag_update(bmain);
        DAG_id_tag_update(id, OB_RECALC_OB | OB_RECALC_DATA);
        
        WM_main_add_notifier(NC_SCENE | ND_FRAME, scene);
@@ -125,7 +145,11 @@ static void rna_ChannelDriver_update_data(Main *bmain, Scene *scene, PointerRNA
 static void rna_ChannelDriver_update_expr(Main *bmain, Scene *scene, PointerRNA *ptr)
 {
        ChannelDriver *driver = ptr->data;
+       
+       /* tag driver as needing to be recompiled */
        driver->flag |= DRIVER_FLAG_RECOMPILE;
+       
+       /* update_data() clears invalid flag and schedules for updates */
        rna_ChannelDriver_update_data(bmain, scene, ptr);
 }
 
@@ -252,6 +276,35 @@ static void rna_DriverVariable_type_set(PointerRNA *ptr, int value)
        driver_change_variable_type(dvar, value);
 }
 
+void rna_DriverVariable_name_set(PointerRNA *ptr, const char *value)
+{
+       DriverVar *data = (DriverVar *)(ptr->data);
+       
+       BLI_strncpy_utf8(data->name, value, 64);
+       driver_variable_name_validate(data);
+}
+
+
+/* ----------- */
+
+static DriverVar *rna_Driver_new_variable(ChannelDriver *driver)
+{
+       /* call the API function for this */
+       return driver_add_new_variable(driver);
+}
+
+static void rna_Driver_remove_variable(ChannelDriver *driver, ReportList *reports, PointerRNA *dvar_ptr)
+{
+       DriverVar *dvar = dvar_ptr->data;
+       if (BLI_findindex(&driver->variables, dvar) == -1) {
+               BKE_report(reports, RPT_ERROR, "Variable does not exist in this driver");
+               return;
+       }
+
+       driver_free_variable_ex(driver, dvar);
+       RNA_POINTER_INVALIDATE(dvar_ptr);
+}
+
 /* ****************************** */
 
 static void rna_FKeyframe_handle1_get(PointerRNA *ptr, float *values)
@@ -403,22 +456,25 @@ static void rna_FCurve_group_set(PointerRNA *ptr, PointerRNA value)
        }
 }
 
-static DriverVar *rna_Driver_new_variable(ChannelDriver *driver)
+/* calculate time extents of F-Curve */
+static void rna_FCurve_range(FCurve *fcu, float range[2])
 {
-       /* call the API function for this */
-       return driver_add_new_variable(driver);
+       calc_fcurve_range(fcu, range, range + 1, false, false);
 }
 
-static void rna_Driver_remove_variable(ChannelDriver *driver, ReportList *reports, PointerRNA *dvar_ptr)
+
+/* allow scripts to update curve after editing manually */
+static void rna_FCurve_update_data_ex(FCurve *fcu)
 {
-       DriverVar *dvar = dvar_ptr->data;
-       if (BLI_findindex(&driver->variables, dvar) == -1) {
-               BKE_report(reports, RPT_ERROR, "Variable does not exist in this driver");
-               return;
-       }
+       sort_time_fcurve(fcu);
+       calchandles_fcurve(fcu);
+}
 
-       driver_free_variable(driver, dvar);
-       RNA_POINTER_INVALIDATE(dvar_ptr);
+/* RNA update callback for F-Curves after curve shape changes */
+static void rna_FCurve_update_data(Main *UNUSED(bmain), Scene *UNUSED(scene), PointerRNA *ptr)
+{
+       BLI_assert(ptr->type == &RNA_FCurve);
+       rna_FCurve_update_data_ex((FCurve *)ptr->data);
 }
 
 
@@ -454,29 +510,70 @@ static void rna_FCurve_modifiers_remove(FCurve *fcu, ReportList *reports, Pointe
 
 static void rna_FModifier_active_set(PointerRNA *ptr, int UNUSED(value))
 {
-       FModifier *fm = (FModifier *)ptr->data;
+       FModifier *fcm = (FModifier *)ptr->data;
 
        /* don't toggle, always switch on */
-       fm->flag |= FMODIFIER_FLAG_ACTIVE;
+       fcm->flag |= FMODIFIER_FLAG_ACTIVE;
 }
 
-static void rna_FModifier_start_frame_range(PointerRNA *ptr, float *min, float *max, float *softmin, float *softmax)
+static void rna_FModifier_start_frame_set(PointerRNA *ptr, float value)
 {
        FModifier *fcm = (FModifier *)ptr->data;
        
-       *min = MINAFRAMEF;
-       *max = (fcm->flag & FMODIFIER_FLAG_RANGERESTRICT) ? fcm->efra : MAXFRAMEF;
+       CLAMP(value, MINAFRAMEF, MAXFRAMEF);
+       fcm->sfra = value;
+       
+       /* XXX: maintain old offset? */
+       if (fcm->sfra >= fcm->efra) {
+               fcm->efra = fcm->sfra;
+       }
 }
 
-static void rna_FModifier_end_frame_range(PointerRNA *ptr, float *min, float *max, float *softmin, float *softmax)
+static void rna_FModifer_end_frame_set(PointerRNA *ptr, float value)
 {
        FModifier *fcm = (FModifier *)ptr->data;
+       
+       CLAMP(value, MINAFRAMEF, MAXFRAMEF);
+       fcm->efra = value;
+       
+       /* XXX: maintain old offset? */
+       if (fcm->efra <= fcm->sfra) {
+               fcm->sfra = fcm->efra;
+       }
+}
 
-       *min = (fcm->flag & FMODIFIER_FLAG_RANGERESTRICT) ? fcm->sfra : MINAFRAMEF;
-       *max = MAXFRAMEF;
+static void rna_FModifier_start_frame_range(PointerRNA *UNUSED(ptr), float *min, float *max,
+                                            float *UNUSED(softmin), float *UNUSED(softmax))
+{
+       // FModifier *fcm = (FModifier *)ptr->data;
+       
+       /* Technically, "sfra <= efra" must hold; however, we can't strictly enforce that, 
+        * or else it becomes tricky to adjust the range...  [#36844] 
+        * 
+        * NOTE: we do not set soft-limits on lower bounds, as it's too confusing when you 
+        *       can't easily use the slider to set things here
+        */
+       *min     = MINAFRAMEF;
+       *max     = MAXFRAMEF;
 }
 
-static void rna_FModifier_blending_range(PointerRNA *ptr, float *min, float *max, float *softmin, float *softmax)
+static void rna_FModifier_end_frame_range(PointerRNA *ptr, float *min, float *max,
+                                          float *softmin, float *softmax)
+{
+       FModifier *fcm = (FModifier *)ptr->data;
+       
+       /* Technically, "sfra <= efra" must hold; however, we can't strictly enforce that, 
+        * or else it becomes tricky to adjust the range...  [#36844] 
+        */
+       *min     = MINAFRAMEF;
+       *softmin = (fcm->flag & FMODIFIER_FLAG_RANGERESTRICT) ? fcm->sfra : MINAFRAMEF;
+       
+       *softmax = MAXFRAMEF;
+       *max     = MAXFRAMEF;
+}
+
+static void rna_FModifier_blending_range(PointerRNA *ptr, float *min, float *max,
+                                         float *UNUSED(softmin), float *UNUSED(softmax))
 {
        FModifier *fcm = (FModifier *)ptr->data;
 
@@ -484,7 +581,29 @@ static void rna_FModifier_blending_range(PointerRNA *ptr, float *min, float *max
        *max = fcm->efra - fcm->sfra;
 }
 
-static void rna_FModifier_active_update(Main *UNUSED(bmain), Scene *UNUSED(scene), PointerRNA *ptr)
+static void rna_FModifier_update(Main *UNUSED(bmain), Scene *UNUSED(scene), PointerRNA *ptr)
+{
+       ID *id = ptr->id.data;
+       AnimData *adt = BKE_animdata_from_id(id);
+       DAG_id_tag_update(id, (GS(id->name) == ID_OB) ? OB_RECALC_OB : OB_RECALC_DATA);
+       if (adt != NULL) {
+               adt->recalc |= ADT_RECALC_ANIM;
+       }
+}
+
+static void rna_FModifier_verify_data_update(Main *bmain, Scene *scene, PointerRNA *ptr)
+{
+       FModifier *fcm = (FModifier *)ptr->data;
+       const FModifierTypeInfo *fmi = fmodifier_get_typeinfo(fcm);
+
+       /* call the verify callback on the modifier if applicable */
+       if (fmi && fmi->verify_data)
+               fmi->verify_data(fcm);
+
+       rna_FModifier_update(bmain, scene, ptr);
+}
+
+static void rna_FModifier_active_update(Main *bmain, Scene *scene, PointerRNA *ptr)
 {
        FModifier *fm, *fmo = (FModifier *)ptr->data;
 
@@ -495,7 +614,8 @@ static void rna_FModifier_active_update(Main *UNUSED(bmain), Scene *UNUSED(scene
        for (fm = fmo->next; fm; fm = fm->next) {
                fm->flag &= ~FMODIFIER_FLAG_ACTIVE;
        }
-       
+
+       rna_FModifier_update(bmain, scene, ptr);
 }
 
 static int rna_FModifierGenerator_coefficients_get_length(PointerRNA *ptr, int length[RNA_MAX_ARRAY_DIMENSION])
@@ -525,45 +645,106 @@ static void rna_FModifierGenerator_coefficients_set(PointerRNA *ptr, const float
        memcpy(gen->coefficients, values, gen->arraysize * sizeof(float));
 }
 
-static void rna_FModifierLimits_minx_range(PointerRNA *ptr, float *min, float *max, float *softmin, float *softmax)
+
+static void rna_FModifierLimits_minx_set(PointerRNA *ptr, float value)
 {
        FModifier *fcm = (FModifier *)ptr->data;
        FMod_Limits *data = fcm->data;
-
-       *min = MINAFRAMEF;
-       *max = (data->flag & FCM_LIMIT_XMAX) ? data->rect.xmax : MAXFRAMEF;
+       
+       data->rect.xmin = value;
+       
+       if (data->rect.xmin >= data->rect.xmax) {
+               data->rect.xmax = data->rect.xmin;
+       }
 }
 
-static void rna_FModifierLimits_maxx_range(PointerRNA *ptr, float *min, float *max, float *softmin, float *softmax)
+static void rna_FModifierLimits_maxx_set(PointerRNA *ptr, float value)
 {
        FModifier *fcm = (FModifier *)ptr->data;
        FMod_Limits *data = fcm->data;
+       
+       data->rect.xmax = value;
+       
+       if (data->rect.xmax <= data->rect.xmin) {
+               data->rect.xmin = data->rect.xmax;
+       }
+}
 
-       *min = (data->flag & FCM_LIMIT_XMIN) ? data->rect.xmin : MINAFRAMEF;
-       *max = MAXFRAMEF;
+static void rna_FModifierLimits_miny_set(PointerRNA *ptr, float value)
+{
+       FModifier *fcm = (FModifier *)ptr->data;
+       FMod_Limits *data = fcm->data;
+       
+       data->rect.ymin = value;
+       
+       if (data->rect.ymin >= data->rect.ymax) {
+               data->rect.ymax = data->rect.ymin;
+       }
 }
 
-static void rna_FModifierLimits_miny_range(PointerRNA *ptr, float *min, float *max, float *softmin, float *softmax)
+static void rna_FModifierLimits_maxy_set(PointerRNA *ptr, float value)
 {
        FModifier *fcm = (FModifier *)ptr->data;
        FMod_Limits *data = fcm->data;
+       
+       data->rect.ymax = value;
+       
+       if (data->rect.ymax <= data->rect.ymin) {
+               data->rect.ymin = data->rect.ymax;
+       }
+}
 
-       *min = -FLT_MAX;
-       *max = (data->flag & FCM_LIMIT_YMAX) ? data->rect.ymax : FLT_MAX;
+static void rna_FModifierLimits_minx_range(PointerRNA *UNUSED(ptr), float *min, float *max,
+                                           float *UNUSED(softmin), float *UNUSED(softmax))
+{
+       // FModifier *fcm = (FModifier *)ptr->data;
+       // FMod_Limits *data = fcm->data;
+       
+       /* no soft-limits on lower bound - it's too confusing when you can't easily use the slider to set things here */
+       *min     = MINAFRAMEF;
+       *max     = MAXFRAMEF;
 }
 
-static void rna_FModifierLimits_maxy_range(PointerRNA *ptr, float *min, float *max, float *softmin, float *softmax)
+static void rna_FModifierLimits_maxx_range(PointerRNA *ptr, float *min, float *max,
+                                           float *softmin, float *softmax)
 {
        FModifier *fcm = (FModifier *)ptr->data;
        FMod_Limits *data = fcm->data;
+       
+       *min     = MINAFRAMEF;
+       *softmin = (data->flag & FCM_LIMIT_XMIN) ? data->rect.xmin : MINAFRAMEF;
+       
+       *softmax = MAXFRAMEF;
+       *max     = MAXFRAMEF;
+}
+
+static void rna_FModifierLimits_miny_range(PointerRNA *UNUSED(ptr), float *min, float *max,
+                                           float *UNUSED(softmin), float *UNUSED(softmax))
+{
+       // FModifier *fcm = (FModifier *)ptr->data;
+       // FMod_Limits *data = fcm->data;
+       
+       /* no soft-limits on lower bound - it's too confusing when you can't easily use the slider to set things here */
+       *min     = -FLT_MAX;
+       *max     = FLT_MAX;
+}
 
-       *min = (data->flag & FCM_LIMIT_YMIN) ? data->rect.ymin : -FLT_MAX;
-       *max = FLT_MAX;
+static void rna_FModifierLimits_maxy_range(PointerRNA *ptr, float *min, float *max,
+                                           float *softmin, float *softmax)
+{
+       FModifier *fcm = (FModifier *)ptr->data;
+       FMod_Limits *data = fcm->data;
+       
+       *min     = -FLT_MAX;
+       *softmin = (data->flag & FCM_LIMIT_YMIN) ? data->rect.ymin : -FLT_MAX;
+       
+       *softmax = FLT_MAX;
+       *max     = FLT_MAX; 
 }
 
 
 static void rna_FModifierStepped_start_frame_range(PointerRNA *ptr, float *min, float *max,
-                                                   float *softmin, float *softmax)
+                                                   float *UNUSED(softmin), float *UNUSED(softmax))
 {
        FModifier *fcm = (FModifier *)ptr->data;
        FMod_Stepped *data = fcm->data;
@@ -573,7 +754,7 @@ static void rna_FModifierStepped_start_frame_range(PointerRNA *ptr, float *min,
 }
 
 static void rna_FModifierStepped_end_frame_range(PointerRNA *ptr, float *min, float *max,
-                                                 float *softmin, float *softmax)
+                                                 float *UNUSED(softmin), float *UNUSED(softmax))
 {
        FModifier *fcm = (FModifier *)ptr->data;
        FMod_Stepped *data = fcm->data;
@@ -582,9 +763,9 @@ static void rna_FModifierStepped_end_frame_range(PointerRNA *ptr, float *min, fl
        *max = MAXFRAMEF;
 }
 
-static BezTriple *rna_FKeyframe_points_insert(FCurve *fcu, float frame, float value, int flag)
+static BezTriple *rna_FKeyframe_points_insert(FCurve *fcu, float frame, float value, int keyframe_type, int flag)
 {
-       int index = insert_vert_fcurve(fcu, frame, value, flag);
+       int index = insert_vert_fcurve(fcu, frame, value, (char)keyframe_type, flag | INSERTKEY_NO_USERPREF);
        return ((fcu->bezt) && (index >= 0)) ? (fcu->bezt + index) : NULL;
 }
 
@@ -592,15 +773,8 @@ static void rna_FKeyframe_points_add(FCurve *fcu, int tot)
 {
        if (tot > 0) {
                BezTriple *bezt;
-               if (fcu->totvert) {
-                       BezTriple *nbezt = MEM_callocN(sizeof(BezTriple) * (fcu->totvert + tot), "rna_FKeyframe_points_add");
-                       memcpy(nbezt, fcu->bezt, sizeof(BezTriple) * fcu->totvert);
-                       MEM_freeN(fcu->bezt);
-                       fcu->bezt = nbezt;
-               }
-               else {
-                       fcu->bezt = MEM_callocN(sizeof(BezTriple) * tot, "rna_FKeyframe_points_add");
-               }
+
+               fcu->bezt = MEM_recallocN(fcu->bezt, sizeof(BezTriple) * (fcu->totvert + tot));
                
                bezt = fcu->bezt + fcu->totvert;
                fcu->totvert += tot;
@@ -628,27 +802,20 @@ static void rna_FKeyframe_points_remove(FCurve *fcu, ReportList *reports, Pointe
        RNA_POINTER_INVALIDATE(bezt_ptr);
 }
 
-static void rna_fcurve_range(FCurve *fcu, float range[2])
-{
-       calc_fcurve_range(fcu, range, range + 1, FALSE, FALSE);
-}
-
-
 static FCM_EnvelopeData *rna_FModifierEnvelope_points_add(FModifier *fmod, ReportList *reports, float frame)
 {
        FCM_EnvelopeData fed;
        FMod_Envelope *env = (FMod_Envelope *)fmod->data;
+       int i;
 
        /* init template data */
        fed.min = -1.0f;
        fed.max = 1.0f;
        fed.time = frame;
        fed.f1 = fed.f2 = 0;
-       int i;
 
        if (env->data) {
-               /* add point to end of control points */
-               short exists = -1;
+               bool exists;
                i = BKE_fcm_envelope_find_index(env->data, frame, env->totvert, &exists);
                if (exists) {
                        BKE_reportf(reports, RPT_ERROR, "Already a control point at frame %.6f", frame);
@@ -676,7 +843,7 @@ static FCM_EnvelopeData *rna_FModifierEnvelope_points_add(FModifier *fmod, Repor
        return (env->data + i);
 }
 
-void rna_FModifierEnvelope_points_remove(FModifier *fmod, ReportList *reports, PointerRNA *point)
+static void rna_FModifierEnvelope_points_remove(FModifier *fmod, ReportList *reports, PointerRNA *point)
 {
        FCM_EnvelopeData *cp = point->data;
        FMod_Envelope *env = (FMod_Envelope *)fmod->data;
@@ -685,7 +852,7 @@ void rna_FModifierEnvelope_points_remove(FModifier *fmod, ReportList *reports, P
 
        /* test point is in range */
        if (index < 0 || index >= env->totvert) {
-               BKE_report(reports, RPT_ERROR, "Control Point not in FEnvelopeModifier");
+               BKE_report(reports, RPT_ERROR, "Control point not in Envelope F-Modifier");
                return;
        }
 
@@ -732,21 +899,19 @@ static void rna_def_fmodifier_generator(BlenderRNA *brna)
        RNA_def_property_ui_text(prop, "Additive",
                                 "Values generated by this modifier are applied on top of "
                                 "the existing values instead of overwriting them");
-       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, "rna_FModifier_update");
        
-       /* XXX this has a special validation func */
        prop = RNA_def_property(srna, "mode", PROP_ENUM, PROP_NONE);
        RNA_def_property_enum_items(prop, generator_mode_items);
        RNA_def_property_ui_text(prop, "Mode", "Type of generator to use");
-       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, NULL);
-       
+       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, "rna_FModifier_verify_data_update");
        
        /* order of the polynomial */
-       /* XXX this has a special validation func */
        prop = RNA_def_property(srna, "poly_order", PROP_INT, PROP_NONE);
        RNA_def_property_ui_text(prop, "Polynomial Order",
                                 "The highest power of 'x' for this polynomial (number of coefficients - 1)");
-       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, NULL);
+       RNA_def_property_range(prop, 1, 100);
+       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, "rna_FModifier_verify_data_update");
        
        /* coefficients array */
        prop = RNA_def_property(srna, "coefficients", PROP_FLOAT, PROP_NONE);
@@ -782,19 +947,19 @@ static void rna_def_fmodifier_function_generator(BlenderRNA *brna)
        /* coefficients */
        prop = RNA_def_property(srna, "amplitude", PROP_FLOAT, PROP_NONE);
        RNA_def_property_ui_text(prop, "Amplitude", "Scale factor determining the maximum/minimum values");
-       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, "rna_FModifier_update");
        
        prop = RNA_def_property(srna, "phase_multiplier", PROP_FLOAT, PROP_NONE);
        RNA_def_property_ui_text(prop, "Phase Multiplier", "Scale factor determining the 'speed' of the function");
-       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, "rna_FModifier_update");
        
        prop = RNA_def_property(srna, "phase_offset", PROP_FLOAT, PROP_NONE);
        RNA_def_property_ui_text(prop, "Phase Offset", "Constant factor to offset time by for function");
-       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, "rna_FModifier_update");
        
        prop = RNA_def_property(srna, "value_offset", PROP_FLOAT, PROP_NONE);
        RNA_def_property_ui_text(prop, "Value Offset", "Constant factor to offset values by");
-       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, "rna_FModifier_update");
        
        /* flags */
        prop = RNA_def_property(srna, "use_additive", PROP_BOOLEAN, PROP_NONE);
@@ -802,13 +967,13 @@ static void rna_def_fmodifier_function_generator(BlenderRNA *brna)
        RNA_def_property_ui_text(prop, "Additive",
                                 "Values generated by this modifier are applied on top of "
                                 "the existing values instead of overwriting them");
-       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, "rna_FModifier_update");
        
        prop = RNA_def_property(srna, "function_type", PROP_ENUM, PROP_NONE);
        RNA_def_property_enum_sdna(prop, NULL, "type");
        RNA_def_property_enum_items(prop, prop_type_items);
        RNA_def_property_ui_text(prop, "Type", "Type of built-in function to use");
-       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, "rna_FModifier_update");
 }
 
 /* --------- */
@@ -829,18 +994,18 @@ static void rna_def_fmodifier_envelope_ctrl(BlenderRNA *brna)
        prop = RNA_def_property(srna, "min", PROP_FLOAT, PROP_NONE);
        RNA_def_property_float_sdna(prop, NULL, "min");
        RNA_def_property_ui_text(prop, "Minimum Value", "Lower bound of envelope at this control-point");
-       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, "rna_FModifier_update");
        
        prop = RNA_def_property(srna, "max", PROP_FLOAT, PROP_NONE);
        RNA_def_property_float_sdna(prop, NULL, "max");
        RNA_def_property_ui_text(prop, "Maximum Value", "Upper bound of envelope at this control-point");
-       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, "rna_FModifier_update");
        
        /* Frame */
        prop = RNA_def_property(srna, "frame", PROP_FLOAT, PROP_TIME);
        RNA_def_property_float_sdna(prop, NULL, "time");
        RNA_def_property_ui_text(prop, "Frame", "Frame this control-point occurs on");
-       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, "rna_FModifier_update");
        
        /* TODO: */
        /*      - selection flags (not implemented in UI yet though) */
@@ -896,17 +1061,17 @@ static void rna_def_fmodifier_envelope(BlenderRNA *brna)
        prop = RNA_def_property(srna, "reference_value", PROP_FLOAT, PROP_NONE);
        RNA_def_property_float_sdna(prop, NULL, "midval");
        RNA_def_property_ui_text(prop, "Reference Value", "Value that envelope's influence is centered around / based on");
-       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, "rna_FModifier_update");
        
        prop = RNA_def_property(srna, "default_min", PROP_FLOAT, PROP_NONE);
        RNA_def_property_float_sdna(prop, NULL, "min");
        RNA_def_property_ui_text(prop, "Default Minimum", "Lower distance from Reference Value for 1:1 default influence");
-       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, "rna_FModifier_update");
        
        prop = RNA_def_property(srna, "default_max", PROP_FLOAT, PROP_NONE);
        RNA_def_property_float_sdna(prop, NULL, "max");
        RNA_def_property_ui_text(prop, "Default Maximum", "Upper distance from Reference Value for 1:1 default influence");
-       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, "rna_FModifier_update");
 }
 
 /* --------- */
@@ -936,26 +1101,26 @@ static void rna_def_fmodifier_cycles(BlenderRNA *brna)
        RNA_def_property_enum_sdna(prop, NULL, "before_mode");
        RNA_def_property_enum_items(prop, prop_type_items);
        RNA_def_property_ui_text(prop, "Before Mode", "Cycling mode to use before first keyframe");
-       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, "rna_FModifier_update");
        
        prop = RNA_def_property(srna, "cycles_before", PROP_INT, PROP_NONE);
        RNA_def_property_int_sdna(prop, NULL, "before_cycles");
        RNA_def_property_ui_text(prop, "Before Cycles",
                                 "Maximum number of cycles to allow before first keyframe (0 = infinite)");
-       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, "rna_FModifier_update");
        
        /* after */
        prop = RNA_def_property(srna, "mode_after", PROP_ENUM, PROP_NONE);
        RNA_def_property_enum_sdna(prop, NULL, "after_mode");
        RNA_def_property_enum_items(prop, prop_type_items);
        RNA_def_property_ui_text(prop, "After Mode", "Cycling mode to use after last keyframe");
-       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, "rna_FModifier_update");
        
        prop = RNA_def_property(srna, "cycles_after", PROP_INT, PROP_NONE);
        RNA_def_property_int_sdna(prop, NULL, "after_cycles");
        RNA_def_property_ui_text(prop, "After Cycles",
                                 "Maximum number of cycles to allow after last keyframe (0 = infinite)");
-       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, "rna_FModifier_update");
 }
 
 /* --------- */
@@ -984,46 +1149,46 @@ static void rna_def_fmodifier_limits(BlenderRNA *brna)
        prop = RNA_def_property(srna, "use_min_x", PROP_BOOLEAN, PROP_NONE);
        RNA_def_property_boolean_sdna(prop, NULL, "flag", FCM_LIMIT_XMIN);
        RNA_def_property_ui_text(prop, "Minimum X", "Use the minimum X value");
-       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, "rna_FModifier_update");
        
        prop = RNA_def_property(srna, "use_min_y", PROP_BOOLEAN, PROP_NONE);
        RNA_def_property_boolean_sdna(prop, NULL, "flag", FCM_LIMIT_YMIN);
        RNA_def_property_ui_text(prop, "Minimum Y", "Use the minimum Y value");
-       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, "rna_FModifier_update");
        
        prop = RNA_def_property(srna, "use_max_x", PROP_BOOLEAN, PROP_NONE);
        RNA_def_property_boolean_sdna(prop, NULL, "flag", FCM_LIMIT_XMAX);
        RNA_def_property_ui_text(prop, "Maximum X", "Use the maximum X value");
-       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, "rna_FModifier_update");
        
        prop = RNA_def_property(srna, "use_max_y", PROP_BOOLEAN, PROP_NONE);
        RNA_def_property_boolean_sdna(prop, NULL, "flag", FCM_LIMIT_YMAX);
        RNA_def_property_ui_text(prop, "Maximum Y", "Use the maximum Y value");
-       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, "rna_FModifier_update");
        
        prop = RNA_def_property(srna, "min_x", PROP_FLOAT, PROP_NONE);
        RNA_def_property_float_sdna(prop, NULL, "rect.xmin");
-       RNA_def_property_float_funcs(prop, NULL, NULL, "rna_FModifierLimits_minx_range");
+       RNA_def_property_float_funcs(prop, NULL, "rna_FModifierLimits_minx_set", "rna_FModifierLimits_minx_range");
        RNA_def_property_ui_text(prop, "Minimum X", "Lowest X value to allow");
-       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, "rna_FModifier_update");
        
        prop = RNA_def_property(srna, "min_y", PROP_FLOAT, PROP_NONE);
        RNA_def_property_float_sdna(prop, NULL, "rect.ymin");
-       RNA_def_property_float_funcs(prop, NULL, NULL, "rna_FModifierLimits_miny_range");
+       RNA_def_property_float_funcs(prop, NULL, "rna_FModifierLimits_miny_set", "rna_FModifierLimits_miny_range");
        RNA_def_property_ui_text(prop, "Minimum Y", "Lowest Y value to allow");
-       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, "rna_FModifier_update");
        
        prop = RNA_def_property(srna, "max_x", PROP_FLOAT, PROP_NONE);
        RNA_def_property_float_sdna(prop, NULL, "rect.xmax");
-       RNA_def_property_float_funcs(prop, NULL, NULL, "rna_FModifierLimits_maxx_range");
+       RNA_def_property_float_funcs(prop, NULL, "rna_FModifierLimits_maxx_set", "rna_FModifierLimits_maxx_range");
        RNA_def_property_ui_text(prop, "Maximum X", "Highest X value to allow");
-       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, "rna_FModifier_update");
        
        prop = RNA_def_property(srna, "max_y", PROP_FLOAT, PROP_NONE);
        RNA_def_property_float_sdna(prop, NULL, "rect.ymax");
-       RNA_def_property_float_funcs(prop, NULL, NULL, "rna_FModifierLimits_maxy_range");
+       RNA_def_property_float_funcs(prop, NULL, "rna_FModifierLimits_maxy_set", "rna_FModifierLimits_maxy_range");
        RNA_def_property_ui_text(prop, "Maximum Y", "Highest Y value to allow");
-       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, "rna_FModifier_update");
 }
 
 /* --------- */
@@ -1049,28 +1214,33 @@ static void rna_def_fmodifier_noise(BlenderRNA *brna)
        RNA_def_property_enum_sdna(prop, NULL, "modification");
        RNA_def_property_enum_items(prop, prop_modification_items);
        RNA_def_property_ui_text(prop, "Blend Type", "Method of modifying the existing F-Curve");
-       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, "rna_FModifier_update");
        
        prop = RNA_def_property(srna, "scale", PROP_FLOAT, PROP_NONE);
        RNA_def_property_float_sdna(prop, NULL, "size");
        RNA_def_property_ui_text(prop, "Scale", "Scaling (in time) of the noise");
-       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, "rna_FModifier_update");
        
        prop = RNA_def_property(srna, "strength", PROP_FLOAT, PROP_NONE);
        RNA_def_property_float_sdna(prop, NULL, "strength");
        RNA_def_property_ui_text(prop, "Strength",
                                 "Amplitude of the noise - the amount that it modifies the underlying curve");
-       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, "rna_FModifier_update");
        
        prop = RNA_def_property(srna, "phase", PROP_FLOAT, PROP_NONE);
        RNA_def_property_float_sdna(prop, NULL, "phase");
        RNA_def_property_ui_text(prop, "Phase", "A random seed for the noise effect");
-       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, "rna_FModifier_update");
+
+       prop = RNA_def_property(srna, "offset", PROP_FLOAT, PROP_NONE);
+       RNA_def_property_float_sdna(prop, NULL, "offset");
+       RNA_def_property_ui_text(prop, "Offset", "Time offset for the noise effect");
+       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, "rna_FModifier_update");
        
        prop = RNA_def_property(srna, "depth", PROP_INT, PROP_UNSIGNED);
        RNA_def_property_int_sdna(prop, NULL, "depth");
        RNA_def_property_ui_text(prop, "Depth", "Amount of fine level detail present in the noise");
-       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, "rna_FModifier_update");
 
 }
 
@@ -1091,36 +1261,36 @@ static void rna_def_fmodifier_stepped(BlenderRNA *brna)
        prop = RNA_def_property(srna, "frame_step", PROP_FLOAT, PROP_NONE);
        RNA_def_property_float_sdna(prop, NULL, "step_size");
        RNA_def_property_ui_text(prop, "Step Size", "Number of frames to hold each value");
-       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, "rna_FModifier_update");
        
        prop = RNA_def_property(srna, "frame_offset", PROP_FLOAT, PROP_NONE);
        RNA_def_property_float_sdna(prop, NULL, "offset");
        RNA_def_property_ui_text(prop, "Offset",
                                 "Reference number of frames before frames get held "
                                 "(use to get hold for '1-3' vs '5-7' holding patterns)");
-       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, "rna_FModifier_update");
        
        prop = RNA_def_property(srna, "use_frame_start", PROP_BOOLEAN, PROP_NONE);
        RNA_def_property_boolean_sdna(prop, NULL, "flag", FCM_STEPPED_NO_BEFORE);
        RNA_def_property_ui_text(prop, "Use Start Frame", "Restrict modifier to only act after its 'start' frame");
-       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, "rna_FModifier_update");
        
        prop = RNA_def_property(srna, "use_frame_end", PROP_BOOLEAN, PROP_NONE);
        RNA_def_property_boolean_sdna(prop, NULL, "flag", FCM_STEPPED_NO_AFTER);
        RNA_def_property_ui_text(prop, "Use End Frame", "Restrict modifier to only act before its 'end' frame");
-       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, "rna_FModifier_update");
        
        prop = RNA_def_property(srna, "frame_start", PROP_FLOAT, PROP_NONE);
        RNA_def_property_float_sdna(prop, NULL, "start_frame");
        RNA_def_property_float_funcs(prop, NULL, NULL, "rna_FModifierStepped_start_frame_range");
        RNA_def_property_ui_text(prop, "Start Frame", "Frame that modifier's influence starts (if applicable)");
-       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, "rna_FModifier_update");
        
        prop = RNA_def_property(srna, "frame_end", PROP_FLOAT, PROP_NONE);
        RNA_def_property_float_sdna(prop, NULL, "end_frame");
        RNA_def_property_float_funcs(prop, NULL, NULL, "rna_FModifierStepped_end_frame_range");
        RNA_def_property_ui_text(prop, "End Frame", "Frame that modifier's influence ends (if applicable)");
-       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, "rna_FModifier_update");
 }
 
 /* --------- */
@@ -1146,7 +1316,7 @@ static void rna_def_fmodifier(BlenderRNA *brna)
        /* type */
        prop = RNA_def_property(srna, "type", PROP_ENUM, PROP_NONE);
        RNA_def_property_clear_flag(prop, PROP_EDITABLE);
-       RNA_def_property_enum_items(prop, fmodifier_type_items);
+       RNA_def_property_enum_items(prop, rna_enum_fmodifier_type_items);
        RNA_def_property_ui_text(prop, "Type", "F-Curve Modifier Type");
        
        /* settings */
@@ -1158,14 +1328,14 @@ static void rna_def_fmodifier(BlenderRNA *brna)
        prop = RNA_def_property(srna, "mute", PROP_BOOLEAN, PROP_NONE);
        RNA_def_property_boolean_sdna(prop, NULL, "flag", FMODIFIER_FLAG_MUTED);
        RNA_def_property_ui_text(prop, "Muted", "F-Curve Modifier will not be evaluated");
-       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME_PROP, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME_PROP, "rna_FModifier_update");
        RNA_def_property_ui_icon(prop, ICON_MUTE_IPO_OFF, 1);
        
        prop = RNA_def_property(srna, "is_valid", PROP_BOOLEAN, PROP_NONE);
        RNA_def_property_clear_flag(prop, PROP_EDITABLE);
        RNA_def_property_boolean_negative_sdna(prop, NULL, "flag", FMODIFIER_FLAG_DISABLED);
        RNA_def_property_ui_text(prop, "Disabled", "F-Curve Modifier has invalid settings and will not be evaluated");
-       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME_PROP, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME_PROP, "rna_FModifier_update");
        
        /* TODO: setting this to true must ensure that all others in stack are turned off too... */
        prop = RNA_def_property(srna, "active", PROP_BOOLEAN, PROP_NONE);
@@ -1181,40 +1351,40 @@ static void rna_def_fmodifier(BlenderRNA *brna)
        RNA_def_property_ui_text(prop, "Restrict Frame Range",
                                 "F-Curve Modifier is only applied for the specified frame range to help "
                                 "mask off effects in order to chain them");
-       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME_PROP, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME_PROP, "rna_FModifier_update");
        RNA_def_property_ui_icon(prop, ICON_TRIA_RIGHT, 1); /* XXX: depends on UI implementation */
        
        prop = RNA_def_property(srna, "frame_start", PROP_FLOAT, PROP_NONE);
        RNA_def_property_float_sdna(prop, NULL, "sfra");
-       RNA_def_property_float_funcs(prop, NULL, NULL, "rna_FModifier_start_frame_range");
+       RNA_def_property_float_funcs(prop, NULL, "rna_FModifier_start_frame_set", "rna_FModifier_start_frame_range");
        RNA_def_property_ui_text(prop, "Start Frame",
                                 "Frame that modifier's influence starts (if Restrict Frame Range is in use)");
-       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME_PROP, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME_PROP, "rna_FModifier_update");
        
        prop = RNA_def_property(srna, "frame_end", PROP_FLOAT, PROP_NONE);
        RNA_def_property_float_sdna(prop, NULL, "efra");
-       RNA_def_property_float_funcs(prop, NULL, NULL, "rna_FModifier_end_frame_range");
+       RNA_def_property_float_funcs(prop, NULL, "rna_FModifer_end_frame_set", "rna_FModifier_end_frame_range");
        RNA_def_property_ui_text(prop, "End Frame",
                                 "Frame that modifier's influence ends (if Restrict Frame Range is in use)");
-       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME_PROP, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME_PROP, "rna_FModifier_update");
        
        prop = RNA_def_property(srna, "blend_in", PROP_FLOAT, PROP_NONE);
        RNA_def_property_float_sdna(prop, NULL, "blendin");
        RNA_def_property_float_funcs(prop, NULL, NULL, "rna_FModifier_blending_range");
        RNA_def_property_ui_text(prop, "Blend In", "Number of frames from start frame for influence to take effect");
-       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME_PROP, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME_PROP, "rna_FModifier_update");
        
        prop = RNA_def_property(srna, "blend_out", PROP_FLOAT, PROP_NONE);
        RNA_def_property_float_sdna(prop, NULL, "blendout");
        RNA_def_property_float_funcs(prop, NULL, NULL, "rna_FModifier_blending_range");
        RNA_def_property_ui_text(prop, "Blend Out", "Number of frames from end frame for influence to fade out");
-       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME_PROP, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME_PROP, "rna_FModifier_update");
        
        /* influence */
        prop = RNA_def_property(srna, "use_influence", PROP_BOOLEAN, PROP_NONE);
        RNA_def_property_boolean_sdna(prop, NULL, "flag", FMODIFIER_FLAG_USEINFLUENCE);
        RNA_def_property_ui_text(prop, "Use Influence", "F-Curve Modifier's effects will be tempered by a default factor");
-       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME_PROP, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME_PROP, "rna_FModifier_update");
        RNA_def_property_ui_icon(prop, ICON_TRIA_RIGHT, 1); /* XXX: depends on UI implementation */
        
        prop = RNA_def_property(srna, "influence", PROP_FLOAT, PROP_FACTOR);
@@ -1223,7 +1393,7 @@ static void rna_def_fmodifier(BlenderRNA *brna)
        RNA_def_property_float_default(prop, 1.0f);
        RNA_def_property_ui_text(prop, "Influence",
                                 "Amount of influence F-Curve Modifier will have when not fading in/out");
-       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME_PROP, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME_PROP, "rna_FModifier_update");
 }
 
 /* *********************** */
@@ -1273,7 +1443,7 @@ static void rna_def_drivertarget(BlenderRNA *brna)
        
        prop = RNA_def_property(srna, "id_type", PROP_ENUM, PROP_NONE);
        RNA_def_property_enum_sdna(prop, NULL, "idtype");
-       RNA_def_property_enum_items(prop, id_type_items);
+       RNA_def_property_enum_items(prop, rna_enum_id_type_items);
        RNA_def_property_enum_default(prop, ID_OB);
        RNA_def_property_enum_funcs(prop, NULL, "rna_DriverTarget_id_type_set", NULL);
        RNA_def_property_editable_func(prop, "rna_DriverTarget_id_type_editable");
@@ -1311,11 +1481,11 @@ static void rna_def_drivervar(BlenderRNA *brna)
        PropertyRNA *prop;
        
        static EnumPropertyItem prop_type_items[] = {
-               {DVAR_TYPE_SINGLE_PROP, "SINGLE_PROP", 0, "Single Property", "Use the value from some RNA property (Default)"},
-               {DVAR_TYPE_TRANSFORM_CHAN, "TRANSFORMS", 0, "Transform Channel",
+               {DVAR_TYPE_SINGLE_PROP, "SINGLE_PROP", ICON_RNA, "Single Property", "Use the value from some RNA property (Default)"},
+               {DVAR_TYPE_TRANSFORM_CHAN, "TRANSFORMS", ICON_MANIPUL, "Transform Channel",
                                           "Final transformation value of object or bone"},
-               {DVAR_TYPE_ROT_DIFF, "ROTATION_DIFF", 0, "Rotational Difference", "Use the angle between two bones"},
-               {DVAR_TYPE_LOC_DIFF, "LOC_DIFF", 0, "Distance", "Distance between two bones or objects"},
+               {DVAR_TYPE_ROT_DIFF, "ROTATION_DIFF", ICON_PARTICLE_TIP, "Rotational Difference", "Use the angle between two bones"},  /* XXX: Icon... */
+               {DVAR_TYPE_LOC_DIFF, "LOC_DIFF", ICON_FULLSCREEN_ENTER, "Distance", "Distance between two bones or objects"},          /* XXX: Icon... */
                {0, NULL, 0, NULL, NULL}
        };
                
@@ -1327,6 +1497,7 @@ static void rna_def_drivervar(BlenderRNA *brna)
        /* Variable Name */
        prop = RNA_def_property(srna, "name", PROP_STRING, PROP_NONE);
        RNA_def_struct_name_property(srna, prop);
+       RNA_def_property_string_funcs(prop, NULL, NULL, "rna_DriverVariable_name_set");
        RNA_def_property_ui_text(prop, "Name",
                                 "Name to use in scripted expressions/functions (no spaces or dots are allowed, "
                                 "and must start with a letter)");
@@ -1346,6 +1517,12 @@ static void rna_def_drivervar(BlenderRNA *brna)
        RNA_def_property_collection_sdna(prop, NULL, "targets", "num_targets");
        RNA_def_property_struct_type(prop, "DriverTarget");
        RNA_def_property_ui_text(prop, "Targets", "Sources of input data for evaluating this variable");
+       
+       /* Name Validity Flags */
+       prop = RNA_def_property(srna, "is_name_valid", PROP_BOOLEAN, PROP_NONE);
+       RNA_def_property_boolean_negative_sdna(prop, NULL, "flag", DVAR_FLAG_INVALID_NAME);
+       RNA_def_property_clear_flag(prop, PROP_EDITABLE);
+       RNA_def_property_ui_text(prop, "Is Name Valid", "Is this a valid name for a driver variable");  
 }
 
 
@@ -1423,7 +1600,13 @@ static void rna_def_channeldriver(BlenderRNA *brna)
        RNA_def_property_boolean_sdna(prop, NULL, "flag", DRIVER_FLAG_SHOWDEBUG);
        RNA_def_property_ui_text(prop, "Show Debug Info",
                                 "Show intermediate values for the driver calculations to allow debugging of drivers");
-       
+
+       prop = RNA_def_property(srna, "use_self", PROP_BOOLEAN, PROP_NONE);
+       RNA_def_property_boolean_sdna(prop, NULL, "flag", DRIVER_FLAG_USE_SELF);
+       RNA_def_property_ui_text(prop, "Use Self",
+                                "Pass 'self' argument to Py-Driver, "
+                                "so it can access the data referenced by the driver (object, bone, etc...)");
+
        /* State Info (for Debugging) */
        prop = RNA_def_property(srna, "is_valid", PROP_BOOLEAN, PROP_NONE);
        RNA_def_property_boolean_negative_sdna(prop, NULL, "flag", DRIVER_FLAG_INVALID);
@@ -1491,19 +1674,19 @@ static void rna_def_fkeyframe(BlenderRNA *brna)
        /* Enums */
        prop = RNA_def_property(srna, "handle_left_type", PROP_ENUM, PROP_NONE);
        RNA_def_property_enum_sdna(prop, NULL, "h1");
-       RNA_def_property_enum_items(prop, keyframe_handle_type_items);
+       RNA_def_property_enum_items(prop, rna_enum_keyframe_handle_type_items);
        RNA_def_property_ui_text(prop, "Left Handle Type", "Handle types");
        RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME_PROP, NULL);
        
        prop = RNA_def_property(srna, "handle_right_type", PROP_ENUM, PROP_NONE);
        RNA_def_property_enum_sdna(prop, NULL, "h2");
-       RNA_def_property_enum_items(prop, keyframe_handle_type_items);
+       RNA_def_property_enum_items(prop, rna_enum_keyframe_handle_type_items);
        RNA_def_property_ui_text(prop, "Right Handle Type", "Handle types");
        RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME_PROP, NULL);
        
        prop = RNA_def_property(srna, "interpolation", PROP_ENUM, PROP_NONE);
        RNA_def_property_enum_sdna(prop, NULL, "ipo");
-       RNA_def_property_enum_items(prop, beztriple_interpolation_mode_items);
+       RNA_def_property_enum_items(prop, rna_enum_beztriple_interpolation_mode_items);
        RNA_def_property_ui_text(prop, "Interpolation",
                                 "Interpolation method to use for segment of the F-Curve from "
                                 "this Keyframe until the next Keyframe");
@@ -1511,10 +1694,35 @@ static void rna_def_fkeyframe(BlenderRNA *brna)
        
        prop = RNA_def_property(srna, "type", PROP_ENUM, PROP_NONE);
        RNA_def_property_enum_sdna(prop, NULL, "hide");
-       RNA_def_property_enum_items(prop, beztriple_keyframe_type_items);
+       RNA_def_property_enum_items(prop, rna_enum_beztriple_keyframe_type_items);
        RNA_def_property_ui_text(prop, "Type", "Type of keyframe (for visual purposes only)");
        RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME_PROP, NULL);
        
+       
+       prop = RNA_def_property(srna, "easing", PROP_ENUM, PROP_NONE);
+       RNA_def_property_enum_sdna(prop, NULL, "easing");
+       RNA_def_property_enum_items(prop, rna_enum_beztriple_interpolation_easing_items);
+       RNA_def_property_ui_text(prop, "Easing", 
+                                "Which ends of the segment between this and the next keyframe easing "
+                                "interpolation is applied to");
+       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME_PROP, NULL);
+
+       prop = RNA_def_property(srna, "back", PROP_FLOAT, PROP_NONE);
+       RNA_def_property_float_sdna(prop, NULL, "back");
+       RNA_def_property_ui_text(prop, "Back", "Amount of overshoot for 'back' easing");
+       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME_PROP, NULL);
+
+       prop = RNA_def_property(srna, "amplitude", PROP_FLOAT, PROP_NONE);
+       RNA_def_property_float_sdna(prop, NULL, "amplitude");
+       RNA_def_property_range(prop, 0.0f, FLT_MAX); /* only positive values... */
+       RNA_def_property_ui_text(prop, "Amplitude", "Amount to boost elastic bounces for 'elastic' easing");
+       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME_PROP, NULL);
+
+       prop = RNA_def_property(srna, "period", PROP_FLOAT, PROP_NONE);
+       RNA_def_property_float_sdna(prop, NULL, "period");
+       RNA_def_property_ui_text(prop, "Period", "Time between bounces for elastic easing");
+       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME_PROP, NULL);
+       
        /* Vector values */
        prop = RNA_def_property(srna, "handle_left", PROP_FLOAT, PROP_COORDS); /* keyframes are dimensionless */
        RNA_def_property_array(prop, 2);
@@ -1565,7 +1773,7 @@ static void rna_def_fcurve_modifiers(BlenderRNA *brna, PropertyRNA *cprop)
        parm = RNA_def_pointer(func, "fmodifier", "FModifier", "", "New fmodifier");
        RNA_def_function_return(func, parm);
        /* object to add */
-       parm = RNA_def_enum(func, "type", fmodifier_type_items, 1, "", "Constraint type to add");
+       parm = RNA_def_enum(func, "type", rna_enum_fmodifier_type_items, 1, "", "Constraint type to add");
        RNA_def_property_flag(parm, PROP_REQUIRED);
 
        func = RNA_def_function(srna, "remove", "rna_FCurve_modifiers_remove");
@@ -1605,8 +1813,9 @@ static void rna_def_fcurve_keyframe_points(BlenderRNA *brna, PropertyRNA *cprop)
        parm = RNA_def_float(func, "value", 0.0f, -FLT_MAX, FLT_MAX, "",
                             "Y Value of this keyframe point", -FLT_MAX, FLT_MAX);
        RNA_def_property_flag(parm, PROP_REQUIRED);
-
        RNA_def_enum_flag(func, "options", keyframe_flag_items, 0, "", "Keyframe options");
+       RNA_def_enum(func, "keyframe_type", rna_enum_beztriple_keyframe_type_items, BEZT_KEYTYPE_KEYFRAME, "",
+                    "Type of keyframe to insert");
 
        parm = RNA_def_pointer(func, "keyframe", "Keyframe", "", "Newly created keyframe");
        RNA_def_function_return(func, parm);
@@ -1633,14 +1842,19 @@ static void rna_def_fcurve(BlenderRNA *brna)
        PropertyRNA *parm;
 
        static EnumPropertyItem prop_mode_extend_items[] = {
-               {FCURVE_EXTRAPOLATE_CONSTANT, "CONSTANT", 0, "Constant", ""},
-               {FCURVE_EXTRAPOLATE_LINEAR, "LINEAR", 0, "Linear", ""},
+               {FCURVE_EXTRAPOLATE_CONSTANT, "CONSTANT", 0, "Constant", "Hold values of endpoint keyframes"},
+               {FCURVE_EXTRAPOLATE_LINEAR, "LINEAR", 0, "Linear", "Use slope of curve leading in/out of endpoint keyframes"},
                {0, NULL, 0, NULL, NULL}
        };
        static EnumPropertyItem prop_mode_color_items[] = {
-               {FCURVE_COLOR_AUTO_RAINBOW, "AUTO_RAINBOW", 0, "Auto Rainbow", ""},
-               {FCURVE_COLOR_AUTO_RGB, "AUTO_RGB", 0, "Auto XYZ to RGB", ""},
-               {FCURVE_COLOR_CUSTOM, "CUSTOM", 0, "User Defined", ""},
+               {FCURVE_COLOR_AUTO_RAINBOW, "AUTO_RAINBOW", 0, "Auto Rainbow",
+                                           "Cycle through the rainbow, trying to give each curve a unique color"},
+               {FCURVE_COLOR_AUTO_RGB, "AUTO_RGB", 0, "Auto XYZ to RGB",
+                                       "Use axis colors for transform and color properties, and auto-rainbow for the rest"},
+               {FCURVE_COLOR_AUTO_YRGB, "AUTO_YRGB", 0, "Auto WXYZ to YRGB",
+                                        "Use axis colors for XYZ parts of transform, and yellow for the 'W' channel"},
+               {FCURVE_COLOR_CUSTOM, "CUSTOM", 0, "User Defined",
+                                     "Use custom hand-picked color for F-Curve"},
                {0, NULL, 0, NULL, NULL}
        };
 
@@ -1652,9 +1866,9 @@ static void rna_def_fcurve(BlenderRNA *brna)
        prop = RNA_def_property(srna, "extrapolation", PROP_ENUM, PROP_NONE);
        RNA_def_property_enum_sdna(prop, NULL, "extend");
        RNA_def_property_enum_items(prop, prop_mode_extend_items);
-       RNA_def_property_ui_text(prop, "Extrapolation", "");
-       /* XXX need an update callback for this so that animation gets evaluated */
-       RNA_def_property_update(prop, NC_ANIMATION, NULL);
+       RNA_def_property_ui_text(prop, "Extrapolation", 
+                                "Method used for evaluating value of F-Curve outside first and last keyframes");
+       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, "rna_FCurve_update_data");
 
        /* Pointers */
        prop = RNA_def_property(srna, "driver", PROP_POINTER, PROP_NONE);
@@ -1742,22 +1956,41 @@ static void rna_def_fcurve(BlenderRNA *brna)
        rna_def_fcurve_modifiers(brna, prop);
 
        /* Functions */
+       /* -- evaluate -- */
        func = RNA_def_function(srna, "evaluate", "evaluate_fcurve"); /* calls the C/API direct */
        RNA_def_function_ui_description(func, "Evaluate F-Curve");
        parm = RNA_def_float(func, "frame", 1.0f, -FLT_MAX, FLT_MAX, "Frame",
                             "Evaluate F-Curve at given frame", -FLT_MAX, FLT_MAX);
        RNA_def_property_flag(parm, PROP_REQUIRED);
        /* return value */
-       parm = RNA_def_float(func, "position", 0, -FLT_MAX, FLT_MAX, "Position", "F-Curve position", -FLT_MAX, FLT_MAX);
+       parm = RNA_def_float(func, "value", 0, -FLT_MAX, FLT_MAX, "Value", "Value of F-Curve specific frame", -FLT_MAX, FLT_MAX);
        RNA_def_function_return(func, parm);
-
-       func = RNA_def_function(srna, "range", "rna_fcurve_range");
+       
+       /* -- update / recalculate -- */
+       func = RNA_def_function(srna, "update", "rna_FCurve_update_data_ex");
+       RNA_def_function_ui_description(func, "Ensure keyframes are sorted in chronological order and handles are set correctly");
+       
+       /* -- time extents/range -- */
+       func = RNA_def_function(srna, "range", "rna_FCurve_range");
        RNA_def_function_ui_description(func, "Get the time extents for F-Curve");
        /* return value */
        parm = RNA_def_float_vector(func, "range", 2, NULL, -FLT_MAX, FLT_MAX, "Range",
                                    "Min/Max values", -FLT_MAX, FLT_MAX);
        RNA_def_property_flag(parm, PROP_THICK_WRAP);
        RNA_def_function_output(func, parm);
+       
+       /* -- auto-flag validity (ensures valid handling for data type) -- */
+       func = RNA_def_function(srna, "update_autoflags", "update_autoflags_fcurve"); /* calls the C/API direct */
+       RNA_def_function_ui_description(func, "Update FCurve flags set automatically from affected property "
+                                             "(currently, integer/discrete flags set when the property is not a float)");
+       RNA_def_function_flag(func, FUNC_USE_CONTEXT | FUNC_USE_REPORTS);
+       parm = RNA_def_pointer(func, "data", "AnyType", "Data",
+                              "Data containing the property controlled by given FCurve");
+       RNA_def_property_flag(parm, PROP_REQUIRED | PROP_RNAPTR | PROP_NEVER_NULL);
+
+
+       /* Functions */
+       RNA_api_fcurves(srna);
 }
 
 /* *********************** */