Cleanup: remove flag from GizmoCommonData
authorCampbell Barton <ideasman42@gmail.com>
Mon, 3 Dec 2018 02:01:04 +0000 (13:01 +1100)
committerCampbell Barton <ideasman42@gmail.com>
Mon, 3 Dec 2018 02:01:04 +0000 (13:01 +1100)
Use struct member instead,
there are already many gizmo flags to keep track of.

source/blender/editors/gizmo_library/gizmo_library_intern.h
source/blender/editors/gizmo_library/gizmo_library_utils.c
source/blender/editors/gizmo_library/gizmo_types/arrow3d_gizmo.c

index db88a52f087c2b20abf35625a8a1382d2fae3d87..f58935e1fba6a2ad2944c8fbf4536068aa0a9155 100644 (file)
@@ -37,8 +37,6 @@
  * Data for common interactions. Used in gizmo_library_utils.c functions.
  */
 typedef struct GizmoCommonData {
-       int flag;
-
        float range_fac;      /* factor for arrow min/max distance */
        float offset;
 
@@ -46,6 +44,8 @@ typedef struct GizmoCommonData {
        float range;
        /* min/max value for constrained gizmos */
        float min, max;
+
+       uint is_custom_range_set : 1;
 } GizmoCommonData;
 
 typedef struct GizmoInteraction {
@@ -62,12 +62,6 @@ typedef struct GizmoInteraction {
        float precision_offset;
 } GizmoInteraction;
 
-/* GizmoCommonData->flag  */
-enum {
-       GIZMO_CUSTOM_RANGE_SET = (1 << 0),
-};
-
-
 float gizmo_offset_from_value(
         GizmoCommonData *data, const float value,
         const bool constrained, const bool inverted);
index 253646527852e92a978258025284341bdd43adb8..d8c29eb6675e64b7cf312d1bb762f5c9451ebfa9 100644 (file)
@@ -99,7 +99,7 @@ float gizmo_value_from_offset(
        }
 
        /* clamp to custom range */
-       if (data->flag & GIZMO_CUSTOM_RANGE_SET) {
+       if (data->is_custom_range_set) {
                CLAMP(value, data->min, max);
        }
 
@@ -124,7 +124,7 @@ void gizmo_property_data_update(
        float value = WM_gizmo_target_property_float_get(gz, gz_prop);
 
        if (constrained) {
-               if ((data->flag & GIZMO_CUSTOM_RANGE_SET) == 0) {
+               if (data->is_custom_range_set == false) {
                        float range[2];
                        if (WM_gizmo_target_property_float_range_get(gz, gz_prop, range)) {
                                data->range = range[1] - range[0];
index 1e39e1cca525cdb3e6c9be9a0802a83823f8d6ef..e02d4b8470cfd3beab90e84b918aa47d9bd3fcb5 100644 (file)
@@ -416,7 +416,8 @@ void ED_gizmo_arrow3d_set_ui_range(wmGizmo *gz, const float min, const float max
 
        arrow->data.range = max - min;
        arrow->data.min = min;
-       arrow->data.flag |= GIZMO_CUSTOM_RANGE_SET;
+       arrow->data.max = max;
+       arrow->data.is_custom_range_set = true;
 }
 
 /**