Commit: 1bb71a82cb2422b0060a0d759a0d0e13f104331d
Author: Campbell Barton
Date:   Fri Mar 1 13:21:08 2019 +1100
Branches: master
https://developer.blender.org/rB1bb71a82cb2422b0060a0d759a0d0e13f104331d

Cleanup: de-duplicate RNA array default args

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

M       source/blender/makesrna/RNA_define.h
M       source/blender/makesrna/intern/rna_define.c
M       source/blender/makesrna/intern/rna_object.c
M       source/blender/makesrna/intern/rna_pose.c
M       source/blender/makesrna/intern/rna_scene.c

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

diff --git a/source/blender/makesrna/RNA_define.h 
b/source/blender/makesrna/RNA_define.h
index 0e694bfe375..ab9b6f74832 100644
--- a/source/blender/makesrna/RNA_define.h
+++ b/source/blender/makesrna/RNA_define.h
@@ -246,6 +246,10 @@ extern const int rna_matrix_dimsize_3x3[];
 extern const int rna_matrix_dimsize_4x4[];
 extern const int rna_matrix_dimsize_4x2[];
 
+extern const float rna_default_axis_angle[4];
+extern const float rna_default_quaternion[4];
+extern const float rna_default_scale_3d[3];
+
 /* max size for dynamic defined type descriptors,
  * this value is arbitrary */
 #define RNA_DYN_DESCR_MAX 240
diff --git a/source/blender/makesrna/intern/rna_define.c 
b/source/blender/makesrna/intern/rna_define.c
index 1cc7a809ce2..e51fa50ddff 100644
--- a/source/blender/makesrna/intern/rna_define.c
+++ b/source/blender/makesrna/intern/rna_define.c
@@ -1411,6 +1411,11 @@ void RNA_def_property_array(PropertyRNA *prop, int 
length)
        }
 }
 
+/* common args for defaults. */
+const float rna_default_quaternion[4] = {1, 0, 0, 0};
+const float rna_default_axis_angle[4] = {0, 0, 1, 0};
+const float rna_default_scale_3d[3] = {1, 1, 1};
+
 /* common args for length */
 const int rna_matrix_dimsize_3x3[] = {3, 3};
 const int rna_matrix_dimsize_4x4[] = {4, 4};
diff --git a/source/blender/makesrna/intern/rna_object.c 
b/source/blender/makesrna/intern/rna_object.c
index a70c8a23f3c..86d3a4fde29 100644
--- a/source/blender/makesrna/intern/rna_object.c
+++ b/source/blender/makesrna/intern/rna_object.c
@@ -2122,9 +2122,6 @@ static void rna_def_object(BlenderRNA *brna)
                {0, NULL, 0, NULL, NULL},
        };
 
-       static float default_quat[4] = {1, 0, 0, 0};    /* default quaternion 
values */
-       static float default_axisAngle[4] = {0, 0, 1, 0};   /* default 
axis-angle rotation values */
-       static float default_scale[3] = {1, 1, 1}; /* default scale values */
        static int boundbox_dimsize[] = {8, 3};
 
        srna = RNA_def_struct(brna, "Object", "ID");
@@ -2258,7 +2255,7 @@ static void rna_def_object(BlenderRNA *brna)
        RNA_def_property_float_sdna(prop, NULL, "quat");
        RNA_def_property_editable_array_func(prop, 
"rna_Object_rotation_4d_editable");
        RNA_def_property_override_flag(prop, PROPOVERRIDE_OVERRIDABLE_STATIC);
-       RNA_def_property_float_array_default(prop, default_quat);
+       RNA_def_property_float_array_default(prop, rna_default_quaternion);
        RNA_def_property_ui_text(prop, "Quaternion Rotation", "Rotation in 
Quaternions");
        RNA_def_property_update(prop, NC_OBJECT | ND_TRANSFORM, 
"rna_Object_internal_update");
 
@@ -2270,7 +2267,7 @@ static void rna_def_object(BlenderRNA *brna)
        RNA_def_property_float_funcs(prop, "rna_Object_rotation_axis_angle_get",
                                     "rna_Object_rotation_axis_angle_set", 
NULL);
        RNA_def_property_editable_array_func(prop, 
"rna_Object_rotation_4d_editable");
-       RNA_def_property_float_array_default(prop, default_axisAngle);
+       RNA_def_property_float_array_default(prop, rna_default_axis_angle);
        RNA_def_property_override_flag(prop, PROPOVERRIDE_OVERRIDABLE_STATIC);
        RNA_def_property_ui_text(prop, "Axis-Angle Rotation", "Angle of 
Rotation for Axis-Angle rotation representation");
        RNA_def_property_update(prop, NC_OBJECT | ND_TRANSFORM, 
"rna_Object_internal_update");
@@ -2294,7 +2291,7 @@ static void rna_def_object(BlenderRNA *brna)
        RNA_def_property_override_flag(prop, PROPOVERRIDE_OVERRIDABLE_STATIC);
        RNA_def_property_editable_array_func(prop, "rna_Object_scale_editable");
        RNA_def_property_ui_range(prop, -FLT_MAX, FLT_MAX, 1, 3);
-       RNA_def_property_float_array_default(prop, default_scale);
+       RNA_def_property_float_array_default(prop, rna_default_scale_3d);
        RNA_def_property_ui_text(prop, "Scale", "Scaling of the object");
        RNA_def_property_update(prop, NC_OBJECT | ND_TRANSFORM, 
"rna_Object_internal_update");
 
@@ -2326,7 +2323,7 @@ static void rna_def_object(BlenderRNA *brna)
 
        prop = RNA_def_property(srna, "delta_rotation_quaternion", PROP_FLOAT, 
PROP_QUATERNION);
        RNA_def_property_float_sdna(prop, NULL, "dquat");
-       RNA_def_property_float_array_default(prop, default_quat);
+       RNA_def_property_float_array_default(prop, rna_default_quaternion);
        RNA_def_property_ui_text(prop, "Delta Rotation (Quaternion)",
                                 "Extra rotation added to the rotation of the 
object (when using Quaternion rotations)");
        RNA_def_property_update(prop, NC_OBJECT | ND_TRANSFORM, 
"rna_Object_internal_update");
@@ -2335,7 +2332,7 @@ static void rna_def_object(BlenderRNA *brna)
        prop = RNA_def_property(srna, "delta_rotation_axis_angle", PROP_FLOAT, 
PROP_AXISANGLE);
        /* FIXME: this is not a single field any more! (drotAxis and drotAngle) 
*/
        RNA_def_property_float_sdna(prop, NULL, "dquat");
-       RNA_def_property_float_array_default(prop, default_axisAngle);
+       RNA_def_property_float_array_default(prop, rna_default_axis_angle);
        RNA_def_property_ui_text(prop, "Delta Rotation (Axis Angle)",
                                 "Extra rotation added to the rotation of the 
object (when using Axis-Angle rotations)");
        RNA_def_property_update(prop, NC_OBJECT | ND_TRANSFORM, 
"rna_Object_internal_update");
@@ -2345,7 +2342,7 @@ static void rna_def_object(BlenderRNA *brna)
        RNA_def_property_float_sdna(prop, NULL, "dscale");
        RNA_def_property_flag(prop, PROP_PROPORTIONAL);
        RNA_def_property_ui_range(prop, -FLT_MAX, FLT_MAX, 1, 3);
-       RNA_def_property_float_array_default(prop, default_scale);
+       RNA_def_property_float_array_default(prop, rna_default_scale_3d);
        RNA_def_property_ui_text(prop, "Delta Scale", "Extra scaling added to 
the scale of the object");
        RNA_def_property_update(prop, NC_OBJECT | ND_TRANSFORM, 
"rna_Object_internal_update");
 
diff --git a/source/blender/makesrna/intern/rna_pose.c 
b/source/blender/makesrna/intern/rna_pose.c
index eff8506ed75..118943dab2e 100644
--- a/source/blender/makesrna/intern/rna_pose.c
+++ b/source/blender/makesrna/intern/rna_pose.c
@@ -849,10 +849,6 @@ static void rna_def_pose_channel_constraints(BlenderRNA 
*brna, PropertyRNA *cpro
 
 static void rna_def_pose_channel(BlenderRNA *brna)
 {
-       static float default_quat[4] = {1, 0, 0, 0};    /* default quaternion 
values */
-       static float default_axisAngle[4] = {0, 0, 1, 0};   /* default 
axis-angle rotation values */
-       static float default_scale[3] = {1, 1, 1}; /* default scale values */
-
        StructRNA *srna;
        PropertyRNA *prop;
 
@@ -916,7 +912,7 @@ static void rna_def_pose_channel(BlenderRNA *brna)
        RNA_def_property_flag(prop, PROP_PROPORTIONAL);
        RNA_def_property_override_flag(prop, PROPOVERRIDE_OVERRIDABLE_STATIC);
        RNA_def_property_editable_array_func(prop, 
"rna_PoseChannel_scale_editable");
-       RNA_def_property_float_array_default(prop, default_scale);
+       RNA_def_property_float_array_default(prop, rna_default_scale_3d);
        RNA_def_property_ui_text(prop, "Scale", "");
        RNA_def_property_update(prop, NC_OBJECT | ND_POSE, 
"rna_Pose_IK_update");
 
@@ -924,7 +920,7 @@ static void rna_def_pose_channel(BlenderRNA *brna)
        RNA_def_property_float_sdna(prop, NULL, "quat");
        RNA_def_property_override_flag(prop, PROPOVERRIDE_OVERRIDABLE_STATIC);
        RNA_def_property_editable_array_func(prop, 
"rna_PoseChannel_rotation_4d_editable");
-       RNA_def_property_float_array_default(prop, default_quat);
+       RNA_def_property_float_array_default(prop, rna_default_quaternion);
        RNA_def_property_ui_text(prop, "Quaternion Rotation", "Rotation in 
Quaternions");
        RNA_def_property_update(prop, NC_OBJECT | ND_POSE, "rna_Pose_update");
 
@@ -937,7 +933,7 @@ static void rna_def_pose_channel(BlenderRNA *brna)
        RNA_def_property_float_funcs(prop, 
"rna_PoseChannel_rotation_axis_angle_get",
                                     "rna_PoseChannel_rotation_axis_angle_set", 
NULL);
        RNA_def_property_editable_array_func(prop, 
"rna_PoseChannel_rotation_4d_editable");
-       RNA_def_property_float_array_default(prop, default_axisAngle);
+       RNA_def_property_float_array_default(prop, rna_default_axis_angle);
        RNA_def_property_ui_text(prop, "Axis-Angle Rotation", "Angle of 
Rotation for Axis-Angle rotation representation");
        RNA_def_property_update(prop, NC_OBJECT | ND_POSE, "rna_Pose_update");
 
diff --git a/source/blender/makesrna/intern/rna_scene.c 
b/source/blender/makesrna/intern/rna_scene.c
index ecf09837c0a..93bbc214085 100644
--- a/source/blender/makesrna/intern/rna_scene.c
+++ b/source/blender/makesrna/intern/rna_scene.c
@@ -2347,9 +2347,6 @@ static void rna_def_transform_orientation_slot(BlenderRNA 
*brna)
 
 static void rna_def_view3d_cursor(BlenderRNA *brna)
 {
-       static float default_quat[4] = {1, 0, 0, 0};    /* default quaternion 
values */
-       static float default_axisAngle[4] = {0, 0, 1, 0};   /* default 
axis-angle rotation values */
-
        StructRNA *srna;
        PropertyRNA *prop;
 
@@ -2367,7 +2364,7 @@ static void rna_def_view3d_cursor(BlenderRNA *brna)
        prop = RNA_def_property(srna, "rotation_quaternion", PROP_FLOAT, 
PROP_QUATERNION);
        RNA_def_property_clear_flag(prop, PROP_ANIMATABLE);
        RNA_def_property_float_sdna(prop, NULL, "rotation_quaternion");
-       RNA_def_property_float_array_default(prop, default_quat);
+       RNA_def_property_float_array_default(prop, rna_default_quaternion);
        RNA_def_property_ui_text(prop, "Quaternion Rotation", "Rotation in 
quaternions (keep normalized)");
        RNA_def_property_update(prop, NC_WINDOW, NULL);
 
@@ -2376,7 +2373,7 @@ static void rna_def_view3d_cursor(BlenderRNA *brna)
        RNA_def_property_array(prop, 4);
        RNA_def_property_float_funcs(prop, 
"rna_View3DCursor_rotation_axis_angle_get",
                                     
"rna_View3DCursor_rotation_axis_angle_set", NULL);
-       RNA_def_property_float_array_default(prop, default_axisAngle);
+       RNA_def_property_float_array_default(prop, rna_default_axis_angle);
        RNA_def_property_ui_text(prop, "Axis-Angle Rotation", "Angle of 
Rotation for Axis-Angle rotation representation");
        RNA_def_property_update(prop, NC_WINDOW, NULL);

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

Reply via email to