Commit: 8001854083a9921a12dc6c1ebc81f875ccd64fb0
Author: Campbell Barton
Date:   Sat Jul 23 04:03:36 2016 +1000
Branches: master
https://developer.blender.org/rB8001854083a9921a12dc6c1ebc81f875ccd64fb0

Cleanup: style

===================================================================

M       source/blender/python/intern/bpy_operator.c
M       source/blender/python/intern/bpy_rna.c
M       source/blender/python/intern/bpy_rna.h

===================================================================

diff --git a/source/blender/python/intern/bpy_operator.c 
b/source/blender/python/intern/bpy_operator.c
index eaa96e6..bd3e573 100644
--- a/source/blender/python/intern/bpy_operator.c
+++ b/source/blender/python/intern/bpy_operator.c
@@ -223,9 +223,9 @@ static PyObject *pyop_call(PyObject *UNUSED(self), PyObject 
*args)
                WM_operator_properties_create_ptr(&ptr, ot);
                WM_operator_properties_sanitize(&ptr, 0);
 
-               if (kw && PyDict_Size(kw))
-                       error_val = pyrna_pydict_to_props(&ptr, kw, 0, 
"Converting py args to operator properties: ");
-
+               if (kw && PyDict_Size(kw)) {
+                       error_val = pyrna_pydict_to_props(&ptr, kw, false, 
"Converting py args to operator properties: ");
+               }
 
                if (error_val == 0) {
                        ReportList *reports;
@@ -353,8 +353,9 @@ static PyObject *pyop_as_string(PyObject *UNUSED(self), 
PyObject *args)
        /* Save another lookup */
        RNA_pointer_create(NULL, ot->srna, NULL, &ptr);
 
-       if (kw && PyDict_Size(kw))
-               error_val = pyrna_pydict_to_props(&ptr, kw, 0, "Converting py 
args to operator properties: ");
+       if (kw && PyDict_Size(kw)) {
+               error_val = pyrna_pydict_to_props(&ptr, kw, false, "Converting 
py args to operator properties: ");
+       }
 
        if (error_val == 0)
                buf = WM_operator_pystring_ex(C, NULL, all_args, macro_args, 
ot, &ptr);
diff --git a/source/blender/python/intern/bpy_rna.c 
b/source/blender/python/intern/bpy_rna.c
index 1c6471c..ab523e0 100644
--- a/source/blender/python/intern/bpy_rna.c
+++ b/source/blender/python/intern/bpy_rna.c
@@ -136,7 +136,9 @@ static void id_release_gc(struct ID *id)
                PyGC_Head *g = gen->gc.gc_next;
                while ((g = g->gc.gc_next) != gen) {
                        PyObject *ob = FROM_GC(g);
-                       if (PyType_IsSubtype(Py_TYPE(ob), &pyrna_struct_Type) 
|| PyType_IsSubtype(Py_TYPE(ob), &pyrna_prop_Type)) {
+                       if (PyType_IsSubtype(Py_TYPE(ob), &pyrna_struct_Type) ||
+                           PyType_IsSubtype(Py_TYPE(ob), &pyrna_prop_Type))
+                       {
                                BPy_DummyPointerRNA *ob_ptr = 
(BPy_DummyPointerRNA *)ob;
                                if (ob_ptr->ptr.id.data == id) {
                                        pyrna_invalidate(ob_ptr);
@@ -358,9 +360,12 @@ static int deferred_register_prop(StructRNA *srna, 
PyObject *key, PyObject *item
 #ifdef USE_MATHUTILS
 #include "../mathutils/mathutils.h" /* so we can have mathutils callbacks */
 
-static PyObject *pyrna_prop_array_subscript_slice(BPy_PropertyArrayRNA *self, 
PointerRNA *ptr, PropertyRNA *prop,
-                                                  Py_ssize_t start, Py_ssize_t 
stop, Py_ssize_t length);
-static short pyrna_rotation_euler_order_get(PointerRNA *ptr, PropertyRNA 
**prop_eul_order, const short order_fallback);
+static PyObject *pyrna_prop_array_subscript_slice(
+        BPy_PropertyArrayRNA *self, PointerRNA *ptr, PropertyRNA *prop,
+        Py_ssize_t start, Py_ssize_t stop, Py_ssize_t length);
+static short pyrna_rotation_euler_order_get(
+        PointerRNA *ptr, const short order_fallback,
+        PropertyRNA **r_prop_eul_order);
 
 /* bpyrna vector/euler/quat callbacks */
 static unsigned char mathutils_rna_array_cb_index = -1; /* index for our 
callbacks */
@@ -395,7 +400,7 @@ static int mathutils_rna_vector_get(BaseMathObject *bmo, 
int subtype)
        if (subtype == MATHUTILS_CB_SUBTYPE_EUL) {
                EulerObject *eul = (EulerObject *)bmo;
                PropertyRNA *prop_eul_order = NULL;
-               eul->order = pyrna_rotation_euler_order_get(&self->ptr, 
&prop_eul_order, eul->order);
+               eul->order = pyrna_rotation_euler_order_get(&self->ptr, 
eul->order, &prop_eul_order);
        }
 
        return 0;
@@ -442,7 +447,7 @@ static int mathutils_rna_vector_set(BaseMathObject *bmo, 
int subtype)
        if (subtype == MATHUTILS_CB_SUBTYPE_EUL) {
                EulerObject *eul = (EulerObject *)bmo;
                PropertyRNA *prop_eul_order = NULL;
-               short order = pyrna_rotation_euler_order_get(&self->ptr, 
&prop_eul_order, eul->order);
+               short order = pyrna_rotation_euler_order_get(&self->ptr, 
eul->order, &prop_eul_order);
                if (order != eul->order) {
                        RNA_property_enum_set(&self->ptr, prop_eul_order, 
eul->order);
                        if (RNA_property_update_check(prop_eul_order)) {
@@ -562,16 +567,21 @@ static Mathutils_Callback mathutils_rna_matrix_cb = {
        NULL
 };
 
-static short pyrna_rotation_euler_order_get(PointerRNA *ptr, PropertyRNA 
**prop_eul_order, const short order_fallback)
+static short pyrna_rotation_euler_order_get(
+        PointerRNA *ptr, const short order_fallback,
+        PropertyRNA **r_prop_eul_order)
 {
        /* attempt to get order */
-       if (*prop_eul_order == NULL)
-               *prop_eul_order = RNA_struct_find_property(ptr, 
"rotation_mode");
+       if (*r_prop_eul_order == NULL) {
+               *r_prop_eul_order = RNA_struct_find_property(ptr, 
"rotation_mode");
+       }
 
-       if (*prop_eul_order) {
-               short order = RNA_property_enum_get(ptr, *prop_eul_order);
-               if (order >= EULER_ORDER_XYZ && order <= EULER_ORDER_ZYX) /* 
could be quat or axisangle */
+       if (*r_prop_eul_order) {
+               short order = RNA_property_enum_get(ptr, *r_prop_eul_order);
+               /* could be quat or axisangle */
+               if (order >= EULER_ORDER_XYZ && order <= EULER_ORDER_ZYX) {
                        return order;
+               }
        }
 
        return order_fallback;
@@ -639,7 +649,8 @@ PyObject *pyrna_math_object_from_array(PointerRNA *ptr, 
PropertyRNA *prop)
                                                
RNA_property_float_get_array(ptr, prop, ((VectorObject *)ret)->vec);
                                        }
                                        else {
-                                               PyObject *vec_cb = 
Vector_CreatePyObject_cb(ret, len, mathutils_rna_array_cb_index, 
MATHUTILS_CB_SUBTYPE_VEC);
+                                               PyObject *vec_cb = 
Vector_CreatePyObject_cb(
+                                                       ret, len, 
mathutils_rna_array_cb_index, MATHUTILS_CB_SUBTYPE_VEC);
                                                Py_DECREF(ret); /* the vector 
owns now */
                                                ret = vec_cb; /* return the 
vector instead */
                                        }
@@ -652,7 +663,8 @@ PyObject *pyrna_math_object_from_array(PointerRNA *ptr, 
PropertyRNA *prop)
                                                
RNA_property_float_get_array(ptr, prop, ((MatrixObject *)ret)->matrix);
                                        }
                                        else {
-                                               PyObject *mat_cb = 
Matrix_CreatePyObject_cb(ret, 4, 4, mathutils_rna_matrix_cb_index, 0);
+                                               PyObject *mat_cb = 
Matrix_CreatePyObject_cb(
+                                                       ret, 4, 4, 
mathutils_rna_matrix_cb_index, 0);
                                                Py_DECREF(ret); /* the matrix 
owns now */
                                                ret = mat_cb; /* return the 
matrix instead */
                                        }
@@ -663,7 +675,8 @@ PyObject *pyrna_math_object_from_array(PointerRNA *ptr, 
PropertyRNA *prop)
                                                
RNA_property_float_get_array(ptr, prop, ((MatrixObject *)ret)->matrix);
                                        }
                                        else {
-                                               PyObject *mat_cb = 
Matrix_CreatePyObject_cb(ret, 3, 3, mathutils_rna_matrix_cb_index, 0);
+                                               PyObject *mat_cb = 
Matrix_CreatePyObject_cb(
+                                                       ret, 3, 3, 
mathutils_rna_matrix_cb_index, 0);
                                                Py_DECREF(ret); /* the matrix 
owns now */
                                                ret = mat_cb; /* return the 
matrix instead */
                                        }
@@ -675,14 +688,16 @@ PyObject *pyrna_math_object_from_array(PointerRNA *ptr, 
PropertyRNA *prop)
                                        if (is_thick) {
                                                /* attempt to get order, only 
needed for thick types since wrapped with update via callbacks */
                                                PropertyRNA *prop_eul_order = 
NULL;
-                                               short order = 
pyrna_rotation_euler_order_get(ptr, &prop_eul_order, EULER_ORDER_XYZ);
+                                               short order = 
pyrna_rotation_euler_order_get(ptr, EULER_ORDER_XYZ, &prop_eul_order);
 
                                                ret = 
Euler_CreatePyObject(NULL, order, NULL);  /* TODO, get order from RNA */
                                                
RNA_property_float_get_array(ptr, prop, ((EulerObject *)ret)->eul);
                                        }
                                        else {
                                                /* order will be updated from 
callback on use */
-                                               PyObject *eul_cb = 
Euler_CreatePyObject_cb(ret, EULER_ORDER_XYZ, mathutils_rna_array_cb_index, 
MATHUTILS_CB_SUBTYPE_EUL); // TODO, get order from RNA
+                                               // TODO, get order from RNA
+                                               PyObject *eul_cb = 
Euler_CreatePyObject_cb(
+                                                       ret, EULER_ORDER_XYZ, 
mathutils_rna_array_cb_index, MATHUTILS_CB_SUBTYPE_EUL);
                                                Py_DECREF(ret); /* the euler 
owns now */
                                                ret = eul_cb; /* return the 
euler instead */
                                        }
@@ -693,7 +708,8 @@ PyObject *pyrna_math_object_from_array(PointerRNA *ptr, 
PropertyRNA *prop)
                                                
RNA_property_float_get_array(ptr, prop, ((QuaternionObject *)ret)->quat);
                                        }
                                        else {
-                                               PyObject *quat_cb = 
Quaternion_CreatePyObject_cb(ret, mathutils_rna_array_cb_index, 
MATHUTILS_CB_SUBTYPE_QUAT);
+                                               PyObject *quat_cb = 
Quaternion_CreatePyObject_cb(
+                                                       ret, 
mathutils_rna_array_cb_index, MATHUTILS_CB_SUBTYPE_QUAT);
                                                Py_DECREF(ret); /* the quat 
owns now */
                                                ret = quat_cb; /* return the 
quat instead */
                                        }
@@ -707,7 +723,8 @@ PyObject *pyrna_math_object_from_array(PointerRNA *ptr, 
PropertyRNA *prop)
                                                
RNA_property_float_get_array(ptr, prop, ((ColorObject *)ret)->col);
                                        }
                                        else {
-                                               PyObject *col_cb = 
Color_CreatePyObject_cb(ret, mathutils_rna_array_cb_index, 
MATHUTILS_CB_SUBTYPE_COLOR);
+                                               PyObject *col_cb = 
Color_CreatePyObject_cb(
+                                                       ret, 
mathutils_rna_array_cb_index, MATHUTILS_CB_SUBTYPE_COLOR);
                                                Py_DECREF(ret); /* the color 
owns now */
                                                ret = col_cb; /* return the 
color instead */
                                        }
@@ -738,9 +755,11 @@ thick_wrap_slice:
 }
 
 /* same as RNA_enum_value_from_id but raises an exception */
-int pyrna_enum_value_from_id(EnumPropertyItem *item, const char *identifier, 
int *value, const char *error_prefix)
+int pyrna_enum_value_from_id(
+        EnumPropertyItem *item, const char *identifier, int *r_value,
+        const char *error_prefix)
 {
-       if (RNA_enum_value_from_id(item, identifier, value) == 0) {
+       if (RNA_enum_value_from_id(item, identifier, r_value) == 0) {
                const char *enum_str = BPy_enum_as_string(item);
                PyErr_Format(PyExc_ValueError,
                             "%s: '%.200s' not found in (%s)",
@@ -1165,7 +1184,9 @@ static const char *pyrna_enum_as_string(PointerRNA *ptr, 
PropertyRNA *prop)
 }
 
 
-static int pyrna_string_to_enum(PyObject *item, PointerRNA *ptr, PropertyRNA 
*prop, int *val, const char *error_prefix)
+static int pyrna_string_to_enum(
+        PyObject *item, PointerRNA *ptr, PropertyRNA *prop, int *r_value,
+        const char *error_prefix)
 {
        const char *param = _PyUnicode_AsString(item);
 
@@ -1176,7 +1197,7 @@ static int pyrna_string_to_enum(PyObject *item, 
PointerRNA *ptr, PropertyRNA *pr
                return -1;
        }
        else {
-               if (!RNA_property_enum_value(BPy_GetContext(), ptr, prop, 
param, val)) {
+               if (!RNA_property_enum_value(BPy_GetContext(), ptr, prop, 
param, r_value)) {
                        const char *enum_str = pyrna_enum_as_string(ptr, prop);
                        PyErr_Format(PyEx

@@ Diff output truncated at 10240 characters. @@

_______________________________________________
Bf-blender-cvs mailing list
[email protected]
https://lists.blender.org/mailman/listinfo/bf-blender-cvs

Reply via email to