merge with/from trunk at r35190
[blender.git] / source / blender / makesrna / intern / rna_fcurve.c
index d5d28ac83ae3dc380ec77f7924ab757c8f240635..72f74f5a3e9f0c23f0b8f323329a00c14f10e131 100644 (file)
@@ -1,4 +1,4 @@
-/**
+/*
  * $Id$
  *
  * ***** BEGIN GPL LICENSE BLOCK *****
@@ -26,7 +26,6 @@
 
 #include "RNA_access.h"
 #include "RNA_define.h"
-#include "RNA_types.h"
 #include "RNA_enum_types.h"
 
 #include "rna_internal.h"
 
 #include "BLI_math.h"
 
+#include "BKE_action.h"
+
 #include "WM_types.h"
 
+#include "ED_keyframing.h"
+#include "ED_keyframes_edit.h"
 EnumPropertyItem fmodifier_type_items[] = {
        {FMODIFIER_TYPE_NULL, "NULL", 0, "Invalid", ""},
        {FMODIFIER_TYPE_GENERATOR, "GENERATOR", 0, "Generator", ""},
@@ -49,14 +53,16 @@ EnumPropertyItem fmodifier_type_items[] = {
        {FMODIFIER_TYPE_CYCLES, "CYCLES", 0, "Cycles", ""},
        {FMODIFIER_TYPE_NOISE, "NOISE", 0, "Noise", ""},
        {FMODIFIER_TYPE_FILTER, "FILTER", 0, "Filter", ""},
-       {FMODIFIER_TYPE_PYTHON, "PYTHON", 0, "Python", ""},
+       //{FMODIFIER_TYPE_PYTHON, "PYTHON", 0, "Python", ""},   // FIXME: not implemented yet!
        {FMODIFIER_TYPE_LIMITS, "LIMITS", 0, "Limits", ""},
+       {FMODIFIER_TYPE_STEPPED, "STEPPED", 0, "Stepped Interpolation", ""},
        {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", ""},
        {0, NULL, 0, NULL, NULL}};
 
 #ifdef RNA_RUNTIME
@@ -84,6 +90,8 @@ static StructRNA *rna_FModifierType_refine(struct PointerRNA *ptr)
                        return &RNA_FModifierPython;
                case FMODIFIER_TYPE_LIMITS:
                        return &RNA_FModifierLimits;
+               case FMODIFIER_TYPE_STEPPED:
+                       return &RNA_FModifierStepped;
                default:
                        return &RNA_UnknownType;
        }
@@ -103,8 +111,8 @@ 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(scene);
-       DAG_id_flush_update(id, OB_RECALC_OB|OB_RECALC_DATA);
+       DAG_scene_sort(bmain, scene);
+       DAG_id_tag_update(id, OB_RECALC_OB|OB_RECALC_DATA);
        
        WM_main_add_notifier(NC_SCENE|ND_FRAME, scene);
 }
@@ -148,6 +156,13 @@ static void rna_DriverTarget_update_name(Main *bmain, Scene *scene, PointerRNA *
 
 /* ----------- */
 
+/* note: this function exists only to avoid id refcounting */
+static void rna_DriverTarget_id_set(PointerRNA *ptr, PointerRNA value)
+{
+    DriverTarget *dtar= (DriverTarget*)ptr->data;
+    dtar->id= value.data;
+}
+
 static StructRNA *rna_DriverTarget_id_typef(PointerRNA *ptr)
 {
        DriverTarget *dtar= (DriverTarget*)ptr->data;
@@ -310,21 +325,91 @@ static void rna_FCurve_RnaPath_set(PointerRNA *ptr, const char *value)
        if (fcu->rna_path)
                MEM_freeN(fcu->rna_path);
        
-       if (strlen(value))
+       if (strlen(value)) {
                fcu->rna_path= BLI_strdup(value);
+               fcu->flag &= ~FCURVE_DISABLED;
+       }
        else 
                fcu->rna_path= NULL;
 }
 
+static void rna_FCurve_group_set(PointerRNA *ptr, PointerRNA value)
+{
+       ID *pid = (ID *)ptr->id.data;
+       ID *vid = (ID *)value.id.data;
+       FCurve *fcu= ptr->data;
+       bAction *act = NULL;
+       
+       /* get action */
+       if (ELEM(NULL, pid, vid)) {
+               printf("ERROR: one of the ID's for the groups to assign to is invalid (ptr=%p, val=%p)\n", pid, vid);
+               return;
+       }
+       else if (value.data && (pid != vid)) {
+               /* id's differ, cant do this, should raise an error */
+               printf("ERROR: ID's differ - ptr=%p vs value=%p \n", pid, vid);
+               return;
+       }
+       
+       if (GS(pid->name)==ID_AC && GS(vid->name)==ID_AC) {
+               /* the ID given is the action already - usually when F-Curve is obtained from an action's pointer */
+               act = (bAction *)pid;
+       }
+       else {
+               /* the ID given is the owner of the F-Curve (for drivers) */
+               AnimData *adt = BKE_animdata_from_id(ptr->id.data);
+               act = (adt)? adt->action : NULL;
+       }
+       
+       /* already belongs to group? */
+       if (fcu->grp == value.data) {
+               /* nothing to do */
+               printf("ERROR: F-Curve already belongs to this group\n");
+               return; 
+       }
+       
+       /* can only change group if we have info about the action the F-Curve is in 
+        * (i.e. for drivers or random F-Curves, this cannot be done)
+        */
+       if (act == NULL) {
+               /* can't change the grouping of F-Curve when it doesn't belong to an action */
+               printf("ERROR: cannot assign F-Curve to group, since F-Curve is not attached to any ID\n");
+               return;
+       }
+       /* make sure F-Curve exists in this action first, otherwise we could still have been tricked */
+       else if (BLI_findindex(&act->curves, fcu) == -1) {
+               printf("ERROR: F-Curve (%p) doesn't exist in action '%s'\n", fcu, act->id.name);
+               return;
+       }
+       
+       /* try to remove F-Curve from action (including from any existing groups) */
+       action_groups_remove_channel(act, fcu);
+       
+       /* add the F-Curve back to the action now in the right place */
+       // TODO: make the api function handle the case where there isn't any group to assign to 
+       if (value.data) {
+               /* add to its group using API function, which makes sure everything goes ok */
+               action_groups_add_channel(act, value.data, fcu);
+       }
+       else {
+               /* need to add this back, but it can only go at the end of the list (or else will corrupt groups) */
+               BLI_addtail(&act->curves, fcu);
+       }
+}
+
 DriverVar *rna_Driver_new_variable(ChannelDriver *driver)
 {
        /* call the API function for this */
        return driver_add_new_variable(driver);
 }
 
-void rna_Driver_remove_variable(ChannelDriver *driver, DriverVar *dvar)
+void rna_Driver_remove_variable(ChannelDriver *driver, ReportList *reports, DriverVar *dvar)
 {
-       /* call the API function for this */
+       if(BLI_findindex(&driver->variables, dvar) == -1) {
+               BKE_report(reports, RPT_ERROR, "Variable does not exist in this driver.");
+               return;
+       }
+
        driver_free_variable(driver, dvar);
 }
 
@@ -342,14 +427,18 @@ static void rna_FCurve_active_modifier_set(PointerRNA *ptr, PointerRNA value)
        set_active_fmodifier(&fcu->modifiers, (FModifier *)value.data);
 }
 
-static FModifier *rna_FCurve_modifiers_new(FCurve *fcu, bContext *C, int type)
+static FModifier *rna_FCurve_modifiers_new(FCurve *fcu, int type)
 {
        return add_fmodifier(&fcu->modifiers, type);
 }
 
-static int rna_FCurve_modifiers_remove(FCurve *fcu, bContext *C, int index)
+static void rna_FCurve_modifiers_remove(FCurve *fcu, ReportList *reports, FModifier *fcm)
 {
-       return remove_fmodifier_index(&fcu->modifiers, index);
+       if(BLI_findindex(&fcu->modifiers, fcm) == -1) {
+               BKE_reportf(reports, RPT_ERROR, "FCurveModifier '%s' not found in fcurve.", fcm->name);
+               return;
+       }
+       remove_fmodifier(&fcu->modifiers, fcm);
 }
 
 static void rna_FModifier_active_set(PointerRNA *ptr, int value)
@@ -403,6 +492,100 @@ 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)
+{
+       FModifier *fcm= (FModifier*)ptr->data;
+       FMod_Limits *data= fcm->data;
+
+       *min= MINAFRAMEF;
+       *max= (data->flag & FCM_LIMIT_XMAX)? data->rect.xmax : MAXFRAMEF;
+}
+
+static void rna_FModifierLimits_maxx_range(PointerRNA *ptr, float *min, float *max)
+{
+       FModifier *fcm= (FModifier*)ptr->data;
+       FMod_Limits *data= fcm->data;
+
+       *min= (data->flag & FCM_LIMIT_XMIN)? data->rect.xmin : MINAFRAMEF;
+       *max= MAXFRAMEF;
+}
+
+static void rna_FModifierLimits_miny_range(PointerRNA *ptr, float *min, float *max)
+{
+       FModifier *fcm= (FModifier*)ptr->data;
+       FMod_Limits *data= fcm->data;
+
+       *min= -FLT_MAX;
+       *max= (data->flag & FCM_LIMIT_YMAX)? data->rect.ymax : FLT_MAX;
+}
+
+static void rna_FModifierLimits_maxy_range(PointerRNA *ptr, float *min, float *max)
+{
+       FModifier *fcm= (FModifier*)ptr->data;
+       FMod_Limits *data= fcm->data;
+
+       *min= (data->flag & FCM_LIMIT_YMIN)? data->rect.ymin : -FLT_MAX;
+       *max= FLT_MAX;
+}
+
+
+static void rna_FModifierStepped_start_frame_range(PointerRNA *ptr, float *min, float *max)
+{
+       FModifier *fcm= (FModifier*)ptr->data;
+       FMod_Stepped *data= fcm->data;
+       
+       *min= MINAFRAMEF;
+       *max= (data->flag & FCM_STEPPED_NO_AFTER)? data->end_frame : MAXFRAMEF;
+}
+
+static void rna_FModifierStepped_end_frame_range(PointerRNA *ptr, float *min, float *max)
+{
+       FModifier *fcm= (FModifier*)ptr->data;
+       FMod_Stepped *data= fcm->data;
+
+       *min= (data->flag & FCM_STEPPED_NO_BEFORE)? data->start_frame : MINAFRAMEF;
+       *max= MAXFRAMEF;
+}
+
+static BezTriple *rna_FKeyframe_points_insert(FCurve *fcu, float frame, float value, int flag)
+{
+       int index= insert_vert_fcurve(fcu, frame, value, flag);
+       return ((fcu->bezt) && (index >= 0))? (fcu->bezt + index) : NULL;
+}
+
+static void rna_FKeyframe_points_add(FCurve *fcu, int tot)
+{
+       if(tot > 0) {
+               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->totvert += tot;
+       }
+}
+
+static void rna_FKeyframe_points_remove(FCurve *fcu, ReportList *reports, BezTriple *bezt, int do_fast)
+{
+       int index= (int)(bezt - fcu->bezt);
+       if (index < 0 || index >= fcu->totvert) {
+               BKE_report(reports, RPT_ERROR, "Keyframe not in F-Curve.");
+               return;
+       }
+
+       delete_fcurve_key(fcu, index, !do_fast);
+}
+
+static void rna_fcurve_range(FCurve *fcu, float range[2])
+{
+       calc_fcurve_range(fcu, range, range+1);
+}
+
 #else
 
 static void rna_def_fmodifier_generator(BlenderRNA *brna)
@@ -420,23 +603,23 @@ static void rna_def_fmodifier_generator(BlenderRNA *brna)
        RNA_def_struct_sdna_from(srna, "FMod_Generator", "data");
        
        /* define common props */
-       prop= RNA_def_property(srna, "additive", PROP_BOOLEAN, PROP_NONE);
+       prop= RNA_def_property(srna, "use_additive", PROP_BOOLEAN, PROP_NONE);
        RNA_def_property_boolean_sdna(prop, NULL, "flag", FCM_GENERATOR_ADDITIVE);
        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_EDIT, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION|ND_KEYFRAME|NA_EDITED, NULL);
        
                // 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_EDIT, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION|ND_KEYFRAME|NA_EDITED, NULL);
        
        
        /* 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_EDIT, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION|ND_KEYFRAME|NA_EDITED, NULL);
        
        /* coefficients array */
        prop= RNA_def_property(srna, "coefficients", PROP_FLOAT, PROP_NONE);
@@ -470,31 +653,31 @@ 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_EDIT, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION|ND_KEYFRAME|NA_EDITED, NULL);
        
        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_EDIT, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION|ND_KEYFRAME|NA_EDITED, NULL);
        
        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_EDIT, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION|ND_KEYFRAME|NA_EDITED, NULL);
        
        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_EDIT, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION|ND_KEYFRAME|NA_EDITED, NULL);
        
        /* flags */
-       prop= RNA_def_property(srna, "additive", PROP_BOOLEAN, PROP_NONE);
+       prop= RNA_def_property(srna, "use_additive", PROP_BOOLEAN, PROP_NONE);
        RNA_def_property_boolean_sdna(prop, NULL, "flag", FCM_GENERATOR_ADDITIVE);
        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_EDIT, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION|ND_KEYFRAME|NA_EDITED, NULL);
        
        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_EDIT, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION|ND_KEYFRAME|NA_EDITED, NULL);
 }
 
 /* --------- */
@@ -512,21 +695,21 @@ static void rna_def_fmodifier_envelope_ctrl(BlenderRNA *brna)
         *      - for now, these are allowed to go past each other, so that we can have inverted action
         *      - technically, the range is limited by the settings in the envelope-modifier data, not here...
         */
-       prop= RNA_def_property(srna, "minimum", PROP_FLOAT, PROP_NONE);
+       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_EDIT, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION|ND_KEYFRAME|NA_EDITED, NULL);
        
-       prop= RNA_def_property(srna, "maximum", PROP_FLOAT, PROP_NONE);
+       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_EDIT, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION|ND_KEYFRAME|NA_EDITED, NULL);
        
        /* 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_EDIT, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION|ND_KEYFRAME|NA_EDITED, NULL);
        
        // TODO:
        //      - selection flags (not implemented in UI yet though)
@@ -551,17 +734,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_EDIT, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION|ND_KEYFRAME|NA_EDITED, NULL);
        
-       prop= RNA_def_property(srna, "default_minimum", PROP_FLOAT, PROP_NONE);
+       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_EDIT, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION|ND_KEYFRAME|NA_EDITED, NULL);
        
-       prop= RNA_def_property(srna, "default_maximum", PROP_FLOAT, PROP_NONE);
+       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_EDIT, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION|ND_KEYFRAME|NA_EDITED, NULL);
 }
 
 /* --------- */
@@ -583,24 +766,28 @@ static void rna_def_fmodifier_cycles(BlenderRNA *brna)
        RNA_def_struct_sdna_from(srna, "FMod_Cycles", "data");
        
        /* before */
-       prop= RNA_def_property(srna, "before_mode", PROP_ENUM, PROP_NONE);
+       prop= RNA_def_property(srna, "mode_before", PROP_ENUM, PROP_NONE);
+       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_EDIT, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION|ND_KEYFRAME|NA_EDITED, NULL);
        
-       prop= RNA_def_property(srna, "before_cycles", PROP_FLOAT, PROP_NONE);
+       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_EDIT, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION|ND_KEYFRAME|NA_EDITED, NULL);
        
        /* after */
-       prop= RNA_def_property(srna, "after_mode", PROP_ENUM, PROP_NONE);
+       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_EDIT, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION|ND_KEYFRAME|NA_EDITED, NULL);
        
-       prop= RNA_def_property(srna, "after_cycles", PROP_FLOAT, PROP_NONE);
+       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_EDIT, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION|ND_KEYFRAME|NA_EDITED, NULL);
 }
 
 /* --------- */
@@ -626,45 +813,49 @@ static void rna_def_fmodifier_limits(BlenderRNA *brna)
        RNA_def_struct_ui_text(srna, "Limits F-Modifier", "Limits the time/value ranges of the modified F-Curve");
        RNA_def_struct_sdna_from(srna, "FMod_Limits", "data");
        
-       prop= RNA_def_property(srna, "use_minimum_x", PROP_BOOLEAN, PROP_NONE);
+       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_EDIT, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION|ND_KEYFRAME|NA_EDITED, NULL);
        
-       prop= RNA_def_property(srna, "use_minimum_y", PROP_BOOLEAN, PROP_NONE);
+       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_EDIT, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION|ND_KEYFRAME|NA_EDITED, NULL);
        
-       prop= RNA_def_property(srna, "use_maximum_x", PROP_BOOLEAN, PROP_NONE);
+       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_EDIT, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION|ND_KEYFRAME|NA_EDITED, NULL);
        
-       prop= RNA_def_property(srna, "use_maximum_y", PROP_BOOLEAN, PROP_NONE);
+       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_EDIT, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION|ND_KEYFRAME|NA_EDITED, NULL);
        
-       prop= RNA_def_property(srna, "minimum_x", PROP_FLOAT, PROP_NONE);
+       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_ui_text(prop, "Minimum X", "Lowest X value to allow");
-       RNA_def_property_update(prop, NC_ANIMATION|ND_KEYFRAME_EDIT, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION|ND_KEYFRAME|NA_EDITED, NULL);
        
-       prop= RNA_def_property(srna, "minimum_y", PROP_FLOAT, PROP_NONE);
+       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_ui_text(prop, "Minimum Y", "Lowest Y value to allow");
-       RNA_def_property_update(prop, NC_ANIMATION|ND_KEYFRAME_EDIT, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION|ND_KEYFRAME|NA_EDITED, NULL);
        
-       prop= RNA_def_property(srna, "maximum_x", PROP_FLOAT, PROP_NONE);
+       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_ui_text(prop, "Maximum X", "Highest X value to allow");
-       RNA_def_property_update(prop, NC_ANIMATION|ND_KEYFRAME_EDIT, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION|ND_KEYFRAME|NA_EDITED, NULL);
        
-       prop= RNA_def_property(srna, "maximum_y", PROP_FLOAT, PROP_NONE);
+       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_ui_text(prop, "Maximum Y", "Highest Y value to allow");
-       RNA_def_property_update(prop, NC_ANIMATION|ND_KEYFRAME_EDIT, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION|ND_KEYFRAME|NA_EDITED, NULL);
 }
 
 /* --------- */
@@ -685,36 +876,82 @@ static void rna_def_fmodifier_noise(BlenderRNA *brna)
        RNA_def_struct_ui_text(srna, "Noise F-Modifier", "Gives randomness to the modified F-Curve");
        RNA_def_struct_sdna_from(srna, "FMod_Noise", "data");
        
-       prop= RNA_def_property(srna, "modification", PROP_ENUM, PROP_NONE);
+       prop= RNA_def_property(srna, "blend_type", PROP_ENUM, PROP_NONE);
+       RNA_def_property_enum_sdna(prop, NULL, "modification");
        RNA_def_property_enum_items(prop, prop_modification_items);
-       RNA_def_property_ui_text(prop, "Modification", "Method of modifying the existing F-Curve");
-       RNA_def_property_update(prop, NC_ANIMATION|ND_KEYFRAME_EDIT, NULL);
+       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);
        
-       prop= RNA_def_property(srna, "size", PROP_FLOAT, PROP_NONE);
+       prop= RNA_def_property(srna, "scale", PROP_FLOAT, PROP_NONE);
        RNA_def_property_float_sdna(prop, NULL, "size");
-       RNA_def_property_ui_text(prop, "Size", "Scaling (in time) of the noise");
-       RNA_def_property_update(prop, NC_ANIMATION|ND_KEYFRAME_EDIT, NULL);
+       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);
        
        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_EDIT, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION|ND_KEYFRAME|NA_EDITED, NULL);
        
        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_EDIT, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION|ND_KEYFRAME|NA_EDITED, NULL);
        
-       prop= RNA_def_property(srna, "depth", PROP_INT, PROP_NONE);
+       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_EDIT, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION|ND_KEYFRAME|NA_EDITED, NULL);
 
 }
 
+/* --------- */
+
+static void rna_def_fmodifier_stepped(BlenderRNA *brna)
+{
+       StructRNA *srna;
+       PropertyRNA *prop;
+       
+       srna= RNA_def_struct(brna, "FModifierStepped", "FModifier");
+       RNA_def_struct_ui_text(srna, "Stepped Interpolation F-Modifier", "Holds each interpolated value from the F-Curve for several frames without changing the timing");
+       RNA_def_struct_sdna_from(srna, "FMod_Stepped", "data");
+       
+       /* properties */
+       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);
+       
+       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);
+       
+       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);
+       
+       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);
+       
+       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);
+       
+       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);
+}
 
 /* --------- */
 
+
 static void rna_def_fmodifier(BlenderRNA *brna)
 {
        StructRNA *srna;
@@ -739,20 +976,20 @@ static void rna_def_fmodifier(BlenderRNA *brna)
        RNA_def_property_ui_text(prop, "Type", "F-Curve Modifier Type");
        
        /* settings */
-       prop= RNA_def_property(srna, "expanded", PROP_BOOLEAN, PROP_NONE);
+       prop= RNA_def_property(srna, "show_expanded", PROP_BOOLEAN, PROP_NONE);
        RNA_def_property_boolean_sdna(prop, NULL, "flag", FMODIFIER_FLAG_EXPANDED);
        RNA_def_property_ui_text(prop, "Expanded", "F-Curve Modifier's panel is expanded in UI");
        RNA_def_property_ui_icon(prop, ICON_TRIA_RIGHT, 1);
        
-       prop= RNA_def_property(srna, "muted", PROP_BOOLEAN, PROP_NONE);
+       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_ui_icon(prop, ICON_MUTE_IPO_OFF, 1);
        
-       prop= RNA_def_property(srna, "disabled", PROP_BOOLEAN, PROP_NONE);
+       prop= RNA_def_property(srna, "is_valid", PROP_BOOLEAN, PROP_NONE);
        RNA_def_property_clear_flag(prop, PROP_EDITABLE);
-       RNA_def_property_boolean_sdna(prop, NULL, "flag", FMODIFIER_FLAG_DISABLED);
+       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);
        
@@ -792,7 +1029,8 @@ static void rna_def_drivertarget(BlenderRNA *brna)
        RNA_def_property_struct_type(prop, "ID");
        RNA_def_property_flag(prop, PROP_EDITABLE);
        RNA_def_property_editable_func(prop, "rna_DriverTarget_id_editable");
-       RNA_def_property_pointer_funcs(prop, NULL, NULL, "rna_DriverTarget_id_typef");
+    /* note: custom set function is ONLY to avoid rna setting a user for this. */
+       RNA_def_property_pointer_funcs(prop, NULL, "rna_DriverTarget_id_set", "rna_DriverTarget_id_typef", NULL);
        RNA_def_property_ui_text(prop, "ID", "ID-block that the specific property used can be found from (id_type property must be set first)");
        RNA_def_property_update(prop, 0, "rna_DriverTarget_update_data");
        
@@ -822,7 +1060,7 @@ static void rna_def_drivertarget(BlenderRNA *brna)
        RNA_def_property_ui_text(prop, "Type", "Driver variable type");
        RNA_def_property_update(prop, 0, "rna_DriverTarget_update_data");
        
-       prop= RNA_def_property(srna, "use_local_space_transforms", PROP_BOOLEAN, PROP_NONE);
+       prop= RNA_def_property(srna, "use_local_space_transform", PROP_BOOLEAN, PROP_NONE);
        RNA_def_property_boolean_sdna(prop, NULL, "flag", DTAR_FLAG_LOCALSPACE);
        RNA_def_property_ui_text(prop, "Local Space", "Use transforms in Local Space (as opposed to the worldspace default)");
        RNA_def_property_update(prop, 0, "rna_DriverTarget_update_data");
@@ -891,10 +1129,11 @@ static void rna_def_channeldriver_variables(BlenderRNA *brna, PropertyRNA *cprop
 
        /* remove variable */
        func= RNA_def_function(srna, "remove", "rna_Driver_remove_variable");
-               RNA_def_function_ui_description(func, "Remove an existing variable from the driver.");
-               /* target to remove */
-       parm= RNA_def_pointer(func, "var", "DriverVariable", "", "Variable to remove from the driver.");
-               RNA_def_property_flag(parm, PROP_REQUIRED);
+       RNA_def_function_ui_description(func, "Remove an existing variable from the driver.");
+       RNA_def_function_flag(func, FUNC_USE_REPORTS);
+       /* target to remove */
+       parm= RNA_def_pointer(func, "variable", "DriverVariable", "", "Variable to remove from the driver.");
+       RNA_def_property_flag(parm, PROP_REQUIRED|PROP_NEVER_NULL);
 }
 
 static void rna_def_channeldriver(BlenderRNA *brna)
@@ -937,6 +1176,12 @@ 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");
        
+       /* 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);
+       RNA_def_property_ui_text(prop, "Invalid", "Driver could not be evaluated in past, so should be skipped");
+       
+       
        /* Functions */
        RNA_api_drivers(srna);
 }
@@ -953,17 +1198,17 @@ static void rna_def_fpoint(BlenderRNA *brna)
        RNA_def_struct_ui_text(srna, "F-Curve Sample", "Sample point for F-Curve");
        
        /* Boolean values */
-       prop= RNA_def_property(srna, "selected", PROP_BOOLEAN, PROP_NONE);
+       prop= RNA_def_property(srna, "select", PROP_BOOLEAN, PROP_NONE);
        RNA_def_property_boolean_sdna(prop, NULL, "flag", 1);
-       RNA_def_property_ui_text(prop, "Selected", "Selection status");
-       RNA_def_property_update(prop, NC_ANIMATION|ND_KEYFRAME_SELECT, NULL);
+       RNA_def_property_ui_text(prop, "Select", "Selection status");
+       RNA_def_property_update(prop, NC_ANIMATION|ND_KEYFRAME|NA_SELECTED, NULL);
        
        /* Vector value */
-       prop= RNA_def_property(srna, "point", PROP_FLOAT, PROP_XYZ);
+       prop= RNA_def_property(srna, "co", PROP_FLOAT, PROP_XYZ);
        RNA_def_property_float_sdna(prop, NULL, "vec");
        RNA_def_property_array(prop, 2);
        RNA_def_property_ui_text(prop, "Point", "Point coordinates");
-       RNA_def_property_update(prop, NC_ANIMATION|ND_KEYFRAME_EDIT, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION|ND_KEYFRAME|NA_EDITED, NULL);
 }
 
 
@@ -980,29 +1225,29 @@ static void rna_def_fkeyframe(BlenderRNA *brna)
        RNA_def_struct_ui_text(srna, "Keyframe", "Bezier curve point with two handles defining a Keyframe on an F-Curve");
        
        /* Boolean values */
-       prop= RNA_def_property(srna, "selected_handle1", PROP_BOOLEAN, PROP_NONE);
+       prop= RNA_def_property(srna, "select_left_handle", PROP_BOOLEAN, PROP_NONE);
        RNA_def_property_boolean_sdna(prop, NULL, "f1", 0);
        RNA_def_property_ui_text(prop, "Handle 1 selected", "Handle 1 selection status");
-       RNA_def_property_update(prop, NC_ANIMATION|ND_KEYFRAME_SELECT, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION|ND_KEYFRAME|NA_SELECTED, NULL);
        
-       prop= RNA_def_property(srna, "selected_handle2", PROP_BOOLEAN, PROP_NONE);
+       prop= RNA_def_property(srna, "select_right_handle", PROP_BOOLEAN, PROP_NONE);
        RNA_def_property_boolean_sdna(prop, NULL, "f3", 0);
        RNA_def_property_ui_text(prop, "Handle 2 selected", "Handle 2 selection status");
-       RNA_def_property_update(prop, NC_ANIMATION|ND_KEYFRAME_SELECT, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION|ND_KEYFRAME|NA_SELECTED, NULL);
        
-       prop= RNA_def_property(srna, "selected", PROP_BOOLEAN, PROP_NONE);
+       prop= RNA_def_property(srna, "select_control_point", PROP_BOOLEAN, PROP_NONE);
        RNA_def_property_boolean_sdna(prop, NULL, "f2", 0);
-       RNA_def_property_ui_text(prop, "Selected", "Control point selection status");
-       RNA_def_property_update(prop, NC_ANIMATION|ND_KEYFRAME_SELECT, NULL);
+       RNA_def_property_ui_text(prop, "Select", "Control point selection status");
+       RNA_def_property_update(prop, NC_ANIMATION|ND_KEYFRAME|NA_SELECTED, NULL);
        
        /* Enums */
-       prop= RNA_def_property(srna, "handle1_type", PROP_ENUM, PROP_NONE);
+       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, beztriple_handle_type_items);
        RNA_def_property_ui_text(prop, "Handle 1 Type", "Handle types");
        RNA_def_property_update(prop, NC_ANIMATION|ND_KEYFRAME_PROP, NULL);
        
-       prop= RNA_def_property(srna, "handle2_type", PROP_ENUM, PROP_NONE);
+       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, beztriple_handle_type_items);
        RNA_def_property_ui_text(prop, "Handle 2 Type", "Handle types");
@@ -1021,26 +1266,25 @@ static void rna_def_fkeyframe(BlenderRNA *brna)
        RNA_def_property_update(prop, NC_ANIMATION|ND_KEYFRAME_PROP, NULL);
        
        /* Vector values */
-       prop= RNA_def_property(srna, "handle1", PROP_FLOAT, PROP_TRANSLATION);
+       prop= RNA_def_property(srna, "handle_left", PROP_FLOAT, PROP_TRANSLATION);
        RNA_def_property_array(prop, 2);
        RNA_def_property_float_funcs(prop, "rna_FKeyframe_handle1_get", "rna_FKeyframe_handle1_set", NULL);
        RNA_def_property_ui_text(prop, "Handle 1", "Coordinates of the first handle");
-       RNA_def_property_update(prop, NC_ANIMATION|ND_KEYFRAME_EDIT, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION|ND_KEYFRAME|NA_EDITED, NULL);
        
        prop= RNA_def_property(srna, "co", PROP_FLOAT, PROP_TRANSLATION);
        RNA_def_property_array(prop, 2);
        RNA_def_property_float_funcs(prop, "rna_FKeyframe_ctrlpoint_get", "rna_FKeyframe_ctrlpoint_set", NULL);
        RNA_def_property_ui_text(prop, "Control Point", "Coordinates of the control point");
-       RNA_def_property_update(prop, NC_ANIMATION|ND_KEYFRAME_EDIT, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION|ND_KEYFRAME|NA_EDITED, NULL);
        
-       prop= RNA_def_property(srna, "handle2", PROP_FLOAT, PROP_TRANSLATION);
+       prop= RNA_def_property(srna, "handle_right", PROP_FLOAT, PROP_TRANSLATION);
        RNA_def_property_array(prop, 2);
        RNA_def_property_float_funcs(prop, "rna_FKeyframe_handle2_get", "rna_FKeyframe_handle2_set", NULL);
        RNA_def_property_ui_text(prop, "Handle 2", "Coordinates of the second handle");
-       RNA_def_property_update(prop, NC_ANIMATION|ND_KEYFRAME_EDIT, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION|ND_KEYFRAME|NA_EDITED, NULL);
 }
 
-
 static void rna_def_fcurve_modifiers(BlenderRNA *brna, PropertyRNA *cprop)
 {
        /* add modifiers */
@@ -1059,13 +1303,12 @@ static void rna_def_fcurve_modifiers(BlenderRNA *brna, PropertyRNA *cprop)
        /* Collection active property */
        prop= RNA_def_property(srna, "active", PROP_POINTER, PROP_NONE);
        RNA_def_property_struct_type(prop, "FModifier");
-       RNA_def_property_pointer_funcs(prop, "rna_FCurve_active_modifier_get", "rna_FCurve_active_modifier_set", NULL);
+       RNA_def_property_pointer_funcs(prop, "rna_FCurve_active_modifier_get", "rna_FCurve_active_modifier_set", NULL, NULL);
        RNA_def_property_flag(prop, PROP_EDITABLE);
        RNA_def_property_ui_text(prop, "Active F-Curve Modifier", "Active F-Curve Modifier");
 
        /* Constraint collection */
        func= RNA_def_function(srna, "new", "rna_FCurve_modifiers_new");
-       RNA_def_function_flag(func, FUNC_USE_CONTEXT);
        RNA_def_function_ui_description(func, "Add a constraint to this object");
        /* return type */
        parm= RNA_def_pointer(func, "fmodifier", "FModifier", "", "New fmodifier.");
@@ -1075,21 +1318,64 @@ static void rna_def_fcurve_modifiers(BlenderRNA *brna, PropertyRNA *cprop)
        RNA_def_property_flag(parm, PROP_REQUIRED);
 
        func= RNA_def_function(srna, "remove", "rna_FCurve_modifiers_remove");
-       RNA_def_function_flag(func, FUNC_USE_CONTEXT);
+       RNA_def_function_flag(func, FUNC_USE_REPORTS);
        RNA_def_function_ui_description(func, "Remove a modifier from this fcurve.");
-       /* return type */
-       parm= RNA_def_boolean(func, "success", 0, "Success", "Removed the constraint successfully.");
-       RNA_def_function_return(func, parm);
-       /* object to add */
-       parm= RNA_def_int(func, "index", 0, 0, INT_MAX, "Index", "", 0, INT_MAX);
+       /* modifier to remove */
+       parm= RNA_def_pointer(func, "modifier", "FModifier", "", "Removed modifier.");
+       RNA_def_property_flag(parm, PROP_REQUIRED|PROP_NEVER_NULL);
+}
+
+/* fcurve.keyframe_points */
+static void rna_def_fcurve_keyframe_points(BlenderRNA *brna, PropertyRNA *cprop)
+{
+       StructRNA *srna;
+
+       FunctionRNA *func;
+       PropertyRNA *parm;
+
+       static EnumPropertyItem keyframe_flag_items[] = {
+               {INSERTKEY_REPLACE, "REPLACE", 0, "Replace", "Don't add any new keyframes, but just replace existing ones"},
+               {INSERTKEY_NEEDED, "NEEDED", 0, "Needed", "Only adds keyframes that are needed"},
+               {INSERTKEY_FAST, "FAST", 0, "Fast", "Fast keyframe insertion to avoid recalculating the curve each time"},
+               {0, NULL, 0, NULL, NULL}};
+
+       RNA_def_property_srna(cprop, "FCurveKeyframePoints");
+       srna= RNA_def_struct(brna, "FCurveKeyframePoints", NULL);
+       RNA_def_struct_sdna(srna, "FCurve");
+       RNA_def_struct_ui_text(srna, "Keyframe Points", "Collection of keyframe points");
+
+       func= RNA_def_function(srna, "insert", "rna_FKeyframe_points_insert");
+       RNA_def_function_ui_description(func, "Add a keyframe point to a F-Curve.");
+       parm= RNA_def_float(func, "frame", 0.0f, -FLT_MAX, FLT_MAX, "", "X Value of this keyframe point", -FLT_MAX, FLT_MAX);
+       RNA_def_property_flag(parm, PROP_REQUIRED);
+       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.");
+
+       parm= RNA_def_pointer(func, "keyframe", "Keyframe", "", "Newly created keyframe");
+       RNA_def_function_return(func, parm);
+
+       func= RNA_def_function(srna, "add", "rna_FKeyframe_points_add");
+       RNA_def_function_ui_description(func, "Add a keyframe point to a F-Curve.");
+       RNA_def_int(func, "count", 1, 1, INT_MAX, "Number", "Number of points to add to the spline", 1, INT_MAX);
+
+       func= RNA_def_function(srna, "remove", "rna_FKeyframe_points_remove");
+       RNA_def_function_ui_description(func, "Remove keyframe from an fcurve.");
+       RNA_def_function_flag(func, FUNC_USE_REPORTS);
+       parm= RNA_def_pointer(func, "keyframe", "Keyframe", "", "Keyframe to remove.");
+       RNA_def_property_flag(parm, PROP_REQUIRED|PROP_NEVER_NULL);
+       /* optional */
+       RNA_def_boolean(func, "fast", 0, "Fast", "Fast keyframe removal to avoid recalculating the curve each time");
 }
 
 static void rna_def_fcurve(BlenderRNA *brna)
 {
        StructRNA *srna;
        PropertyRNA *prop;
-       
+       FunctionRNA *func;
+       PropertyRNA *parm;
+
        static EnumPropertyItem prop_mode_extend_items[] = {
                {FCURVE_EXTRAPOLATE_CONSTANT, "CONSTANT", 0, "Constant", ""},
                {FCURVE_EXTRAPOLATE_LINEAR, "LINEAR", 0, "Linear", ""},
@@ -1118,10 +1404,11 @@ static void rna_def_fcurve(BlenderRNA *brna)
        
        prop= RNA_def_property(srna, "group", PROP_POINTER, PROP_NONE);
        RNA_def_property_pointer_sdna(prop, NULL, "grp");
-       RNA_def_property_clear_flag(prop, PROP_EDITABLE); // XXX this is not editable for now, since editing this will easily break the visible hierarchy
+       RNA_def_property_flag(prop, PROP_EDITABLE);
        RNA_def_property_ui_text(prop, "Group", "Action Group that this F-Curve belongs to");
-       RNA_def_property_update(prop, NC_ANIMATION|ND_FCURVES_ORDER, NULL);
-       
+       RNA_def_property_pointer_funcs(prop, NULL, "rna_FCurve_group_set", NULL, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION, NULL);
+
        /* Path + Array Index */
        prop= RNA_def_property(srna, "data_path", PROP_STRING, PROP_NONE);
        RNA_def_property_string_funcs(prop, "rna_FCurve_RnaPath_get", "rna_FCurve_RnaPath_length", "rna_FCurve_RnaPath_set");
@@ -1140,34 +1427,41 @@ static void rna_def_fcurve(BlenderRNA *brna)
        
        prop= RNA_def_property(srna, "color", PROP_FLOAT, PROP_COLOR);
        RNA_def_property_array(prop, 3);
+       RNA_def_property_range(prop, 0.0f, 1.0f);
        RNA_def_property_ui_text(prop, "Color", "Color of the F-Curve in the Graph Editor");
        RNA_def_property_update(prop, NC_ANIMATION, NULL);      
        
        /* Flags */
-       prop= RNA_def_property(srna, "selected", PROP_BOOLEAN, PROP_NONE);
+       prop= RNA_def_property(srna, "select", PROP_BOOLEAN, PROP_NONE);
        RNA_def_property_boolean_sdna(prop, NULL, "flag", FCURVE_SELECTED);
-       RNA_def_property_ui_text(prop, "Selected", "F-Curve is selected for editing");
-       RNA_def_property_update(prop, NC_ANIMATION|ND_KEYFRAME_SELECT, NULL);
+       RNA_def_property_ui_text(prop, "Select", "F-Curve is selected for editing");
+       RNA_def_property_update(prop, NC_ANIMATION|ND_KEYFRAME|NA_SELECTED, NULL);
        
-       prop= RNA_def_property(srna, "locked", PROP_BOOLEAN, PROP_NONE);
+       prop= RNA_def_property(srna, "lock", PROP_BOOLEAN, PROP_NONE);
        RNA_def_property_boolean_sdna(prop, NULL, "flag", FCURVE_PROTECTED);
-       RNA_def_property_ui_text(prop, "Locked", "F-Curve's settings cannot be edited");
-       RNA_def_property_update(prop, NC_ANIMATION|ND_ANIMCHAN_EDIT, NULL);
+       RNA_def_property_ui_text(prop, "Lock", "F-Curve's settings cannot be edited");
+       RNA_def_property_update(prop, NC_ANIMATION|ND_ANIMCHAN|NA_EDITED, NULL);
        
-       prop= RNA_def_property(srna, "muted", PROP_BOOLEAN, PROP_NONE);
+       prop= RNA_def_property(srna, "mute", PROP_BOOLEAN, PROP_NONE);
        RNA_def_property_boolean_sdna(prop, NULL, "flag", FCURVE_MUTED);
        RNA_def_property_ui_text(prop, "Muted", "F-Curve is not evaluated");
-       RNA_def_property_update(prop, NC_ANIMATION|ND_ANIMCHAN_EDIT, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION|ND_ANIMCHAN|NA_EDITED, NULL);
        
-       prop= RNA_def_property(srna, "auto_clamped_handles", PROP_BOOLEAN, PROP_NONE);
+       prop= RNA_def_property(srna, "use_auto_handle_clamp", PROP_BOOLEAN, PROP_NONE);
        RNA_def_property_boolean_sdna(prop, NULL, "flag", FCURVE_AUTO_HANDLES);
        RNA_def_property_ui_text(prop, "Auto Clamped Handles", "All auto-handles for F-Curve are clamped");
        RNA_def_property_update(prop, NC_ANIMATION|ND_KEYFRAME_PROP, NULL);
        
-       prop= RNA_def_property(srna, "visible", PROP_BOOLEAN, PROP_NONE);
-       RNA_def_property_boolean_sdna(prop, NULL, "flag", FCURVE_VISIBLE);
-       RNA_def_property_ui_text(prop, "Visible", "F-Curve and its keyframes are shown in the Graph Editor graphs");
+       prop= RNA_def_property(srna, "hide", PROP_BOOLEAN, PROP_NONE);
+       RNA_def_property_boolean_negative_sdna(prop, NULL, "flag", FCURVE_VISIBLE);
+       RNA_def_property_ui_text(prop, "Hide", "F-Curve and its keyframes are hidden in the Graph Editor graphs");
        RNA_def_property_update(prop, NC_SPACE|ND_SPACE_GRAPH, NULL);
+
+       /* State Info (for Debugging) */
+       prop= RNA_def_property(srna, "is_valid", PROP_BOOLEAN, PROP_NONE);
+       RNA_def_property_boolean_negative_sdna(prop, NULL, "flag", FCURVE_DISABLED);
+       RNA_def_property_ui_text(prop, "Valid", "False when F-Curve could not be evaluated in past, so should be skipped when evaluating");
+       RNA_def_property_update(prop, NC_ANIMATION|ND_KEYFRAME_PROP, NULL);
        
        /* Collections */
        prop= RNA_def_property(srna, "sampled_points", PROP_COLLECTION, PROP_NONE);
@@ -1179,12 +1473,28 @@ static void rna_def_fcurve(BlenderRNA *brna)
        RNA_def_property_collection_sdna(prop, NULL, "bezt", "totvert");
        RNA_def_property_struct_type(prop, "Keyframe");
        RNA_def_property_ui_text(prop, "Keyframes", "User-editable keyframes");
+       rna_def_fcurve_keyframe_points(brna, prop);
        
        prop= RNA_def_property(srna, "modifiers", PROP_COLLECTION, PROP_NONE);
        RNA_def_property_struct_type(prop, "FModifier");
        RNA_def_property_ui_text(prop, "Modifiers", "Modifiers affecting the shape of the F-Curve");
-
        rna_def_fcurve_modifiers(brna, prop);
+
+       /* Functions */
+       func= RNA_def_function(srna, "evaluate", "evaluate_fcurve"); /* calls the C/API direct */
+       RNA_def_function_ui_description(func, "Evaluate fcurve.");
+       parm= RNA_def_float(func, "frame", 1.0f, -FLT_MAX, FLT_MAX, "Frame", "Evaluate fcurve 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", "FCurve position", -FLT_MAX, FLT_MAX);
+       RNA_def_function_return(func, parm);
+
+       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);
 }
 
 /* *********************** */
@@ -1209,6 +1519,7 @@ void RNA_def_fcurve(BlenderRNA *brna)
        rna_def_fmodifier_python(brna);
        rna_def_fmodifier_limits(brna);
        rna_def_fmodifier_noise(brna);
+       rna_def_fmodifier_stepped(brna);
 }