Manual merge of soc-2009-kazanbas branch:
[blender.git] / source / blender / python / intern / bpy_rna.c
index 3679d74..1f800be 100644 (file)
@@ -130,6 +130,70 @@ Mathutils_Callback mathutils_rna_matrix_cb = {
        (BaseMathSetIndexFunc)  NULL
 };
 
+PyObject *pyrna_math_object_from_array(PointerRNA *ptr, PropertyRNA *prop)
+{
+       PyObject *ret= NULL;
+
+#ifdef USE_MATHUTILS
+       int type, subtype, totdim;
+       int len;
+
+       len= RNA_property_array_length(ptr, prop);
+       type= RNA_property_type(prop);
+       subtype= RNA_property_subtype(prop);
+       totdim= RNA_property_array_dimension(ptr, prop, NULL);
+
+       if (type != PROP_FLOAT) return NULL;
+
+       if (totdim == 1 || (totdim == 2 && subtype == PROP_MATRIX)) {
+               ret = pyrna_prop_CreatePyObject(ptr, prop);
+
+               switch(RNA_property_subtype(prop)) {
+               case PROP_TRANSLATION:
+               case PROP_DIRECTION:
+               case PROP_VELOCITY:
+               case PROP_ACCELERATION:
+               case PROP_XYZ:
+                       if(len>=2 && len <= 4) {
+                               PyObject *vec_cb= newVectorObject_cb(ret, len, mathutils_rna_array_cb_index, FALSE);
+                               Py_DECREF(ret); /* the vector owns now */
+                               ret= vec_cb; /* return the vector instead */
+                       }
+                       break;
+               case PROP_MATRIX:
+                       if(len==16) {
+                               PyObject *mat_cb= newMatrixObject_cb(ret, 4,4, mathutils_rna_matrix_cb_index, FALSE);
+                               Py_DECREF(ret); /* the matrix owns now */
+                               ret= mat_cb; /* return the matrix instead */
+                       }
+                       else if (len==9) {
+                               PyObject *mat_cb= newMatrixObject_cb(ret, 3,3, mathutils_rna_matrix_cb_index, FALSE);
+                               Py_DECREF(ret); /* the matrix owns now */
+                               ret= mat_cb; /* return the matrix instead */
+                       }
+                       break;
+               case PROP_EULER:
+               case PROP_QUATERNION:
+                       if(len==3) { /* euler */
+                               PyObject *eul_cb= newEulerObject_cb(ret, mathutils_rna_array_cb_index, FALSE);
+                               Py_DECREF(ret); /* the matrix owns now */
+                               ret= eul_cb; /* return the matrix instead */
+                       }
+                       else if (len==4) {
+                               PyObject *quat_cb= newQuaternionObject_cb(ret, mathutils_rna_array_cb_index, FALSE);
+                               Py_DECREF(ret); /* the matrix owns now */
+                               ret= quat_cb; /* return the matrix instead */
+                       }
+                       break;
+               default:
+                       break;
+               }
+       }
+#endif
+
+       return ret;
+}
+
 #endif
 
 static StructRNA *pyrna_struct_as_srna(PyObject *self);
@@ -144,25 +208,64 @@ static int pyrna_prop_compare( BPy_PropertyRNA * a, BPy_PropertyRNA * b )
        return (a->prop==b->prop && a->ptr.data==b->ptr.data ) ? 0 : -1;
 }
 
-/* For some reason python3 needs these :/ */
-static PyObject *pyrna_struct_richcmp(BPy_StructRNA * a, BPy_StructRNA * b, int op)
+static PyObject *pyrna_struct_richcmp(PyObject *a, PyObject *b, int op)
 {
-       int cmp_result= -1; /* assume false */
-       if (BPy_StructRNA_Check(a) && BPy_StructRNA_Check(b)) {
-               cmp_result= pyrna_struct_compare(a, b);
+       PyObject *res;
+       int ok= -1; /* zero is true */
+
+       if (BPy_StructRNA_Check(a) && BPy_StructRNA_Check(b))
+               ok= pyrna_struct_compare((BPy_StructRNA *)a, (BPy_StructRNA *)b);
+
+       switch (op) {
+       case Py_NE:
+               ok = !ok; /* pass through */
+       case Py_EQ:
+               res = ok ? Py_False : Py_True;
+               break;
+
+       case Py_LT:
+       case Py_LE:
+       case Py_GT:
+       case Py_GE:
+               res = Py_NotImplemented;
+               break;
+       default:
+               PyErr_BadArgument();
+               return NULL;
        }
 
-       return Py_CmpToRich(op, cmp_result);
+       Py_INCREF(res);
+       return res;
 }
 
-static PyObject *pyrna_prop_richcmp(BPy_PropertyRNA * a, BPy_PropertyRNA * b, int op)
+static PyObject *pyrna_prop_richcmp(PyObject *a, PyObject *b, int op)
 {
-       int cmp_result= -1; /* assume false */
-       if (BPy_PropertyRNA_Check(a) && BPy_PropertyRNA_Check(b)) {
-               cmp_result= pyrna_prop_compare(a, b);
+       PyObject *res;
+       int ok= -1; /* zero is true */
+
+       if (BPy_PropertyRNA_Check(a) && BPy_PropertyRNA_Check(b))
+               ok= pyrna_prop_compare((BPy_PropertyRNA *)a, (BPy_PropertyRNA *)b);
+
+       switch (op) {
+       case Py_NE:
+               ok = !ok; /* pass through */
+       case Py_EQ:
+               res = ok ? Py_False : Py_True;
+               break;
+
+       case Py_LT:
+       case Py_LE:
+       case Py_GT:
+       case Py_GE:
+               res = Py_NotImplemented;
+               break;
+       default:
+               PyErr_BadArgument();
+               return NULL;
        }
 
-       return Py_CmpToRich(op, cmp_result);
+       Py_INCREF(res);
+       return res;
 }
 
 /*----------------------repr--------------------------------------------*/
@@ -267,61 +370,9 @@ PyObject * pyrna_prop_to_py(PointerRNA *ptr, PropertyRNA *prop)
 {
        PyObject *ret;
        int type = RNA_property_type(prop);
-       int len = RNA_property_array_length(prop);
-
-       if (len > 0) {
-               /* resolve the array from a new pytype */
-               PyObject *ret = pyrna_prop_CreatePyObject(ptr, prop);
-               
-#ifdef USE_MATHUTILS
-
-               /* return a mathutils vector where possible */
-               if(RNA_property_type(prop)==PROP_FLOAT) {
-                       switch(RNA_property_subtype(prop)) {
-                       case PROP_TRANSLATION:
-                       case PROP_DIRECTION:
-                       case PROP_VELOCITY:
-                       case PROP_ACCELERATION:
-                       case PROP_XYZ:
-                               if(len>=2 && len <= 4) {
-                                       PyObject *vec_cb= newVectorObject_cb(ret, len, mathutils_rna_array_cb_index, FALSE);
-                                       Py_DECREF(ret); /* the vector owns now */
-                                       ret= vec_cb; /* return the vector instead */
-                               }
-                               break;
-                       case PROP_MATRIX:
-                               if(len==16) {
-                                       PyObject *mat_cb= newMatrixObject_cb(ret, 4,4, mathutils_rna_matrix_cb_index, FALSE);
-                                       Py_DECREF(ret); /* the matrix owns now */
-                                       ret= mat_cb; /* return the matrix instead */
-                               }
-                               else if (len==9) {
-                                       PyObject *mat_cb= newMatrixObject_cb(ret, 3,3, mathutils_rna_matrix_cb_index, FALSE);
-                                       Py_DECREF(ret); /* the matrix owns now */
-                                       ret= mat_cb; /* return the matrix instead */
-                               }
-                               break;
-                       case PROP_EULER:
-                       case PROP_QUATERNION:
-                               if(len==3) { /* euler */
-                                       PyObject *eul_cb= newEulerObject_cb(ret, mathutils_rna_array_cb_index, FALSE);
-                                       Py_DECREF(ret); /* the matrix owns now */
-                                       ret= eul_cb; /* return the matrix instead */
-                               }
-                               else if (len==4) {
-                                       PyObject *quat_cb= newQuaternionObject_cb(ret, mathutils_rna_array_cb_index, FALSE);
-                                       Py_DECREF(ret); /* the matrix owns now */
-                                       ret= quat_cb; /* return the matrix instead */
-                               }
-                               break;
-                       default:
-                               break;
-                       }
-               }
 
-#endif
-               
-               return ret;
+       if (RNA_property_array_check(ptr, prop)) {
+               return pyrna_py_from_array(ptr, prop);
        }
        
        /* see if we can coorce into a python type - PropertyType */
@@ -490,127 +541,33 @@ int pyrna_py_to_prop(PointerRNA *ptr, PropertyRNA *prop, void *data, PyObject *v
 {
        /* XXX hard limits should be checked here */
        int type = RNA_property_type(prop);
-       int len = RNA_property_array_length(prop);
        
-       if (len > 0) {
-               PyObject *item;
-               int py_len = -1;
-               int i;
-               
+
+       if (RNA_property_array_check(ptr, prop)) {
+
+               /* char error_str[512]; */
+               int ok= 1;
+
 #ifdef USE_MATHUTILS
                if(MatrixObject_Check(value)) {
                        MatrixObject *mat = (MatrixObject*)value;
                        if(!BaseMath_ReadCallback(mat))
                                return -1;
-
-                       py_len = mat->rowSize * mat->colSize;
                } else /* continue... */
 #endif
-               if (PySequence_Check(value)) {
-                       py_len= (int)PySequence_Length(value);
-               }
-               else {
+               if (!PySequence_Check(value)) {
                        PyErr_Format(PyExc_TypeError, "%.200s RNA array assignment expected a sequence instead of %.200s instance.", error_prefix, Py_TYPE(value)->tp_name);
                        return -1;
                }
                /* done getting the length */
-               
-               if (py_len != len) {
-                       PyErr_Format(PyExc_TypeError, "%.200s python sequence length %d did not match the RNA array length %d.", error_prefix, py_len, len);
-                       return -1;
-               }
-               
-               /* for arrays we have a limited number of types */
-               switch (type) {
-               case PROP_BOOLEAN:
-               {
-                       int *param_arr;
-                       if(data)        param_arr= (int*)data;
-                       else            param_arr= MEM_mallocN(sizeof(int) * len, "pyrna bool array");
+               ok= pyrna_py_to_array(ptr, prop, data, value, error_prefix);
 
-                       
-                       /* collect the variables before assigning, incase one of them is incorrect */
-                       for (i=0; i<len; i++) {
-                               item = PySequence_GetItem(value, i);
-                               param_arr[i] = PyObject_IsTrue( item );
-                               Py_DECREF(item);
-                               
-                               if (param_arr[i] < 0) {
-                                       if(data==NULL)
-                                               MEM_freeN(param_arr);
-                                       PyErr_Format(PyExc_AttributeError, "%.200s one or more of the values in the sequence is not a boolean", error_prefix);
-                                       return -1;
-                               }
-                       }
-                       if(data==NULL) {
-                               RNA_property_boolean_set_array(ptr, prop, param_arr);
-                               MEM_freeN(param_arr);
-                       }
-
-                       break;
-               }
-               case PROP_INT:
-               {
-                       int *param_arr;
-                       if(data)        param_arr= (int*)data;
-                       else            param_arr= MEM_mallocN(sizeof(int) * len, "pyrna int array");
-
-                       
-                       /* collect the variables */
-                       for (i=0; i<len; i++) {
-                               item = PySequence_GetItem(value, i);
-                               param_arr[i] = (int)PyLong_AsSsize_t(item); /* deal with any errors later */
-                               Py_DECREF(item);
-                       }
-                       
-                       if (PyErr_Occurred()) {
-                               if(data==NULL)
-                                       MEM_freeN(param_arr);
-                               PyErr_Format(PyExc_AttributeError, "%.200s one or more of the values in the sequence could not be used as an int", error_prefix);
-                               return -1;
-                       }
-                       if(data==NULL) {
-                               RNA_property_int_set_array(ptr, prop, param_arr);
-                               MEM_freeN(param_arr);
-                       }
-                       break;
-               }
-               case PROP_FLOAT:
-               {
-                       float *param_arr;
-                       if(data)        param_arr = (float*)data;
-                       else            param_arr = MEM_mallocN(sizeof(float) * len, "pyrna float array");
-
-
-#ifdef USE_MATHUTILS
-                       if(MatrixObject_Check(value) && RNA_property_subtype(prop) == PROP_MATRIX) {
-                               MatrixObject *mat = (MatrixObject*)value;
-                               memcpy(param_arr, mat->contigPtr, sizeof(float) * len);
-                       } else /* continue... */
-#endif
-                       {
-                               /* collect the variables */
-                               for (i=0; i<len; i++) {
-                                       item = PySequence_GetItem(value, i);
-                                       param_arr[i] = (float)PyFloat_AsDouble(item); /* deal with any errors later */
-                                       Py_DECREF(item);
-                               }
-                       }
-
-                       if (PyErr_Occurred()) {
-                               if(data==NULL)
-                                       MEM_freeN(param_arr);
-                               PyErr_Format(PyExc_AttributeError, "%.200s one or more of the values in the sequence could not be used as a float", error_prefix);
-                               return -1;
-                       }
-                       if(data==NULL) {
-                               RNA_property_float_set_array(ptr, prop, param_arr);                             
-                               MEM_freeN(param_arr);
-                       }
-                       break;
-               }
+               if (!ok) {
+                       /* PyErr_Format(PyExc_AttributeError, "%.200s %s", error_prefix, error_str); */
+                       return -1;
                }
-       } else {
+       }
+       else {
                /* Normal Property (not an array) */
                
                /* see if we can coorce into a python type - PropertyType */
@@ -701,17 +658,18 @@ int pyrna_py_to_prop(PointerRNA *ptr, PropertyRNA *prop, void *data, PyObject *v
                case PROP_POINTER:
                {
                        StructRNA *ptype= RNA_property_pointer_type(ptr, prop);
+                       int flag = RNA_property_flag(prop);
 
                        if(!BPy_StructRNA_Check(value) && value != Py_None) {
-                               PointerRNA tmp;
-                               RNA_pointer_create(NULL, ptype, NULL, &tmp);
-                               PyErr_Format(PyExc_TypeError, "%.200s expected a %.200s type", error_prefix, RNA_struct_identifier(tmp.type));
+                               PyErr_Format(PyExc_TypeError, "%.200s expected a %.200s type", error_prefix, RNA_struct_identifier(ptype));
+                               return -1;
+                       } else if((flag & PROP_NEVER_NULL) && value == Py_None) {
+                               PyErr_Format(PyExc_TypeError, "property can't be assigned a None value");
                                return -1;
                        } else {
                                BPy_StructRNA *param= (BPy_StructRNA*)value;
                                int raise_error= FALSE;
                                if(data) {
-                                       int flag = RNA_property_flag(prop);
 
                                        if(flag & PROP_RNAPTR) {
                                                if(value == Py_None)
@@ -808,95 +766,95 @@ int pyrna_py_to_prop(PointerRNA *ptr, PropertyRNA *prop, void *data, PyObject *v
        return 0;
 }
 
-static PyObject * pyrna_prop_to_py_index(PointerRNA *ptr, PropertyRNA *prop, int index)
+static PyObject * pyrna_prop_to_py_index(BPy_PropertyRNA *self, int index)
 {
-       PyObject *ret;
-       int type = RNA_property_type(prop);
-       
-       /* see if we can coorce into a python type - PropertyType */
-       switch (type) {
-       case PROP_BOOLEAN:
-               ret = PyBool_FromLong( RNA_property_boolean_get_index(ptr, prop, index) );
-               break;
-       case PROP_INT:
-               ret = PyLong_FromSsize_t( (Py_ssize_t)RNA_property_int_get_index(ptr, prop, index) );
-               break;
-       case PROP_FLOAT:
-               ret = PyFloat_FromDouble( RNA_property_float_get_index(ptr, prop, index) );
-               break;
-       default:
-               PyErr_SetString(PyExc_AttributeError, "not an array type");
-               ret = NULL;
-               break;
-       }
-       
-       return ret;
+       return pyrna_py_from_array_index(self, index);
 }
 
-static int pyrna_py_to_prop_index(PointerRNA *ptr, PropertyRNA *prop, int index, PyObject *value)
+static int pyrna_py_to_prop_index(BPy_PropertyRNA *self, int index, PyObject *value)
 {
        int ret = 0;
+       int totdim;
+       PointerRNA *ptr= &self->ptr;
+       PropertyRNA *prop= self->prop;
        int type = RNA_property_type(prop);
-       
-       /* see if we can coorce into a python type - PropertyType */
-       switch (type) {
-       case PROP_BOOLEAN:
-       {
-               int param = PyObject_IsTrue( value );
-               
-               if( param < 0 ) {
-                       PyErr_SetString(PyExc_TypeError, "expected True/False or 0/1");
-                       ret = -1;
-               } else {
-                       RNA_property_boolean_set_index(ptr, prop, index, param);
-               }
-               break;
-       }
-       case PROP_INT:
-       {
-               int param = PyLong_AsSsize_t(value);
-               if (PyErr_Occurred()) {
-                       PyErr_SetString(PyExc_TypeError, "expected an int type");
-                       ret = -1;
-               } else {
-                       RNA_property_int_set_index(ptr, prop, index, param);
+
+       totdim= RNA_property_array_dimension(ptr, prop, NULL);
+
+       if (totdim > 1) {
+               /* char error_str[512]; */
+               if (!pyrna_py_to_array_index(&self->ptr, self->prop, self->arraydim, self->arrayoffset, index, value, "")) {
+                       /* PyErr_SetString(PyExc_AttributeError, error_str); */
+                       ret= -1;
                }
-               break;
        }
-       case PROP_FLOAT:
-       {
-               float param = PyFloat_AsDouble(value);
-               if (PyErr_Occurred()) {
-                       PyErr_SetString(PyExc_TypeError, "expected a float type");
+       else {
+               /* see if we can coorce into a python type - PropertyType */
+               switch (type) {
+               case PROP_BOOLEAN:
+                       {
+                               int param = PyObject_IsTrue( value );
+               
+                               if( param < 0 ) {
+                                       PyErr_SetString(PyExc_TypeError, "expected True/False or 0/1");
+                                       ret = -1;
+                               } else {
+                                       RNA_property_boolean_set_index(ptr, prop, index, param);
+                               }
+                               break;
+                       }
+               case PROP_INT:
+                       {
+                               int param = PyLong_AsSsize_t(value);
+                               if (PyErr_Occurred()) {
+                                       PyErr_SetString(PyExc_TypeError, "expected an int type");
+                                       ret = -1;
+                               } else {
+                                       RNA_property_int_set_index(ptr, prop, index, param);
+                               }
+                               break;
+                       }
+               case PROP_FLOAT:
+                       {
+                               float param = PyFloat_AsDouble(value);
+                               if (PyErr_Occurred()) {
+                                       PyErr_SetString(PyExc_TypeError, "expected a float type");
+                                       ret = -1;
+                               } else {
+                                       RNA_property_float_set_index(ptr, prop, index, param);
+                               }
+                               break;
+                       }
+               default:
+                       PyErr_SetString(PyExc_AttributeError, "not an array type");
                        ret = -1;
-               } else {
-                       RNA_property_float_set_index(ptr, prop, index, param);
+                       break;
                }
-               break;
-       }
-       default:
-               PyErr_SetString(PyExc_AttributeError, "not an array type");
-               ret = -1;
-               break;
        }
        
        return ret;
 }
 
 //---------------sequence-------------------------------------------
+static int pyrna_prop_array_length(BPy_PropertyRNA *self)
+{
+       if (RNA_property_array_dimension(&self->ptr, self->prop, NULL) > 1)
+               return RNA_property_multi_array_length(&self->ptr, self->prop, self->arraydim);
+       else
+               return RNA_property_array_length(&self->ptr, self->prop);
+}
+
 static Py_ssize_t pyrna_prop_len( BPy_PropertyRNA * self )
 {
        Py_ssize_t len;
        
        if (RNA_property_type(self->prop) == PROP_COLLECTION) {
                len = RNA_property_collection_length(&self->ptr, self->prop);
+       } else if (RNA_property_array_check(&self->ptr, self->prop)) {
+               len = pyrna_prop_array_length(self);
        } else {
-               len = RNA_property_array_length(self->prop);
-               
-               if (len==0) { /* not an array*/
-                       PyErr_SetString(PyExc_AttributeError, "len() only available for collection RNA types");
-                       return -1;
-               }
+               PyErr_SetString(PyExc_AttributeError, "len() only available for collection and array RNA types");
+               len = -1; /* error value */
        }
        
        return len;
@@ -915,14 +873,15 @@ static PyObject *prop_subscript_collection_int(BPy_PropertyRNA * self, int keynu
        PyErr_Format(PyExc_IndexError, "index %d out of range", keynum);
        return NULL;
 }
+
 static PyObject *prop_subscript_array_int(BPy_PropertyRNA * self, int keynum)
 {
-       int len= RNA_property_array_length(self->prop);
+       int len= pyrna_prop_array_length(self);
 
        if(keynum < 0) keynum += len;
 
        if(keynum >= 0 && keynum < len)
-               return pyrna_prop_to_py_index(&self->ptr, self->prop, keynum);
+               return pyrna_prop_to_py_index(self, keynum);
 
        PyErr_Format(PyExc_IndexError, "index %d out of range", keynum);
        return NULL;
@@ -969,7 +928,7 @@ static PyObject *prop_subscript_array_slice(BPy_PropertyRNA * self, int start, i
        start = MIN2(start,stop); /* values are clamped from PySlice_GetIndicesEx */
 
        for(count = start; count < stop; count++)
-               PyList_SetItem(list, count - start, pyrna_prop_to_py_index(&self->ptr, self->prop, count));
+               PyList_SetItem(list, count - start, pyrna_prop_to_py_index(self, count));
 
        return list;
 }
@@ -1022,8 +981,8 @@ static PyObject *prop_subscript_array(BPy_PropertyRNA * self, PyObject *key)
                return prop_subscript_array_int(self, PyLong_AsSsize_t(key));
        }
        else if (PySlice_Check(key)) {
-               int len= RNA_property_array_length(self->prop);
                Py_ssize_t start, stop, step, slicelength;
+               int len = pyrna_prop_array_length(self);
 
                if (PySlice_GetIndicesEx((PySliceObject*)key, len, &start, &stop, &step, &slicelength) < 0)
                        return NULL;
@@ -1049,13 +1008,12 @@ static PyObject *pyrna_prop_subscript( BPy_PropertyRNA * self, PyObject *key )
 {
        if (RNA_property_type(self->prop) == PROP_COLLECTION) {
                return prop_subscript_collection(self, key);
-       } else if (RNA_property_array_length(self->prop)) { /* arrays are currently fixed length, zero length means its not an array */
+       } else if (RNA_property_array_check(&self->ptr, self->prop)) {
                return prop_subscript_array(self, key);
-       } else {
-               PyErr_SetString(PyExc_TypeError, "rna type is not an array or a collection");
-               return NULL;
-       }
+       } 
 
+       PyErr_SetString(PyExc_TypeError, "rna type is not an array or a collection");
+       return NULL;
 }
 
 static int prop_subscript_ass_array_slice(BPy_PropertyRNA * self, int begin, int end, PyObject *value)
@@ -1066,7 +1024,7 @@ static int prop_subscript_ass_array_slice(BPy_PropertyRNA * self, int begin, int
        begin = MIN2(begin,end);
 
        for(count = begin; count < end; count++) {
-               if(pyrna_py_to_prop_index(&self->ptr, self->prop, count - begin, value) == -1) {
+               if(pyrna_py_to_prop_index(self, count - begin, value) == -1) {
                        /* TODO - this is wrong since some values have been assigned... will need to fix that */
                        return -1; /* pyrna_struct_CreatePyObject should set the error */
                }
@@ -1077,13 +1035,12 @@ static int prop_subscript_ass_array_slice(BPy_PropertyRNA * self, int begin, int
 
 static int prop_subscript_ass_array_int(BPy_PropertyRNA * self, int keynum, PyObject *value)
 {
-
-       int len= RNA_property_array_length(self->prop);
+       int len= pyrna_prop_array_length(self);
 
        if(keynum < 0) keynum += len;
 
        if(keynum >= 0 && keynum < len)
-               return pyrna_py_to_prop_index(&self->ptr, self->prop, keynum, value);
+               return pyrna_py_to_prop_index(self, keynum, value);
 
        PyErr_SetString(PyExc_IndexError, "out of range");
        return -1;
@@ -1112,7 +1069,7 @@ static int pyrna_prop_ass_subscript( BPy_PropertyRNA * self, PyObject *key, PyOb
                return prop_subscript_ass_array_int(self, i, value);
        }
        else if (PySlice_Check(key)) {
-               int len= RNA_property_array_length(self->prop);
+               int len= RNA_property_array_length(&self->ptr, self->prop);
                Py_ssize_t start, stop, step, slicelength;
 
                if (PySlice_GetIndicesEx((PySliceObject*)key, len, &start, &stop, &step, &slicelength) < 0)
@@ -1525,7 +1482,7 @@ static void foreach_attr_type(    BPy_PropertyRNA *self, char *attr,
        RNA_PROP_BEGIN(&self->ptr, itemptr, self->prop) {
                prop = RNA_struct_find_property(&itemptr, attr);
                *raw_type= RNA_property_raw_type(prop);
-               *attr_tot = RNA_property_array_length(prop);
+               *attr_tot = RNA_property_array_length(&itemptr, prop);
                *attr_signed= (RNA_property_subtype(prop)==PROP_UNSIGNED) ? FALSE:TRUE;
                break;
        }
@@ -1564,7 +1521,7 @@ static int foreach_parse_args(
                if (RNA_property_type(self->prop) == PROP_COLLECTION)
                        array_tot = RNA_property_collection_length(&self->ptr, self->prop);
                else
-                       array_tot = RNA_property_array_length(self->prop);
+                       array_tot = RNA_property_array_length(&self->ptr, self->prop);
 
 
                target_tot= array_tot * (*attr_tot);
@@ -1753,32 +1710,32 @@ static  PyObject *pyrna_prop_foreach_set(BPy_PropertyRNA *self, PyObject *args)
 PyObject *pyrna_prop_iter(BPy_PropertyRNA *self)
 {
        /* Try get values from a collection */
-       PyObject *ret = pyrna_prop_values(self);
+       PyObject *ret;
+       PyObject *iter;
        
-       if (ret==NULL) {
-               /* collection did not work, try array */
-               int len = RNA_property_array_length(self->prop);
+       if(RNA_property_array_check(&self->ptr, self->prop)) {
+               int len = pyrna_prop_array_length(self);
+               int i;
+               PyErr_Clear();
+               ret = PyList_New(len);
                
-               if (len) {
-                       int i;
-                       PyErr_Clear();
-                       ret = PyList_New(len);
-                       
-                       for (i=0; i < len; i++) {
-                               PyList_SET_ITEM(ret, i, pyrna_prop_to_py_index(&self->ptr, self->prop, i));
-                       }
+               for (i=0; i < len; i++) {
+                       PyList_SET_ITEM(ret, i, pyrna_prop_to_py_index(self, i));
                }
        }
-       
-       if (ret) {
-               /* we know this is a list so no need to PyIter_Check */
-               PyObject *iter = PyObject_GetIter(ret); 
-               Py_DECREF(ret);
-               return iter;
+       else if ((ret = pyrna_prop_values(self))) {
+               /* do nothing */
+       }
+       else {
+               PyErr_SetString( PyExc_TypeError, "this BPy_PropertyRNA object is not iterable" );
+               return NULL;
        }
        
-       PyErr_SetString( PyExc_TypeError, "this BPy_PropertyRNA object is not iterable" );
-       return NULL;
+       
+       /* we know this is a list so no need to PyIter_Check */
+       iter = PyObject_GetIter(ret);
+       Py_DECREF(ret);
+       return iter;
 }
 
 static struct PyMethodDef pyrna_struct_methods[] = {
@@ -1848,17 +1805,16 @@ PyObject *pyrna_param_to_py(PointerRNA *ptr, PropertyRNA *prop, void *data)
 {
        PyObject *ret;
        int type = RNA_property_type(prop);
-       int len = RNA_property_array_length(prop);
 
        int a;
 
-       if(len > 0) {
+       if(RNA_property_array_check(ptr, prop)) {
+               int len = RNA_property_array_length(ptr, prop);
+
                /* resolve the array from a new pytype */
                ret = PyTuple_New(len);
 
-               /* for return values, data is a pointer to an array, not first element pointer */
-               if (RNA_property_flag(prop) & PROP_DYNAMIC_ARRAY)
-                       data = *(char**)(char*)data;
+               /* kazanbas: TODO make multidim sequences here */
 
                switch (type) {
                case PROP_BOOLEAN:
@@ -2068,8 +2024,10 @@ static PyObject * pyrna_func_call(PyObject * self, PyObject *args, PyObject *kw)
 
        /* Check if we gave args that dont exist in the function
         * printing the error is slow but it should only happen when developing.
-        * the if below is quick, checking if it passed less keyword args then we gave */
-       if(kw && (PyDict_Size(kw) > kw_tot)) {
+        * the if below is quick, checking if it passed less keyword args then we gave.
+        * (Dont overwrite the error if we have one, otherwise can skip important messages and confuse with args)
+        */
+       if(err == 0 && kw && (PyDict_Size(kw) > kw_tot)) {
                PyObject *key, *value;
                Py_ssize_t pos = 0;
 
@@ -2524,6 +2482,9 @@ PyObject *pyrna_prop_CreatePyObject( PointerRNA *ptr, PropertyRNA *prop )
        
        pyrna->ptr = *ptr;
        pyrna->prop = prop;
+
+       pyrna->arraydim= 0;
+       pyrna->arrayoffset= 0;
                
        return ( PyObject * ) pyrna;
 }
@@ -2688,8 +2649,17 @@ PyObject *BPY_rna_props( void )
 
 static StructRNA *pyrna_struct_as_srna(PyObject *self)
 {
-       BPy_StructRNA *py_srna= (BPy_StructRNA*)PyObject_GetAttrString(self, "__rna__");
+       BPy_StructRNA *py_srna;
        StructRNA *srna;
+       
+       /* ack, PyObject_GetAttrString wont look up this types tp_dict first :/ */
+       if(PyType_Check(self)) {
+               py_srna = (BPy_StructRNA *)PyDict_GetItemString(((PyTypeObject *)self)->tp_dict, "__rna__");
+               Py_XINCREF(py_srna);
+       }
+       
+       if(py_srna==NULL)
+               py_srna = (BPy_StructRNA*)PyObject_GetAttrString(self, "__rna__");
 
        if(py_srna==NULL) {
                PyErr_SetString(PyExc_SystemError, "internal error, self had no __rna__ attribute, should never happen.");
@@ -2964,12 +2934,12 @@ static StructRNA *pointer_type_from_py(PyObject *value)
 
        srna= srna_from_self(value);
        if(!srna) {
-               PyErr_SetString(PyExc_SystemError, "expected an RNA type derived from IDPropertyGroup (1)");
+               PyErr_SetString(PyExc_SystemError, "expected an RNA type derived from IDPropertyGroup");
                return NULL;
        }
 
        if(!RNA_struct_is_a(srna, &RNA_IDPropertyGroup)) {
-               PyErr_SetString(PyExc_SystemError, "expected an RNA type derived from IDPropertyGroup (3)");
+               PyErr_SetString(PyExc_SystemError, "expected an RNA type derived from IDPropertyGroup");
                return NULL;
        }
 
@@ -3046,6 +3016,52 @@ PyObject *BPy_CollectionProperty(PyObject *self, PyObject *args, PyObject *kw)
        return NULL;
 }
 
+static int deferred_register_props(PyObject *py_class, StructRNA *srna)
+{
+       PyObject *props, *dummy_args, *item;
+       int i;
+
+       props= PyObject_GetAttrString(py_class, "__props__");
+       
+       if(!props) {
+               PyErr_Clear();
+               return 1;
+       }
+
+       dummy_args = PyTuple_New(0);
+
+       for(i=0; i<PyList_Size(props); i++) {
+               PyObject *py_func_ptr, *py_kw, *py_srna_cobject, *py_ret;
+               item = PyList_GET_ITEM(props, i);
+               
+               if(PyArg_ParseTuple(item, "O!O!", &PyCObject_Type, &py_func_ptr, &PyDict_Type, &py_kw)) {
+                       PyObject *(*pyfunc)(PyObject *, PyObject *, PyObject *);
+                       pyfunc = PyCObject_AsVoidPtr(py_func_ptr);
+                       py_srna_cobject = PyCObject_FromVoidPtr(srna, NULL);
+                       
+                       py_ret = pyfunc(py_srna_cobject, dummy_args, py_kw);
+                       Py_DECREF(py_srna_cobject);
+
+                       if(py_ret) {
+                               Py_DECREF(py_ret);
+                       }
+                       else {
+                               Py_DECREF(dummy_args);
+                               return 0;
+                       }
+               }
+               else {
+                       PyErr_Clear();
+                       PyErr_SetString(PyExc_AttributeError, "expected list of dicts for __props__.");
+                       Py_DECREF(dummy_args);
+                       return 0;
+               }
+       }
+
+       Py_DECREF(dummy_args);
+       return 1;
+}
+
 /*-------------------- Type Registration ------------------------*/
 
 static int rna_function_arg_count(FunctionRNA *func)
@@ -3406,6 +3422,9 @@ PyObject *pyrna_basetype_register(PyObject *self, PyObject *py_class)
                // Py_DECREF(py_class); // shuld be able to do this XXX since the old rna adds a new ref.
        }
 
+       if(!deferred_register_props(py_class, srna_new))
+               return NULL;
+
        Py_RETURN_NONE;
 }
 
@@ -3429,7 +3448,6 @@ PyObject *pyrna_basetype_unregister(PyObject *self, PyObject *py_class)
        
        /* get the context, so register callback can do necessary refreshes */
        C= BPy_GetContext();
-       
 
        /* call unregister */
        unreg(C, srna); /* calls bpy_class_free, this decref's py_class */