doxygen: prevent GPL license block from being parsed as doxygen comment.
[blender.git] / source / blender / makesrna / intern / rna_define.c
index 7629387..97c7df2 100644 (file)
@@ -1,4 +1,4 @@
-/**
+/*
  * $Id$
  *
  * ***** BEGIN GPL LICENSE BLOCK *****
 #include "DNA_genfile.h"
 #include "DNA_sdna_types.h"
 
-#include "RNA_access.h"
-#include "RNA_define.h"
-#include "RNA_types.h"
-
-#include "BLI_ghash.h"
 #include "BLI_string.h"
+#include "BLI_utildefines.h"
+#include "BLI_ghash.h"
+
+#include "RNA_define.h"
 
 #include "rna_internal.h"
 
@@ -266,8 +265,8 @@ static ContainerDefRNA *rna_find_container_def(ContainerRNA *cont)
 /* DNA utility function for looking up members */
 
 typedef struct DNAStructMember {
-       char *type;
-       char *name;
+       const char *type;
+       const char *name;
        int arraylength;
        int pointerlevel;
 } DNAStructMember;
@@ -297,7 +296,7 @@ static int rna_member_cmp(const char *name, const char *oname)
 
 static int rna_find_sdna_member(SDNA *sdna, const char *structname, const char *membername, DNAStructMember *smember)
 {
-       char *dnaname;
+       const char *dnaname;
        short *sp;
        int a, b, structnr, totmember, cmp;
 
@@ -361,7 +360,7 @@ static int rna_validate_identifier(const char *identifier, char *error, int prop
        int a=0;
        
        /*  list from http://docs.python.org/reference/lexical_analysis.html#id5 */
-       static char *kwlist[] = {
+       static const char *kwlist[] = {
                "and", "as", "assert", "break",
                "class", "continue", "def", "del",
                "elif", "else", "except", "exec",
@@ -413,7 +412,7 @@ static int rna_validate_identifier(const char *identifier, char *error, int prop
 
 /* Blender Data Definition */
 
-BlenderRNA *RNA_create()
+BlenderRNA *RNA_create(void)
 {
        BlenderRNA *brna;
 
@@ -476,11 +475,12 @@ void RNA_struct_free(BlenderRNA *brna, StructRNA *srna)
        PropertyRNA *prop, *nextprop;
        PropertyRNA *parm, *nextparm;
 
+       /*
        if(srna->flag & STRUCT_RUNTIME) {
                if(RNA_struct_py_type_get(srna)) {
-                       fprintf(stderr, "StructRNA \"%s\" freed while holding a python reference\n", srna->name);
+                       fprintf(stderr, "RNA_struct_free '%s' freed while holding a python reference\n", srna->identifier);
                }
-       }
+       } */
 
        for(prop=srna->cont.properties.first; prop; prop=nextprop) {
                nextprop= prop->next;
@@ -672,7 +672,7 @@ StructRNA *RNA_def_struct(BlenderRNA *brna, const char *identifier, const char *
 
                if(DefRNA.preprocess) {
                        RNA_def_property_struct_type(prop, "Struct");
-                       RNA_def_property_pointer_funcs(prop, "rna_builtin_type_get", NULL, NULL);
+                       RNA_def_property_pointer_funcs(prop, "rna_builtin_type_get", NULL, NULL, NULL);
                }
                else {
 #ifdef RNA_RUNTIME
@@ -739,7 +739,7 @@ void RNA_def_struct_sdna_from(StructRNA *srna, const char *structname, const cha
 void RNA_def_struct_name_property(struct StructRNA *srna, struct PropertyRNA *prop)
 {
        if(prop->type != PROP_STRING) {
-               fprintf(stderr, "RNA_def_struct_name_property: %s.%s, must be a string property.\n", srna->identifier, prop->identifier);
+               fprintf(stderr, "RNA_def_struct_name_property: \"%s.%s\", must be a string property.\n", srna->identifier, prop->identifier);
                DefRNA.error= 1;
        }
        else
@@ -783,10 +783,10 @@ void RNA_def_struct_refine_func(StructRNA *srna, const char *refine)
        if(refine) srna->refine= (StructRefineFunc)refine;
 }
 
-void RNA_def_struct_idproperties_func(StructRNA *srna, const char *idproperties)
+void RNA_def_struct_idprops_func(StructRNA *srna, const char *idproperties)
 {
        if(!DefRNA.preprocess) {
-               fprintf(stderr, "RNA_def_struct_idproperties_func: only during preprocessing.\n");
+               fprintf(stderr, "RNA_def_struct_idprops_func: only during preprocessing.\n");
                return;
        }
 
@@ -839,7 +839,7 @@ void RNA_def_struct_ui_icon(StructRNA *srna, int icon)
 
 PropertyRNA *RNA_def_property(StructOrFunctionRNA *cont_, const char *identifier, int type, int subtype)
 {
-       StructRNA *srna= DefRNA.laststruct;
+       /*StructRNA *srna= DefRNA.laststruct;*/ /* invalid for python defined props */
        ContainerRNA *cont= cont_;
        ContainerDefRNA *dcont;
        PropertyDefRNA *dprop= NULL;
@@ -849,7 +849,7 @@ PropertyRNA *RNA_def_property(StructOrFunctionRNA *cont_, const char *identifier
                char error[512];
                
                if (rna_validate_identifier(identifier, error, 1) == 0) {
-                       fprintf(stderr, "RNA_def_property: property identifier \"%s\" - %s\n", identifier, error);
+                       fprintf(stderr, "RNA_def_property: property identifier \"%s.%s\" - %s\n", CONTAINER_RNA_ID(cont), identifier, error);
                        DefRNA.error= 1;
                }
                
@@ -857,7 +857,7 @@ PropertyRNA *RNA_def_property(StructOrFunctionRNA *cont_, const char *identifier
 
                /* XXX - toto, detect supertype collisions */
                if(rna_findlink(&dcont->properties, identifier)) {
-                       fprintf(stderr, "RNA_def_property: duplicate identifier \"%s\"\n", identifier);
+                       fprintf(stderr, "RNA_def_property: duplicate identifier \"%s.%s\"\n", CONTAINER_RNA_ID(cont), identifier);
                        DefRNA.error= 1;
                }
 
@@ -887,7 +887,7 @@ PropertyRNA *RNA_def_property(StructOrFunctionRNA *cont_, const char *identifier
                        fprop->hardmin= (subtype == PROP_UNSIGNED)? 0.0f: -FLT_MAX;
                        fprop->hardmax= FLT_MAX;
 
-                       if(subtype == PROP_COLOR) {
+                       if(ELEM(subtype, PROP_COLOR, PROP_COLOR_GAMMA)) {
                                fprop->softmin= 0.0f;
                                fprop->softmax= 1.0f;
                        }
@@ -915,7 +915,7 @@ PropertyRNA *RNA_def_property(StructOrFunctionRNA *cont_, const char *identifier
                case PROP_COLLECTION:
                        break;
                default:
-                       fprintf(stderr, "RNA_def_property: %s.%s, invalid property type.\n", srna->identifier, identifier);
+                       fprintf(stderr, "RNA_def_property: \"%s.%s\", invalid property type.\n", CONTAINER_RNA_ID(cont), identifier);
                        DefRNA.error= 1;
                        return NULL;
        }
@@ -1006,18 +1006,29 @@ void RNA_def_property_clear_flag(PropertyRNA *prop, int flag)
        prop->flag &= ~flag;
 }
 
+void RNA_def_property_subtype(PropertyRNA *prop, PropertySubType subtype)
+{
+       prop->subtype= subtype;
+}
+
 void RNA_def_property_array(PropertyRNA *prop, int length)
 {
        StructRNA *srna= DefRNA.laststruct;
 
        if(length<0) {
-               fprintf(stderr, "RNA_def_property_array: %s.%s, array length must be zero of greater.\n", srna->identifier, prop->identifier);
+               fprintf(stderr, "RNA_def_property_array: \"%s.%s\", array length must be zero of greater.\n", srna->identifier, prop->identifier);
                DefRNA.error= 1;
                return;
        }
 
        if(length>RNA_MAX_ARRAY_LENGTH) {
-               fprintf(stderr, "RNA_def_property_array: %s.%s, array length must be smaller than %d.\n", srna->identifier, prop->identifier, RNA_MAX_ARRAY_LENGTH);
+               fprintf(stderr, "RNA_def_property_array: \"%s.%s\", array length must be smaller than %d.\n", srna->identifier, prop->identifier, RNA_MAX_ARRAY_LENGTH);
+               DefRNA.error= 1;
+               return;
+       }
+
+       if(prop->arraydimension > 1) {
+               fprintf(stderr, "RNA_def_property_array: \"%s.%s\", array dimensions has been set to %d but would be overwritten as 1.\n", srna->identifier, prop->identifier, prop->arraydimension);
                DefRNA.error= 1;
                return;
        }
@@ -1031,19 +1042,19 @@ void RNA_def_property_array(PropertyRNA *prop, int length)
                        prop->arraydimension= 1;
                        break;
                default:
-                       fprintf(stderr, "RNA_def_property_array: %s.%s, only boolean/int/float can be array.\n", srna->identifier, prop->identifier);
+                       fprintf(stderr, "RNA_def_property_array: \"%s.%s\", only boolean/int/float can be array.\n", srna->identifier, prop->identifier);
                        DefRNA.error= 1;
                        break;
        }
 }
 
-void RNA_def_property_multi_array(PropertyRNA *prop, int dimension, int length[])
+void RNA_def_property_multi_array(PropertyRNA *prop, int dimension, const int length[])
 {
        StructRNA *srna= DefRNA.laststruct;
        int i;
        
        if (dimension < 1 || dimension > RNA_MAX_ARRAY_DIMENSION) {
-               fprintf(stderr, "RNA_def_property_multi_array: %s.%s, array dimension must be between 1 and %d.\n", srna->identifier, prop->identifier, RNA_MAX_ARRAY_DIMENSION);
+               fprintf(stderr, "RNA_def_property_multi_array: \"%s.%s\", array dimension must be between 1 and %d.\n", srna->identifier, prop->identifier, RNA_MAX_ARRAY_DIMENSION);
                DefRNA.error= 1;
                return;
        }
@@ -1054,7 +1065,7 @@ void RNA_def_property_multi_array(PropertyRNA *prop, int dimension, int length[]
                case PROP_FLOAT:
                        break;
                default:
-                       fprintf(stderr, "RNA_def_property_multi_array: %s.%s, only boolean/int/float can be array.\n", srna->identifier, prop->identifier);
+                       fprintf(stderr, "RNA_def_property_multi_array: \"%s.%s\", only boolean/int/float can be array.\n", srna->identifier, prop->identifier);
                        DefRNA.error= 1;
                        break;
        }
@@ -1109,7 +1120,7 @@ void RNA_def_property_ui_range(PropertyRNA *prop, double min, double max, double
                        break;
                }
                default:
-                       fprintf(stderr, "RNA_def_property_ui_range: %s.%s, invalid type for ui range.\n", srna->identifier, prop->identifier);
+                       fprintf(stderr, "RNA_def_property_ui_range: \"%s.%s\", invalid type for ui range.\n", srna->identifier, prop->identifier);
                        DefRNA.error= 1;
                        break;
        }
@@ -1137,7 +1148,7 @@ void RNA_def_property_range(PropertyRNA *prop, double min, double max)
                        break;
                }
                default:
-                       fprintf(stderr, "RNA_def_property_range: %s.%s, invalid type for range.\n", srna->identifier, prop->identifier);
+                       fprintf(stderr, "RNA_def_property_range: \"%s.%s\", invalid type for range.\n", srna->identifier, prop->identifier);
                        DefRNA.error= 1;
                        break;
        }
@@ -1148,7 +1159,7 @@ void RNA_def_property_struct_type(PropertyRNA *prop, const char *type)
        StructRNA *srna= DefRNA.laststruct;
 
        if(!DefRNA.preprocess) {
-               fprintf(stderr, "RNA_def_property_struct_type %s.%s: only during preprocessing.\n", srna->identifier, prop->identifier);
+               fprintf(stderr, "RNA_def_property_struct_type \"%s.%s\": only during preprocessing.\n", srna->identifier, prop->identifier);
                return;
        }
 
@@ -1164,7 +1175,7 @@ void RNA_def_property_struct_type(PropertyRNA *prop, const char *type)
                        break;
                }
                default:
-                       fprintf(stderr, "RNA_def_property_struct_type: %s.%s, invalid type for struct type.\n", srna->identifier, prop->identifier);
+                       fprintf(stderr, "RNA_def_property_struct_type: \"%s.%s\", invalid type for struct type.\n", srna->identifier, prop->identifier);
                        DefRNA.error= 1;
                        break;
        }
@@ -1195,7 +1206,7 @@ void RNA_def_property_struct_runtime(PropertyRNA *prop, StructRNA *type)
                        break;
                }
                default:
-                       fprintf(stderr, "RNA_def_property_struct_runtime: %s.%s, invalid type for struct type.\n", srna->identifier, prop->identifier);
+                       fprintf(stderr, "RNA_def_property_struct_runtime: \"%s.%s\", invalid type for struct type.\n", srna->identifier, prop->identifier);
                        DefRNA.error= 1;
                        break;
        }
@@ -1230,7 +1241,7 @@ void RNA_def_property_enum_items(PropertyRNA *prop, const EnumPropertyItem *item
                        break;
                }
                default:
-                       fprintf(stderr, "RNA_def_property_enum_items: %s.%s, invalid type for struct type.\n", srna->identifier, prop->identifier);
+                       fprintf(stderr, "RNA_def_property_enum_items: \"%s.%s\", invalid type for struct type.\n", srna->identifier, prop->identifier);
                        DefRNA.error= 1;
                        break;
        }
@@ -1247,7 +1258,7 @@ void RNA_def_property_string_maxlength(PropertyRNA *prop, int maxlength)
                        break;
                }
                default:
-                       fprintf(stderr, "RNA_def_property_string_maxlength: %s.%s, type is not string.\n", srna->identifier, prop->identifier);
+                       fprintf(stderr, "RNA_def_property_string_maxlength: \"%s.%s\", type is not string.\n", srna->identifier, prop->identifier);
                        DefRNA.error= 1;
                        break;
        }
@@ -1264,7 +1275,7 @@ void RNA_def_property_boolean_default(PropertyRNA *prop, int value)
                        break;
                }
                default:
-                       fprintf(stderr, "RNA_def_property_boolean_default: %s.%s, type is not boolean.\n", srna->identifier, prop->identifier);
+                       fprintf(stderr, "RNA_def_property_boolean_default: \"%s.%s\", type is not boolean.\n", srna->identifier, prop->identifier);
                        DefRNA.error= 1;
                        break;
        }
@@ -1281,7 +1292,7 @@ void RNA_def_property_boolean_array_default(PropertyRNA *prop, const int *array)
                        break;
                }
                default:
-                       fprintf(stderr, "RNA_def_property_boolean_default: %s.%s, type is not boolean.\n", srna->identifier, prop->identifier);
+                       fprintf(stderr, "RNA_def_property_boolean_default: \"%s.%s\", type is not boolean.\n", srna->identifier, prop->identifier);
                        DefRNA.error= 1;
                        break;
        }
@@ -1298,7 +1309,7 @@ void RNA_def_property_int_default(PropertyRNA *prop, int value)
                        break;
                }
                default:
-                       fprintf(stderr, "RNA_def_property_int_default: %s.%s, type is not int.\n", srna->identifier, prop->identifier);
+                       fprintf(stderr, "RNA_def_property_int_default: \"%s.%s\", type is not int.\n", srna->identifier, prop->identifier);
                        DefRNA.error= 1;
                        break;
        }
@@ -1315,7 +1326,7 @@ void RNA_def_property_int_array_default(PropertyRNA *prop, const int *array)
                        break;
                }
                default:
-                       fprintf(stderr, "RNA_def_property_int_default: %s.%s, type is not int.\n", srna->identifier, prop->identifier);
+                       fprintf(stderr, "RNA_def_property_int_default: \"%s.%s\", type is not int.\n", srna->identifier, prop->identifier);
                        DefRNA.error= 1;
                        break;
        }
@@ -1332,7 +1343,7 @@ void RNA_def_property_float_default(PropertyRNA *prop, float value)
                        break;
                }
                default:
-                       fprintf(stderr, "RNA_def_property_float_default: %s.%s, type is not float.\n", srna->identifier, prop->identifier);
+                       fprintf(stderr, "RNA_def_property_float_default: \"%s.%s\", type is not float.\n", srna->identifier, prop->identifier);
                        DefRNA.error= 1;
                        break;
        }
@@ -1349,7 +1360,7 @@ void RNA_def_property_float_array_default(PropertyRNA *prop, const float *array)
                        break;
                }
                default:
-                       fprintf(stderr, "RNA_def_property_float_default: %s.%s, type is not float.\n", srna->identifier, prop->identifier);
+                       fprintf(stderr, "RNA_def_property_float_default: \"%s.%s\", type is not float.\n", srna->identifier, prop->identifier);
                        DefRNA.error= 1;
                        break;
        }
@@ -1366,7 +1377,7 @@ void RNA_def_property_string_default(PropertyRNA *prop, const char *value)
                        break;
                }
                default:
-                       fprintf(stderr, "RNA_def_property_string_default: %s.%s, type is not string.\n", srna->identifier, prop->identifier);
+                       fprintf(stderr, "RNA_def_property_string_default: \"%s.%s\", type is not string.\n", srna->identifier, prop->identifier);
                        DefRNA.error= 1;
                        break;
        }
@@ -1382,25 +1393,41 @@ void RNA_def_property_enum_default(PropertyRNA *prop, int value)
                        EnumPropertyRNA *eprop= (EnumPropertyRNA*)prop;
                        eprop->defaultvalue= value;
 
-                       for(i=0; i<eprop->totitem; i++) {
-                               if(eprop->item[i].identifier[0] && eprop->item[i].value == eprop->defaultvalue)
-                                       defaultfound= 1;
-                       }
-
-                       if(!defaultfound && eprop->totitem) {
-                               if(value == 0) {
-                                       eprop->defaultvalue= eprop->item[0].value;
+                       if(prop->flag & PROP_ENUM_FLAG) {
+                               /* check all bits are accounted for */
+                               int totflag= 0;
+                               for(i=0; i<eprop->totitem; i++) {
+                                       if(eprop->item[i].identifier[0]) {
+                                               totflag |= eprop->item[i].value;
+                                       }
                                }
-                               else {
-                                       fprintf(stderr, "RNA_def_property_enum_default: %s.%s, default is not in items.\n", srna->identifier, prop->identifier);
+
+                               if(eprop->defaultvalue & ~totflag) {
+                                       fprintf(stderr, "RNA_def_property_enum_default: \"%s.%s\", default includes unused bits (%d).\n", srna->identifier, prop->identifier, eprop->defaultvalue & ~totflag);
                                        DefRNA.error= 1;
                                }
                        }
+                       else {
+                               for(i=0; i<eprop->totitem; i++) {
+                                       if(eprop->item[i].identifier[0] && eprop->item[i].value == eprop->defaultvalue)
+                                               defaultfound= 1;
+                               }
+
+                               if(!defaultfound && eprop->totitem) {
+                                       if(value == 0) {
+                                               eprop->defaultvalue= eprop->item[0].value;
+                                       }
+                                       else {
+                                               fprintf(stderr, "RNA_def_property_enum_default: \"%s.%s\", default is not in items.\n", srna->identifier, prop->identifier);
+                                               DefRNA.error= 1;
+                                       }
+                               }
+                       }
 
                        break;
                }
                default:
-                       fprintf(stderr, "RNA_def_property_enum_default: %s.%s, type is not enum.\n", srna->identifier, prop->identifier);
+                       fprintf(stderr, "RNA_def_property_enum_default: \"%s.%s\", type is not enum.\n", srna->identifier, prop->identifier);
                        DefRNA.error= 1;
                        break;
        }
@@ -1439,7 +1466,7 @@ static PropertyDefRNA *rna_def_property_sdna(PropertyRNA *prop, const char *stru
                        return dp;
                }
                else {
-                       fprintf(stderr, "rna_def_property_sdna: %s.%s not found.\n", structname, propname);
+                       fprintf(stderr, "rna_def_property_sdna: \"%s.%s\" not found.\n", structname, propname);
                        DefRNA.error= 1;
                        return NULL;
                }
@@ -1477,13 +1504,24 @@ void RNA_def_property_boolean_sdna(PropertyRNA *prop, const char *structname, co
        }
 
        if(prop->type != PROP_BOOLEAN) {
-               fprintf(stderr, "RNA_def_property_boolean_sdna: %s.%s, type is not boolean.\n", srna->identifier, prop->identifier);
+               fprintf(stderr, "RNA_def_property_boolean_sdna: \"%s.%s\", type is not boolean.\n", srna->identifier, prop->identifier);
                DefRNA.error= 1;
                return;
        }
 
-       if((dp=rna_def_property_sdna(prop, structname, propname)))
+       if((dp=rna_def_property_sdna(prop, structname, propname))) {
+
+               if(DefRNA.silent == 0) {
+                       /* error check to ensure floats are not wrapped as ints/bools */
+                       if(dp->dnatype && *dp->dnatype && IS_DNATYPE_INT_COMPAT(dp->dnatype) == 0) {
+                               fprintf(stderr, "RNA_def_property_boolean_sdna: %s.%s is a '%s' but wrapped as type '%s'.\n", srna->identifier, prop->identifier, dp->dnatype, RNA_property_typename(prop->type));
+                               DefRNA.error= 1;
+                               return;
+                       }
+               }
+
                dp->booleanbit= bit;
+       }
 }
 
 void RNA_def_property_boolean_negative_sdna(PropertyRNA *prop, const char *structname, const char *propname, int booleanbit)
@@ -1510,12 +1548,22 @@ void RNA_def_property_int_sdna(PropertyRNA *prop, const char *structname, const
        }
 
        if(prop->type != PROP_INT) {
-               fprintf(stderr, "RNA_def_property_int_sdna: %s.%s, type is not int.\n", srna->identifier, prop->identifier);
+               fprintf(stderr, "RNA_def_property_int_sdna: \"%s.%s\", type is not int.\n", srna->identifier, prop->identifier);
                DefRNA.error= 1;
                return;
        }
 
        if((dp= rna_def_property_sdna(prop, structname, propname))) {
+
+               /* error check to ensure floats are not wrapped as ints/bools */
+               if(DefRNA.silent == 0) {
+                       if(dp->dnatype && *dp->dnatype && IS_DNATYPE_INT_COMPAT(dp->dnatype) == 0) {
+                               fprintf(stderr, "RNA_def_property_int_sdna: %s.%s is a '%s' but wrapped as type '%s'.\n", srna->identifier, prop->identifier, dp->dnatype, RNA_property_typename(prop->type));
+                               DefRNA.error= 1;
+                               return;
+                       }
+               }
+
                /* SDNA doesn't pass us unsigned unfortunately .. */
                if(dp->dnatype && strcmp(dp->dnatype, "char") == 0) {
                        iprop->hardmin= iprop->softmin= CHAR_MIN;
@@ -1540,6 +1588,7 @@ void RNA_def_property_int_sdna(PropertyRNA *prop, const char *structname, const
 
 void RNA_def_property_float_sdna(PropertyRNA *prop, const char *structname, const char *propname)
 {
+       PropertyDefRNA *dp;
        StructRNA *srna= DefRNA.laststruct;
 
        if(!DefRNA.preprocess) {
@@ -1548,17 +1597,30 @@ void RNA_def_property_float_sdna(PropertyRNA *prop, const char *structname, cons
        }
 
        if(prop->type != PROP_FLOAT) {
-               fprintf(stderr, "RNA_def_property_float_sdna: %s.%s, type is not float.\n", srna->identifier, prop->identifier);
+               fprintf(stderr, "RNA_def_property_float_sdna: \"%s.%s\", type is not float.\n", srna->identifier, prop->identifier);
                DefRNA.error= 1;
                return;
        }
 
+       if((dp= rna_def_property_sdna(prop, structname, propname))) {
+               /* silent is for internal use */
+               if(DefRNA.silent == 0) {
+                       if(dp->dnatype && *dp->dnatype && IS_DNATYPE_FLOAT_COMPAT(dp->dnatype) == 0) {
+                               if(prop->subtype != PROP_COLOR_GAMMA) { /* colors are an exception. these get translated */
+                                       fprintf(stderr, "RNA_def_property_float_sdna: %s.%s is a '%s' but wrapped as type '%s'.\n", srna->identifier, prop->identifier, dp->dnatype, RNA_property_typename(prop->type));
+                                       DefRNA.error= 1;
+                                       return;
+                               }
+                       }
+               }
+       }
+
        rna_def_property_sdna(prop, structname, propname);
 }
 
 void RNA_def_property_enum_sdna(PropertyRNA *prop, const char *structname, const char *propname)
 {
-       PropertyDefRNA *dp;
+       /* PropertyDefRNA *dp; */
        StructRNA *srna= DefRNA.laststruct;
        
        if(!DefRNA.preprocess) {
@@ -1567,18 +1629,18 @@ void RNA_def_property_enum_sdna(PropertyRNA *prop, const char *structname, const
        }
 
        if(prop->type != PROP_ENUM) {
-               fprintf(stderr, "RNA_def_property_enum_sdna: %s.%s, type is not enum.\n", srna->identifier, prop->identifier);
+               fprintf(stderr, "RNA_def_property_enum_sdna: \"%s.%s\", type is not enum.\n", srna->identifier, prop->identifier);
                DefRNA.error= 1;
                return;
        }
 
-       if((dp=rna_def_property_sdna(prop, structname, propname))) {
+       if(( /* dp= */ rna_def_property_sdna(prop, structname, propname))) {
                if(prop->arraydimension) {
                        prop->arraydimension= 0;
                        prop->totarraylength= 0;
 
                        if(!DefRNA.silent) {
-                               fprintf(stderr, "RNA_def_property_enum_sdna: %s.%s, array not supported for enum type.\n", structname, propname);
+                               fprintf(stderr, "RNA_def_property_enum_sdna: \"%s.%s\", array not supported for enum type.\n", structname, propname);
                                DefRNA.error= 1;
                        }
                }
@@ -1599,7 +1661,7 @@ void RNA_def_property_enum_bitflag_sdna(PropertyRNA *prop, const char *structnam
 
 void RNA_def_property_string_sdna(PropertyRNA *prop, const char *structname, const char *propname)
 {
-       PropertyDefRNA *dp;
+       /* PropertyDefRNA *dp; */
        StringPropertyRNA *sprop= (StringPropertyRNA*)prop;
        StructRNA *srna= DefRNA.laststruct;
 
@@ -1609,12 +1671,12 @@ void RNA_def_property_string_sdna(PropertyRNA *prop, const char *structname, con
        }
 
        if(prop->type != PROP_STRING) {
-               fprintf(stderr, "RNA_def_property_string_sdna: %s.%s, type is not string.\n", srna->identifier, prop->identifier);
+               fprintf(stderr, "RNA_def_property_string_sdna: \"%s.%s\", type is not string.\n", srna->identifier, prop->identifier);
                DefRNA.error= 1;
                return;
        }
 
-       if((dp=rna_def_property_sdna(prop, structname, propname))) {
+       if((/* dp= */ rna_def_property_sdna(prop, structname, propname))) {
                if(prop->arraydimension) {
                        sprop->maxlength= prop->totarraylength;
                        prop->arraydimension= 0;
@@ -1625,7 +1687,7 @@ void RNA_def_property_string_sdna(PropertyRNA *prop, const char *structname, con
 
 void RNA_def_property_pointer_sdna(PropertyRNA *prop, const char *structname, const char *propname)
 {
-       PropertyDefRNA *dp;
+       /* PropertyDefRNA *dp; */
        StructRNA *srna= DefRNA.laststruct;
        
        if(!DefRNA.preprocess) {
@@ -1634,18 +1696,18 @@ void RNA_def_property_pointer_sdna(PropertyRNA *prop, const char *structname, co
        }
 
        if(prop->type != PROP_POINTER) {
-               fprintf(stderr, "RNA_def_property_pointer_sdna: %s.%s, type is not pointer.\n", srna->identifier, prop->identifier);
+               fprintf(stderr, "RNA_def_property_pointer_sdna: \"%s.%s\", type is not pointer.\n", srna->identifier, prop->identifier);
                DefRNA.error= 1;
                return;
        }
 
-       if((dp=rna_def_property_sdna(prop, structname, propname))) {
+       if((/* dp= */ rna_def_property_sdna(prop, structname, propname))) {
                if(prop->arraydimension) {
                        prop->arraydimension= 0;
                        prop->totarraylength= 0;
 
                        if(!DefRNA.silent) {
-                               fprintf(stderr, "RNA_def_property_pointer_sdna: %s.%s, array not supported for pointer type.\n", structname, propname);
+                               fprintf(stderr, "RNA_def_property_pointer_sdna: \"%s.%s\", array not supported for pointer type.\n", structname, propname);
                                DefRNA.error= 1;
                        }
                }
@@ -1664,7 +1726,7 @@ void RNA_def_property_collection_sdna(PropertyRNA *prop, const char *structname,
        }
 
        if(prop->type != PROP_COLLECTION) {
-               fprintf(stderr, "RNA_def_property_collection_sdna: %s.%s, type is not collection.\n", srna->identifier, prop->identifier);
+               fprintf(stderr, "RNA_def_property_collection_sdna: \"%s.%s\", type is not collection.\n", srna->identifier, prop->identifier);
                DefRNA.error= 1;
                return;
        }
@@ -1675,7 +1737,7 @@ void RNA_def_property_collection_sdna(PropertyRNA *prop, const char *structname,
                        prop->totarraylength= 0;
 
                        if(!DefRNA.silent) {
-                               fprintf(stderr, "RNA_def_property_collection_sdna: %s.%s, array of collections not supported.\n", structname, propname);
+                               fprintf(stderr, "RNA_def_property_collection_sdna: \"%s.%s\", array of collections not supported.\n", structname, propname);
                                DefRNA.error= 1;
                        }
                }
@@ -1716,7 +1778,7 @@ void RNA_def_property_collection_sdna(PropertyRNA *prop, const char *structname,
                }
                else {
                        if(!DefRNA.silent) {
-                               fprintf(stderr, "RNA_def_property_collection_sdna: %s.%s not found.\n", structname, lengthpropname);
+                               fprintf(stderr, "RNA_def_property_collection_sdna: \"%s.%s\" not found.\n", structname, lengthpropname);
                                DefRNA.error= 1;
                        }
                }
@@ -1796,7 +1858,7 @@ void RNA_def_property_boolean_funcs(PropertyRNA *prop, const char *get, const ch
                        break;
                }
                default:
-                       fprintf(stderr, "RNA_def_property_boolean_funcs: %s.%s, type is not boolean.\n", srna->identifier, prop->identifier);
+                       fprintf(stderr, "RNA_def_property_boolean_funcs: \"%s.%s\", type is not boolean.\n", srna->identifier, prop->identifier);
                        DefRNA.error= 1;
                        break;
        }
@@ -1827,7 +1889,7 @@ void RNA_def_property_int_funcs(PropertyRNA *prop, const char *get, const char *
                        break;
                }
                default:
-                       fprintf(stderr, "RNA_def_property_int_funcs: %s.%s, type is not int.\n", srna->identifier, prop->identifier);
+                       fprintf(stderr, "RNA_def_property_int_funcs: \"%s.%s\", type is not int.\n", srna->identifier, prop->identifier);
                        DefRNA.error= 1;
                        break;
        }
@@ -1858,7 +1920,7 @@ void RNA_def_property_float_funcs(PropertyRNA *prop, const char *get, const char
                        break;
                }
                default:
-                       fprintf(stderr, "RNA_def_property_float_funcs: %s.%s, type is not float.\n", srna->identifier, prop->identifier);
+                       fprintf(stderr, "RNA_def_property_float_funcs: \"%s.%s\", type is not float.\n", srna->identifier, prop->identifier);
                        DefRNA.error= 1;
                        break;
        }
@@ -1883,7 +1945,7 @@ void RNA_def_property_enum_funcs(PropertyRNA *prop, const char *get, const char
                        break;
                }
                default:
-                       fprintf(stderr, "RNA_def_property_enum_funcs: %s.%s, type is not enum.\n", srna->identifier, prop->identifier);
+                       fprintf(stderr, "RNA_def_property_enum_funcs: \"%s.%s\", type is not enum.\n", srna->identifier, prop->identifier);
                        DefRNA.error= 1;
                        break;
        }
@@ -1908,13 +1970,13 @@ void RNA_def_property_string_funcs(PropertyRNA *prop, const char *get, const cha
                        break;
                }
                default:
-                       fprintf(stderr, "RNA_def_property_string_funcs: %s.%s, type is not string.\n", srna->identifier, prop->identifier);
+                       fprintf(stderr, "RNA_def_property_string_funcs: \"%s.%s\", type is not string.\n", srna->identifier, prop->identifier);
                        DefRNA.error= 1;
                        break;
        }
 }
 
-void RNA_def_property_pointer_funcs(PropertyRNA *prop, const char *get, const char *set, const char *typef)
+void RNA_def_property_pointer_funcs(PropertyRNA *prop, const char *get, const char *set, const char *typef, const char *poll)
 {
        StructRNA *srna= DefRNA.laststruct;
 
@@ -1930,10 +1992,11 @@ void RNA_def_property_pointer_funcs(PropertyRNA *prop, const char *get, const ch
                        if(get) pprop->get= (PropPointerGetFunc)get;
                        if(set) pprop->set= (PropPointerSetFunc)set;
                        if(typef) pprop->typef= (PropPointerTypeFunc)typef;
+                       if(poll) pprop->poll= (PropPointerPollFunc)poll;
                        break;
                }
                default:
-                       fprintf(stderr, "RNA_def_property_pointer_funcs: %s.%s, type is not pointer.\n", srna->identifier, prop->identifier);
+                       fprintf(stderr, "RNA_def_property_pointer_funcs: \"%s.%s\", type is not pointer.\n", srna->identifier, prop->identifier);
                        DefRNA.error= 1;
                        break;
        }
@@ -1962,7 +2025,7 @@ void RNA_def_property_collection_funcs(PropertyRNA *prop, const char *begin, con
                        break;
                }
                default:
-                       fprintf(stderr, "RNA_def_property_collection_funcs: %s.%s, type is not collection.\n", srna->identifier, prop->identifier);
+                       fprintf(stderr, "RNA_def_property_collection_funcs: \"%s.%s\", type is not collection.\n", srna->identifier, prop->identifier);
                        DefRNA.error= 1;
                        break;
        }
@@ -2165,6 +2228,27 @@ PropertyRNA *RNA_def_enum(StructOrFunctionRNA *cont_, const char *identifier, co
        return prop;
 }
 
+/* same as above but sets 'PROP_ENUM_FLAG' before setting the default value */
+PropertyRNA *RNA_def_enum_flag(StructOrFunctionRNA *cont_, const char *identifier, const EnumPropertyItem *items, int default_value,
+       const char *ui_name, const char *ui_description)
+{
+       ContainerRNA *cont= cont_;
+       PropertyRNA *prop;
+
+       if(!items) {
+               printf("RNA_def_enum_flag: items not allowed to be NULL.\n");
+               return NULL;
+       }
+
+       prop= RNA_def_property(cont, identifier, PROP_ENUM, PROP_NONE);
+       RNA_def_property_flag(prop, PROP_ENUM_FLAG); /* important to run before default set */
+       if(items) RNA_def_property_enum_items(prop, items);
+       RNA_def_property_enum_default(prop, default_value);
+       RNA_def_property_ui_text(prop, ui_name, ui_description);
+
+       return prop;
+}
+
 void RNA_def_enum_funcs(PropertyRNA *prop, EnumPropertyItemFunc itemfunc)
 {
        EnumPropertyRNA *eprop= (EnumPropertyRNA*)prop;
@@ -2202,6 +2286,17 @@ PropertyRNA *RNA_def_float_vector(StructOrFunctionRNA *cont_, const char *identi
        return prop;
 }
 
+PropertyRNA *RNA_def_float_vector_xyz(StructOrFunctionRNA *cont_, const char *identifier, int len, const float *default_value, 
+       float hardmin, float hardmax, const char *ui_name, const char *ui_description, float softmin, float softmax)
+{
+       PropertyRNA *prop;
+       
+       prop= RNA_def_float_vector(cont_, identifier, len, default_value, hardmin, hardmax, ui_name, ui_description, softmin, softmax);
+       prop->subtype = PROP_XYZ_LENGTH;
+
+       return prop;
+}
+
 PropertyRNA *RNA_def_float_color(StructOrFunctionRNA *cont_, const char *identifier, int len, const float *default_value, 
        float hardmin, float hardmax, const char *ui_name, const char *ui_description, float softmin, float softmax)
 {
@@ -2224,8 +2319,11 @@ PropertyRNA *RNA_def_float_matrix(StructOrFunctionRNA *cont_, const char *identi
 {
        ContainerRNA *cont= cont_;
        PropertyRNA *prop;
-       int length[2]= {rows, columns};
-       
+       int length[2];
+
+       length[0]= rows;
+       length[1]= columns;
+
        prop= RNA_def_property(cont, identifier, PROP_FLOAT, PROP_MATRIX);
        RNA_def_property_multi_array(prop, 2, length);
        if(default_value) RNA_def_property_float_array_default(prop, default_value);
@@ -2424,11 +2522,11 @@ FunctionRNA *RNA_def_function_runtime(StructRNA *srna, const char *identifier, C
 void RNA_def_function_return(FunctionRNA *func, PropertyRNA *ret)
 {
        if (ret->flag & PROP_DYNAMIC) {
-               fprintf(stderr, "RNA_def_function_return: %s.%s, dynamic values are not allowed as strict returns, use RNA_def_function_output instead.\n", func->identifier, ret->identifier);
+               fprintf(stderr, "RNA_def_function_return: \"%s.%s\", dynamic values are not allowed as strict returns, use RNA_def_function_output instead.\n", func->identifier, ret->identifier);
                return;
        }
        else if (ret->arraydimension) {
-               fprintf(stderr, "RNA_def_function_return: %s.%s, arrays are not allowed as strict returns, use RNA_def_function_output instead.\n", func->identifier, ret->identifier);
+               fprintf(stderr, "RNA_def_function_return: \"%s.%s\", arrays are not allowed as strict returns, use RNA_def_function_output instead.\n", func->identifier, ret->identifier);
                return;
        }
 
@@ -2515,7 +2613,7 @@ int rna_parameter_size_alloc(PropertyRNA *parm)
        int size = rna_parameter_size(parm);
 
        if (parm->flag & PROP_DYNAMIC)
-               size+= sizeof(int);
+               size+= sizeof(((ParameterDynAlloc *)NULL)->array_tot);
 
        return size;
 }
@@ -2607,14 +2705,26 @@ void RNA_def_func_free_pointers(FunctionRNA *func)
        }
 }
 
-void RNA_def_property_duplicate_pointers(PropertyRNA *prop)
+void RNA_def_property_duplicate_pointers(StructOrFunctionRNA *cont_, PropertyRNA *prop)
 {
+       ContainerRNA *cont= cont_;
        EnumPropertyItem *earray;
        float *farray;
        int *iarray;
        int a;
 
-       if(prop->identifier) prop->identifier= BLI_strdup(prop->identifier);
+       /* annoying since we just added this to a hash, could make this add the correct key to the hash in the first place */
+       if(prop->identifier) {
+               if(cont->prophash) {
+                       BLI_ghash_remove(cont->prophash, (void*)prop->identifier, NULL, NULL);
+                       prop->identifier= BLI_strdup(prop->identifier);
+                       BLI_ghash_insert(cont->prophash, (void*)prop->identifier, prop);
+               }
+               else {
+                       prop->identifier= BLI_strdup(prop->identifier);
+               }
+       }
+
        if(prop->name) prop->name= BLI_strdup(prop->name);
        if(prop->description) prop->description= BLI_strdup(prop->description);
 
@@ -2724,5 +2834,55 @@ void RNA_def_property_free_pointers(PropertyRNA *prop)
                }
        }
 }
+
+void RNA_def_property_free(StructOrFunctionRNA *cont_, PropertyRNA *prop)
+{
+       ContainerRNA *cont= cont_;
+       
+       if(prop->flag & PROP_RUNTIME) {
+               if(cont->prophash)
+                       BLI_ghash_remove(cont->prophash, (void*)prop->identifier, NULL, NULL);
+
+               RNA_def_property_free_pointers(prop);
+               rna_freelinkN(&cont->properties, prop);
+       }
+       else {
+               RNA_def_property_free_pointers(prop);
+       }
+}
+
+/* note: only intended for removing dynamic props */
+int RNA_def_property_free_identifier(StructOrFunctionRNA *cont_, const char *identifier)
+{
+       ContainerRNA *cont= cont_;
+       PropertyRNA *prop;
+       
+       for(prop= cont->properties.first; prop; prop= prop->next) {
+               if(strcmp(prop->identifier, identifier)==0) {
+                       if(prop->flag & PROP_RUNTIME) {
+                               RNA_def_property_free(cont_, prop);
+                               return 1;
+                       }
+                       else {
+                               return -1;
+                       }
+               }
+       }
+       return 0;
+}
 #endif
 
+const char *RNA_property_typename(PropertyType type)
+{
+       switch(type) {
+               case PROP_BOOLEAN: return "PROP_BOOLEAN";
+               case PROP_INT: return "PROP_INT";
+               case PROP_FLOAT: return "PROP_FLOAT";
+               case PROP_STRING: return "PROP_STRING";
+               case PROP_ENUM: return "PROP_ENUM";
+               case PROP_POINTER: return "PROP_POINTER";
+               case PROP_COLLECTION: return "PROP_COLLECTION";
+       }
+
+       return "PROP_UNKNOWN";
+}