Cleanup: use `rna_enum_` prefix for RNA enums
[blender.git] / source / blender / makesrna / intern / rna_fcurve.c
index 122e42e29e7e0981824d806a5016c0335f759536..398c9aa45820b31694f3882d917f081c3e9c4e19 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_FILTER, "FILTER", 0, "Filter", ""},
-       /*{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}};
+       {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", ""},
+       {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 rna_enum_beztriple_keyframe_type_items[] = {
+       {BEZT_KEYTYPE_KEYFRAME, "KEYFRAME", 0, "Keyframe", "Normal keyframe - e.g. for key poses"},
+       {BEZT_KEYTYPE_BREAKDOWN, "BREAKDOWN", 0, "Breakdown", "A breakdown pose - e.g. for transitions between key poses"},
+       {BEZT_KEYTYPE_EXTREME, "EXTREME", 0, "Extreme", "An 'extreme' pose, or some other purpose as needed"},
+       {BEZT_KEYTYPE_JITTER, "JITTER", 0, "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}
+};
 
 #ifdef RNA_RUNTIME
 
@@ -114,16 +135,20 @@ 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_id_tag_update(id, OB_RECALC_OB|OB_RECALC_DATA);
+       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);
+       WM_main_add_notifier(NC_SCENE | ND_FRAME, scene);
 }
 
 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);
 }
 
@@ -163,25 +188,25 @@ 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;
+       DriverTarget *dtar = (DriverTarget *)ptr->data;
        dtar->id = value.data;
 }
 
 static StructRNA *rna_DriverTarget_id_typef(PointerRNA *ptr)
 {
-       DriverTarget *dtar = (DriverTarget*)ptr->data;
+       DriverTarget *dtar = (DriverTarget *)ptr->data;
        return ID_code_to_RNA_type(dtar->idtype);
 }
 
 static int rna_DriverTarget_id_editable(PointerRNA *ptr)
 {
-       DriverTarget *dtar = (DriverTarget*)ptr->data;
-       return (dtar->idtype)? PROP_EDITABLE : 0;
+       DriverTarget *dtar = (DriverTarget *)ptr->data;
+       return (dtar->idtype) ? PROP_EDITABLE : 0;
 }
 
 static int rna_DriverTarget_id_type_editable(PointerRNA *ptr)
 {
-       DriverTarget *dtar = (DriverTarget*)ptr->data;
+       DriverTarget *dtar = (DriverTarget *)ptr->data;
        
        /* when the id-type can only be object, don't allow editing
         * otherwise, there may be strange crashes
@@ -191,7 +216,7 @@ static int rna_DriverTarget_id_type_editable(PointerRNA *ptr)
 
 static void rna_DriverTarget_id_type_set(PointerRNA *ptr, int value)
 {
-       DriverTarget *data = (DriverTarget*)(ptr->data);
+       DriverTarget *data = (DriverTarget *)(ptr->data);
        
        /* check if ID-type is settable */
        if ((data->flag & DTAR_FLAG_ID_OB_ONLY) == 0) {
@@ -250,11 +275,31 @@ static void rna_DriverVariable_type_set(PointerRNA *ptr, int value)
        driver_change_variable_type(dvar, value);
 }
 
+/* ----------- */
+
+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(driver, dvar);
+       RNA_POINTER_INVALIDATE(dvar_ptr);
+}
+
 /* ****************************** */
 
 static void rna_FKeyframe_handle1_get(PointerRNA *ptr, float *values)
 {
-       BezTriple *bezt = (BezTriple*)ptr->data;
+       BezTriple *bezt = (BezTriple *)ptr->data;
        
        values[0] = bezt->vec[0][0];
        values[1] = bezt->vec[0][1];
@@ -262,7 +307,7 @@ static void rna_FKeyframe_handle1_get(PointerRNA *ptr, float *values)
 
 static void rna_FKeyframe_handle1_set(PointerRNA *ptr, const float *values)
 {
-       BezTriple *bezt = (BezTriple*)ptr->data;
+       BezTriple *bezt = (BezTriple *)ptr->data;
        
        bezt->vec[0][0] = values[0];
        bezt->vec[0][1] = values[1];
@@ -270,7 +315,7 @@ static void rna_FKeyframe_handle1_set(PointerRNA *ptr, const float *values)
 
 static void rna_FKeyframe_handle2_get(PointerRNA *ptr, float *values)
 {
-       BezTriple *bezt = (BezTriple*)ptr->data;
+       BezTriple *bezt = (BezTriple *)ptr->data;
        
        values[0] = bezt->vec[2][0];
        values[1] = bezt->vec[2][1];
@@ -278,7 +323,7 @@ static void rna_FKeyframe_handle2_get(PointerRNA *ptr, float *values)
 
 static void rna_FKeyframe_handle2_set(PointerRNA *ptr, const float *values)
 {
-       BezTriple *bezt = (BezTriple*)ptr->data;
+       BezTriple *bezt = (BezTriple *)ptr->data;
        
        bezt->vec[2][0] = values[0];
        bezt->vec[2][1] = values[1];
@@ -286,7 +331,7 @@ static void rna_FKeyframe_handle2_set(PointerRNA *ptr, const float *values)
 
 static void rna_FKeyframe_ctrlpoint_get(PointerRNA *ptr, float *values)
 {
-       BezTriple *bezt = (BezTriple*)ptr->data;
+       BezTriple *bezt = (BezTriple *)ptr->data;
        
        values[0] = bezt->vec[1][0];
        values[1] = bezt->vec[1][1];
@@ -294,7 +339,7 @@ static void rna_FKeyframe_ctrlpoint_get(PointerRNA *ptr, float *values)
 
 static void rna_FKeyframe_ctrlpoint_set(PointerRNA *ptr, const float *values)
 {
-       BezTriple *bezt = (BezTriple*)ptr->data;
+       BezTriple *bezt = (BezTriple *)ptr->data;
        
        bezt->vec[1][0] = values[0];
        bezt->vec[1][1] = values[1];
@@ -362,7 +407,7 @@ static void rna_FCurve_group_set(PointerRNA *ptr, PointerRNA value)
        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;
+               act = (adt) ? adt->action : NULL;
        }
        
        /* already belongs to group? */
@@ -401,33 +446,38 @@ static void rna_FCurve_group_set(PointerRNA *ptr, PointerRNA value)
        }
 }
 
-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);
 }
 
-void rna_Driver_remove_variable(ChannelDriver *driver, ReportList *reports, DriverVar *dvar)
+
+/* allow scripts to update curve after editing manually */
+static void rna_FCurve_update_data_ex(FCurve *fcu)
 {
-       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 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);
 }
 
 
 static PointerRNA rna_FCurve_active_modifier_get(PointerRNA *ptr)
 {
-       FCurve *fcu = (FCurve*)ptr->data;
+       FCurve *fcu = (FCurve *)ptr->data;
        FModifier *fcm = find_active_fmodifier(&fcu->modifiers);
        return rna_pointer_inherit_refine(ptr, &RNA_FModifier, fcm);
 }
 
 static void rna_FCurve_active_modifier_set(PointerRNA *ptr, PointerRNA value)
 {
-       FCurve *fcu = (FCurve*)ptr->data;
+       FCurve *fcu = (FCurve *)ptr->data;
        set_active_fmodifier(&fcu->modifiers, (FModifier *)value.data);
 }
 
@@ -436,50 +486,104 @@ static FModifier *rna_FCurve_modifiers_new(FCurve *fcu, int type)
        return add_fmodifier(&fcu->modifiers, type);
 }
 
-static void rna_FCurve_modifiers_remove(FCurve *fcu, ReportList *reports, FModifier *fcm)
+static void rna_FCurve_modifiers_remove(FCurve *fcu, ReportList *reports, PointerRNA *fcm_ptr)
 {
+       FModifier *fcm = fcm_ptr->data;
        if (BLI_findindex(&fcu->modifiers, fcm) == -1) {
                BKE_reportf(reports, RPT_ERROR, "F-Curve modifier '%s' not found in F-Curve", fcm->name);
                return;
        }
+
        remove_fmodifier(&fcu->modifiers, fcm);
+       RNA_POINTER_INVALIDATE(fcm_ptr);
 }
 
 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;
+       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;
+       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;
+       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;
 
        *min = 0.0f;
        *max = fcm->efra - fcm->sfra;
 }
 
+static void rna_FModifier_verify_data_update(Main *UNUSED(bmain), Scene *UNUSED(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);
+}
+
 static void rna_FModifier_active_update(Main *UNUSED(bmain), Scene *UNUSED(scene), PointerRNA *ptr)
 {
-       FModifier *fm, *fmo = (FModifier*)ptr->data;
+       FModifier *fm, *fmo = (FModifier *)ptr->data;
 
        /* clear active state of other FModifiers in this list */
        for (fm = fmo->prev; fm; fm = fm->prev) {
@@ -493,107 +597,161 @@ static void rna_FModifier_active_update(Main *UNUSED(bmain), Scene *UNUSED(scene
 
 static int rna_FModifierGenerator_coefficients_get_length(PointerRNA *ptr, int length[RNA_MAX_ARRAY_DIMENSION])
 {
-       FModifier *fcm = (FModifier*)ptr->data;
+       FModifier *fcm = (FModifier *)ptr->data;
        FMod_Generator *gen = fcm->data;
 
        if (gen)
                length[0] = gen->arraysize;
        else
-               length[0] = 100; /* for raw_access, untested */
+               length[0] = 100;  /* for raw_access, untested */
 
        return length[0];
 }
 
 static void rna_FModifierGenerator_coefficients_get(PointerRNA *ptr, float *values)
 {
-       FModifier *fcm = (FModifier*)ptr->data;
+       FModifier *fcm = (FModifier *)ptr->data;
        FMod_Generator *gen = fcm->data;
        memcpy(values, gen->coefficients, gen->arraysize * sizeof(float));
 }
 
 static void rna_FModifierGenerator_coefficients_set(PointerRNA *ptr, const float *values)
 {
-       FModifier *fcm = (FModifier*)ptr->data;
+       FModifier *fcm = (FModifier *)ptr->data;
        FMod_Generator *gen = fcm->data;
        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;
+       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;
+       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;
+       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;
+       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;
+}
 
-       *min = (data->flag & FCM_LIMIT_YMIN)? data->rect.ymin : -FLT_MAX;
-       *max = FLT_MAX;
+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;
+}
+
+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;
+       FModifier *fcm = (FModifier *)ptr->data;
        FMod_Stepped *data = fcm->data;
        
        *min = MINAFRAMEF;
-       *max = (data->flag & FCM_STEPPED_NO_AFTER)? data->end_frame : MAXFRAMEF;
+       *max = (data->flag & FCM_STEPPED_NO_AFTER) ? data->end_frame : MAXFRAMEF;
 }
 
 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;
+       FModifier *fcm = (FModifier *)ptr->data;
        FMod_Stepped *data = fcm->data;
 
-       *min = (data->flag & FCM_STEPPED_NO_BEFORE)? data->start_frame : MINAFRAMEF;
+       *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;
+       int index = insert_vert_fcurve(fcu, frame, value, flag | INSERTKEY_NO_USERPREF);
+       return ((fcu->bezt) && (index >= 0)) ? (fcu->bezt + index) : NULL;
 }
 
 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;
@@ -608,8 +766,9 @@ static void rna_FKeyframe_points_add(FCurve *fcu, int tot)
        }
 }
 
-static void rna_FKeyframe_points_remove(FCurve *fcu, ReportList *reports, BezTriple *bezt, int do_fast)
+static void rna_FKeyframe_points_remove(FCurve *fcu, ReportList *reports, PointerRNA *bezt_ptr, int do_fast)
 {
+       BezTriple *bezt = bezt_ptr->data;
        int index = (int)(bezt - fcu->bezt);
        if (index < 0 || index >= fcu->totvert) {
                BKE_report(reports, RPT_ERROR, "Keyframe not in F-Curve");
@@ -617,11 +776,81 @@ static void rna_FKeyframe_points_remove(FCurve *fcu, ReportList *reports, BezTri
        }
 
        delete_fcurve_key(fcu, index, !do_fast);
+       RNA_POINTER_INVALIDATE(bezt_ptr);
 }
 
-static void rna_fcurve_range(FCurve *fcu, float range[2])
+static FCM_EnvelopeData *rna_FModifierEnvelope_points_add(FModifier *fmod, ReportList *reports, float frame)
 {
-       calc_fcurve_range(fcu, range, range+1, FALSE, FALSE);
+       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;
+
+       if (env->data) {
+               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);
+                       return NULL;
+               }
+
+               /* realloc memory for extra point */
+               env->data = (FCM_EnvelopeData *) MEM_reallocN((void *)env->data, (env->totvert + 1) * sizeof(FCM_EnvelopeData));
+
+               /* move the points after the added point */
+               if (i < env->totvert) {
+                       memmove(env->data + i + 1, env->data + i, (env->totvert - i) * sizeof(FCM_EnvelopeData));
+               }
+
+               env->totvert++;
+       }
+       else {
+               env->data = MEM_mallocN(sizeof(FCM_EnvelopeData), "FCM_EnvelopeData");
+               env->totvert = 1;
+               i = 0;
+       }
+
+       /* add point to paste at index i */
+       *(env->data + i) = fed;
+       return (env->data + i);
+}
+
+static void rna_FModifierEnvelope_points_remove(FModifier *fmod, ReportList *reports, PointerRNA *point)
+{
+       FCM_EnvelopeData *cp = point->data;
+       FMod_Envelope *env = (FMod_Envelope *)fmod->data;
+
+       int index = (int)(cp - env->data);
+
+       /* test point is in range */
+       if (index < 0 || index >= env->totvert) {
+               BKE_report(reports, RPT_ERROR, "Control point not in Envelope F-Modifier");
+               return;
+       }
+
+       if (env->totvert > 1) {
+               /* move data after the removed point */
+
+               memmove(env->data + index, env->data + (index + 1), sizeof(FCM_EnvelopeData) * ((env->totvert - index) - 1));
+
+               /* realloc smaller array */
+               env->totvert--;
+               env->data = (FCM_EnvelopeData *) MEM_reallocN((void *)env->data, (env->totvert) * sizeof(FCM_EnvelopeData));
+       }
+       else {
+               /* just free array, since the only vert was deleted */
+               if (env->data) {
+                       MEM_freeN(env->data);
+                       env->data = NULL;
+               }
+               env->totvert = 0;
+       }
+       RNA_POINTER_INVALIDATE(point);
 }
 
 #else
@@ -634,7 +863,8 @@ static void rna_def_fmodifier_generator(BlenderRNA *brna)
        static EnumPropertyItem generator_mode_items[] = {
                {FCM_GENERATOR_POLYNOMIAL, "POLYNOMIAL", 0, "Expanded Polynomial", ""},
                {FCM_GENERATOR_POLYNOMIAL_FACTORISED, "POLYNOMIAL_FACTORISED", 0, "Factorized Polynomial", ""},
-               {0, NULL, 0, NULL, NULL}};
+               {0, NULL, 0, NULL, NULL}
+       };
        
        srna = RNA_def_struct(brna, "FModifierGenerator", "FModifier");
        RNA_def_struct_ui_text(srna, "Generator F-Modifier", "Deterministically generate values for the modified F-Curve");
@@ -646,21 +876,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, 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|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);
@@ -685,8 +913,9 @@ static void rna_def_fmodifier_function_generator(BlenderRNA *brna)
                {2, "TAN", 0, "Tangent", ""},
                {3, "SQRT", 0, "Square Root", ""},
                {4, "LN", 0, "Natural Logarithm", ""},
-               {5, "SINC", 0, "Normalised Sine", "sin(x) / x"},
-               {0, NULL, 0, NULL, NULL}};
+               {5, "SINC", 0, "Normalized Sine", "sin(x) / x"},
+               {0, NULL, 0, NULL, NULL}
+       };
        
        srna = RNA_def_struct(brna, "FModifierFunctionGenerator", "FModifier");
        RNA_def_struct_ui_text(srna, "Built-In Function F-Modifier", "Generate values using a Built-In Function");
@@ -695,19 +924,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, 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|NA_EDITED, 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|NA_EDITED, 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|NA_EDITED, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, NULL);
        
        /* flags */
        prop = RNA_def_property(srna, "use_additive", PROP_BOOLEAN, PROP_NONE);
@@ -715,13 +944,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, 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|NA_EDITED, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, NULL);
 }
 
 /* --------- */
@@ -742,23 +971,53 @@ 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, NULL);
        
        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, 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|NA_EDITED, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, NULL);
        
        /* TODO: */
        /*      - selection flags (not implemented in UI yet though) */
 }
 
+static void rna_def_fmodifier_envelope_control_points(BlenderRNA *brna, PropertyRNA *cprop)
+{
+       StructRNA *srna;
+
+       FunctionRNA *func;
+       PropertyRNA *parm;
+
+       RNA_def_property_srna(cprop, "FModifierEnvelopeControlPoints");
+       srna = RNA_def_struct(brna, "FModifierEnvelopeControlPoints", NULL);
+       RNA_def_struct_sdna(srna, "FModifier");
+
+       RNA_def_struct_ui_text(srna, "Control Points", "Control points defining the shape of the envelope");
+
+       func = RNA_def_function(srna, "add", "rna_FModifierEnvelope_points_add");
+       RNA_def_function_ui_description(func, "Add a control point to a FModifierEnvelope");
+       RNA_def_function_flag(func, FUNC_USE_REPORTS);
+       parm = RNA_def_float(func, "frame", 0.0f, -FLT_MAX, FLT_MAX, "",
+                            "Frame to add this control-point", -FLT_MAX, FLT_MAX);
+       RNA_def_property_flag(parm, PROP_REQUIRED);
+       parm = RNA_def_pointer(func, "point", "FModifierEnvelopeControlPoint", "", "Newly created control-point");
+       RNA_def_function_return(func, parm);
+
+       func = RNA_def_function(srna, "remove", "rna_FModifierEnvelope_points_remove");
+       RNA_def_function_ui_description(func, "Remove a control-point from an FModifierEnvelope");
+       RNA_def_function_flag(func, FUNC_USE_REPORTS);
+       parm = RNA_def_pointer(func, "point", "FModifierEnvelopeControlPoint", "", "Control-point to remove");
+       RNA_def_property_flag(parm, PROP_REQUIRED | PROP_NEVER_NULL | PROP_RNAPTR);
+}
+
+
 static void rna_def_fmodifier_envelope(BlenderRNA *brna)
 {
        StructRNA *srna;
@@ -773,22 +1032,23 @@ static void rna_def_fmodifier_envelope(BlenderRNA *brna)
        RNA_def_property_collection_sdna(prop, NULL, "data", "totvert");
        RNA_def_property_struct_type(prop, "FModifierEnvelopeControlPoint");
        RNA_def_property_ui_text(prop, "Control Points", "Control points defining the shape of the envelope");
+       rna_def_fmodifier_envelope_control_points(brna, prop);
        
        /* Range Settings */
        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, NULL);
        
        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, NULL);
        
        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, NULL);
 }
 
 /* --------- */
@@ -806,7 +1066,8 @@ static void rna_def_fmodifier_cycles(BlenderRNA *brna)
                                                "start and end values"},
                {FCM_EXTRAPOLATE_MIRROR, "MIRROR", 0, "Repeat Mirrored",
                                         "Alternate between forward and reverse playback of keyframe range"},
-               {0, NULL, 0, NULL, NULL}};
+               {0, NULL, 0, NULL, NULL}
+       };
        
        srna = RNA_def_struct(brna, "FModifierCycles", "FModifier");
        RNA_def_struct_ui_text(srna, "Cycles F-Modifier", "Repeat the values of the modified F-Curve");
@@ -817,26 +1078,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, NULL);
        
        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, NULL);
        
        /* 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, NULL);
        
        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, NULL);
 }
 
 /* --------- */
@@ -865,46 +1126,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, NULL);
        
        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, NULL);
        
        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, NULL);
        
        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, NULL);
        
        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, NULL);
        
        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, NULL);
        
        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, NULL);
        
        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, NULL);
 }
 
 /* --------- */
@@ -919,7 +1180,8 @@ static void rna_def_fmodifier_noise(BlenderRNA *brna)
                {FCM_NOISE_MODIF_ADD, "ADD", 0, "Add", ""},
                {FCM_NOISE_MODIF_SUBTRACT, "SUBTRACT", 0, "Subtract", ""},
                {FCM_NOISE_MODIF_MULTIPLY, "MULTIPLY", 0, "Multiply", ""},
-               {0, NULL, 0, NULL, NULL}};
+               {0, NULL, 0, NULL, NULL}
+       };
        
        srna = RNA_def_struct(brna, "FModifierNoise", "FModifier");
        RNA_def_struct_ui_text(srna, "Noise F-Modifier", "Give randomness to the modified F-Curve");
@@ -929,28 +1191,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, NULL);
        
        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, 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|NA_EDITED, 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|NA_EDITED, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, NULL);
+
+       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, NULL);
        
        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, NULL);
 
 }
 
@@ -971,36 +1238,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, 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);
+       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);
+       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);
+       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);
+       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);
+       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, NULL);
 }
 
 /* --------- */
@@ -1017,7 +1284,7 @@ static void rna_def_fmodifier(BlenderRNA *brna)
        RNA_def_struct_ui_text(srna, "F-Modifier", "Modifier for values of F-Curve");
        
 #if 0 /* XXX not used yet */
-       /* name */
+         /* name */
        prop = RNA_def_property(srna, "name", PROP_STRING, PROP_NONE);
        RNA_def_struct_name_property(srna, prop);
        RNA_def_property_ui_text(prop, "Name", "Short description of F-Curve Modifier");
@@ -1026,7 +1293,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 */
@@ -1038,21 +1305,21 @@ 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, NULL);
        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, NULL);
        
-               /* TODO: setting this to true must ensure that all others in stack are turned off too... */
+       /* 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);
        RNA_def_property_boolean_sdna(prop, NULL, "flag", FMODIFIER_FLAG_ACTIVE);
        RNA_def_property_ui_text(prop, "Active", "F-Curve Modifier is the one being edited ");
        RNA_def_property_boolean_funcs(prop, NULL, "rna_FModifier_active_set");
-       RNA_def_property_update(prop, NC_ANIMATION|ND_KEYFRAME_PROP, "rna_FModifier_active_update");
+       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME_PROP, "rna_FModifier_active_update");
        RNA_def_property_ui_icon(prop, ICON_RADIOBUT_OFF, 1);
        
        /* restricted range */
@@ -1061,40 +1328,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, NULL);
        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, NULL);
        
        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, NULL);
        
        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, NULL);
        
        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, NULL);
        
        /* 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, NULL);
        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);
@@ -1103,7 +1370,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, NULL);
 }
 
 /* *********************** */
@@ -1123,16 +1390,18 @@ static void rna_def_drivertarget(BlenderRNA *brna)
                {DTAR_TRANSCHAN_SCALEX, "SCALE_X", 0, "X Scale", ""},
                {DTAR_TRANSCHAN_SCALEY, "SCALE_Y", 0, "Y Scale", ""},
                {DTAR_TRANSCHAN_SCALEZ, "SCALE_Z", 0, "Z Scale", ""},
-               {0, NULL, 0, NULL, NULL}};
+               {0, NULL, 0, NULL, NULL}
+       };
                
        static EnumPropertyItem prop_local_space_items[] = {
                {0, "WORLD_SPACE", 0, "World Space", "Transforms include effects of parenting/restpose and constraints"},
                {DTAR_FLAG_LOCALSPACE, "TRANSFORM_SPACE", 0, "Transform Space",
                                       "Transforms don't include parenting/restpose or constraints"},
-               {DTAR_FLAG_LOCALSPACE|DTAR_FLAG_LOCAL_CONSTS, "LOCAL_SPACE", 0, "Local Space",
-                                                             "Transforms include effects of constraints but not "
-                                                             "parenting/restpose"},
-               {0, NULL, 0, NULL, NULL}};
+               {DTAR_FLAG_LOCALSPACE | DTAR_FLAG_LOCAL_CONSTS, "LOCAL_SPACE", 0, "Local Space",
+                                                               "Transforms include effects of constraints but not "
+                                                               "parenting/restpose"},
+               {0, NULL, 0, NULL, NULL}
+       };
        
        srna = RNA_def_struct(brna, "DriverTarget", NULL);
        RNA_def_struct_ui_text(srna, "Driver Target", "Source of input values for driver variables");
@@ -1151,7 +1420,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");
@@ -1194,7 +1463,8 @@ static void rna_def_drivervar(BlenderRNA *brna)
                                           "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"},
-               {0, NULL, 0, NULL, NULL}};
+               {0, NULL, 0, NULL, NULL}
+       };
                
        
        srna = RNA_def_struct(brna, "DriverVariable", NULL);
@@ -1244,9 +1514,9 @@ static void rna_def_channeldriver_variables(BlenderRNA *brna, PropertyRNA *cprop
        /* add variable */
        func = RNA_def_function(srna, "new", "rna_Driver_new_variable");
        RNA_def_function_ui_description(func, "Add a new variable for the driver");
-               /* return type */
+       /* return type */
        parm = RNA_def_pointer(func, "var", "DriverVariable", "", "Newly created Driver Variable");
-               RNA_def_function_return(func, parm);
+       RNA_def_function_return(func, parm);
 
        /* remove variable */
        func = RNA_def_function(srna, "remove", "rna_Driver_remove_variable");
@@ -1254,7 +1524,8 @@ static void rna_def_channeldriver_variables(BlenderRNA *brna, PropertyRNA *cprop
        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);
+       RNA_def_property_flag(parm, PROP_REQUIRED | PROP_NEVER_NULL | PROP_RNAPTR);
+       RNA_def_property_clear_flag(parm, PROP_THICK_WRAP);
 }
 
 static void rna_def_channeldriver(BlenderRNA *brna)
@@ -1268,7 +1539,8 @@ static void rna_def_channeldriver(BlenderRNA *brna)
                {DRIVER_TYPE_PYTHON, "SCRIPTED", 0, "Scripted Expression", ""},
                {DRIVER_TYPE_MIN, "MIN", 0, "Minimum Value", ""},
                {DRIVER_TYPE_MAX, "MAX", 0, "Maximum Value", ""},
-               {0, NULL, 0, NULL, NULL}};
+               {0, NULL, 0, NULL, NULL}
+       };
 
        srna = RNA_def_struct(brna, "Driver", NULL);
        RNA_def_struct_sdna(srna, "ChannelDriver");
@@ -1324,14 +1596,14 @@ static void rna_def_fpoint(BlenderRNA *brna)
        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, "Select", "Selection status");
-       RNA_def_property_update(prop, NC_ANIMATION|ND_KEYFRAME|NA_SELECTED, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME | NA_SELECTED, NULL);
        
        /* Vector value */
        prop = RNA_def_property(srna, "co", PROP_FLOAT, PROP_COORDS); /* keyframes are dimensionless */
        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|NA_EDITED, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, NULL);
 }
 
 
@@ -1351,63 +1623,88 @@ static void rna_def_fkeyframe(BlenderRNA *brna)
        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", "Left handle selection status");
-       RNA_def_property_update(prop, NC_ANIMATION|ND_KEYFRAME|NA_SELECTED, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME | NA_SELECTED, NULL);
        
        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", "Right handle selection status");
-       RNA_def_property_update(prop, NC_ANIMATION|ND_KEYFRAME|NA_SELECTED, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME | NA_SELECTED, NULL);
        
        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, "Select", "Control point selection status");
-       RNA_def_property_update(prop, NC_ANIMATION|ND_KEYFRAME|NA_SELECTED, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME | NA_SELECTED, NULL);
        
        /* 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);
+       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);
+       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");
-       RNA_def_property_update(prop, NC_ANIMATION|ND_KEYFRAME_PROP, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME_PROP, NULL);
        
        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);
+       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);
        RNA_def_property_float_funcs(prop, "rna_FKeyframe_handle1_get", "rna_FKeyframe_handle1_set", NULL);
        RNA_def_property_ui_text(prop, "Left Handle", "Coordinates of the left handle (before the 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, NULL);
        
        prop = RNA_def_property(srna, "co", PROP_FLOAT, PROP_COORDS); /* keyframes are dimensionless */
        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|NA_EDITED, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, NULL);
        
        prop = RNA_def_property(srna, "handle_right", PROP_FLOAT, PROP_COORDS); /* keyframes are dimensionless */
        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, "Right Handle", "Coordinates of the right handle (after the 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, NULL);
 }
 
 static void rna_def_fcurve_modifiers(BlenderRNA *brna, PropertyRNA *cprop)
@@ -1440,7 +1737,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");
@@ -1448,7 +1745,8 @@ static void rna_def_fcurve_modifiers(BlenderRNA *brna, PropertyRNA *cprop)
        RNA_def_function_ui_description(func, "Remove a modifier from this F-Curve");
        /* modifier to remove */
        parm = RNA_def_pointer(func, "modifier", "FModifier", "", "Removed modifier");
-       RNA_def_property_flag(parm, PROP_REQUIRED|PROP_NEVER_NULL);
+       RNA_def_property_flag(parm, PROP_REQUIRED | PROP_NEVER_NULL | PROP_RNAPTR);
+       RNA_def_property_clear_flag(parm, PROP_THICK_WRAP);
 }
 
 /* fcurve.keyframe_points */
@@ -1463,7 +1761,8 @@ static void rna_def_fcurve_keyframe_points(BlenderRNA *brna, PropertyRNA *cprop)
                {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}};
+               {0, NULL, 0, NULL, NULL}
+       };
 
        RNA_def_property_srna(cprop, "FCurveKeyframePoints");
        srna = RNA_def_struct(brna, "FCurveKeyframePoints", NULL);
@@ -1486,13 +1785,14 @@ static void rna_def_fcurve_keyframe_points(BlenderRNA *brna, PropertyRNA *cprop)
 
        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);
+       RNA_def_int(func, "count", 1, 0, INT_MAX, "Number", "Number of points to add to the spline", 0, INT_MAX);
 
        func = RNA_def_function(srna, "remove", "rna_FKeyframe_points_remove");
        RNA_def_function_ui_description(func, "Remove keyframe from an F-Curve");
        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);
+       RNA_def_property_flag(parm, PROP_REQUIRED | PROP_NEVER_NULL | PROP_RNAPTR);
+       RNA_def_property_clear_flag(parm, PROP_THICK_WRAP);
        /* optional */
        RNA_def_boolean(func, "fast", 0, "Fast", "Fast keyframe removal to avoid recalculating the curve each time");
 }
@@ -1505,14 +1805,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", ""},
-               {0, NULL, 0, NULL, NULL}};
+               {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", ""},
-               {0, NULL, 0, NULL, NULL}};
+               {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_CUSTOM, "CUSTOM", 0, "User Defined",
+                                     "Use custom hand-picked color for F-Curve"},
+               {0, NULL, 0, NULL, NULL}
+       };
 
        srna = RNA_def_struct(brna, "FCurve", NULL);
        RNA_def_struct_ui_text(srna, "F-Curve", "F-Curve defining values of a period of time");
@@ -1522,9 +1827,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);
@@ -1543,14 +1848,14 @@ static void rna_def_fcurve(BlenderRNA *brna)
        RNA_def_property_string_funcs(prop, "rna_FCurve_RnaPath_get", "rna_FCurve_RnaPath_length",
                                      "rna_FCurve_RnaPath_set");
        RNA_def_property_ui_text(prop, "Data Path", "RNA Path to property affected by F-Curve");
-               /* XXX need an update callback for this to that animation gets evaluated */
+       /* XXX need an update callback for this to that animation gets evaluated */
        RNA_def_property_update(prop, NC_ANIMATION, NULL);
 
        /* called 'index' when given as function arg */
        prop = RNA_def_property(srna, "array_index", PROP_INT, PROP_NONE);
        RNA_def_property_ui_text(prop, "RNA Array Index",
                                 "Index to the specific property affected by F-Curve if applicable");
-               /* XXX need an update callback for this so that animation gets evaluated */
+       /* XXX need an update callback for this so that animation gets evaluated */
        RNA_def_property_update(prop, NC_ANIMATION, NULL);
        
        /* Color */
@@ -1569,22 +1874,22 @@ static void rna_def_fcurve(BlenderRNA *brna)
        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, "Select", "F-Curve is selected for editing");
-       RNA_def_property_update(prop, NC_ANIMATION|ND_KEYFRAME|NA_SELECTED, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME | NA_SELECTED, NULL);
        
        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, "Lock", "F-Curve's settings cannot be edited");
-       RNA_def_property_update(prop, NC_ANIMATION|ND_ANIMCHAN|NA_EDITED, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION | ND_ANIMCHAN | NA_EDITED, NULL);
        
        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|NA_EDITED, NULL);
+       RNA_def_property_update(prop, NC_ANIMATION | ND_ANIMCHAN | NA_EDITED, NULL);
        
        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);
+       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);
@@ -1592,7 +1897,7 @@ static void rna_def_fcurve(BlenderRNA *brna)
        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);
+       RNA_def_property_update(prop, NC_ANIMATION | ND_KEYFRAME_PROP, NULL);
        
        /* Collections */
        prop = RNA_def_property(srna, "sampled_points", PROP_COLLECTION, PROP_NONE);
@@ -1612,22 +1917,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);
 }
 
 /* *********************** */
@@ -1635,8 +1959,8 @@ static void rna_def_fcurve(BlenderRNA *brna)
 void RNA_def_fcurve(BlenderRNA *brna)
 {
        rna_def_fcurve(brna);
-               rna_def_fkeyframe(brna);
-               rna_def_fpoint(brna);
+       rna_def_fkeyframe(brna);
+       rna_def_fpoint(brna);
        
        rna_def_drivertarget(brna);
        rna_def_drivervar(brna);
@@ -1646,8 +1970,10 @@ void RNA_def_fcurve(BlenderRNA *brna)
        
        rna_def_fmodifier_generator(brna);
        rna_def_fmodifier_function_generator(brna);
+
        rna_def_fmodifier_envelope(brna);
-               rna_def_fmodifier_envelope_ctrl(brna);
+       rna_def_fmodifier_envelope_ctrl(brna);
+
        rna_def_fmodifier_cycles(brna);
        rna_def_fmodifier_python(brna);
        rna_def_fmodifier_limits(brna);