structure: Use GLib's CLAMP macro for fixating ranges
authorJan Alexander Steffens (heftig) <jsteffens@make.tv>
Wed, 9 Jan 2019 12:37:30 +0000 (13:37 +0100)
committerTim-Philipp Müller <tim@centricular.com>
Wed, 9 Jan 2019 23:03:08 +0000 (23:03 +0000)
Just a bit of refactoring.

gst/gststructure.c

index d9b1dcf..43686b2 100644 (file)
@@ -2208,14 +2208,13 @@ gst_structure_fixate_field_nearest_int (GstStructure * structure,
     /* already fixed */
     return FALSE;
   } else if (G_VALUE_TYPE (value) == GST_TYPE_INT_RANGE) {
-    int x;
+    int min, max;
+
+    min = gst_value_get_int_range_min (value);
+    max = gst_value_get_int_range_max (value);
+
+    target = CLAMP (target, min, max);
 
-    x = gst_value_get_int_range_min (value);
-    if (target < x)
-      target = x;
-    x = gst_value_get_int_range_max (value);
-    if (target > x)
-      target = x;
     gst_structure_set (structure, field_name, G_TYPE_INT, target, NULL);
     return TRUE;
   } else if (G_VALUE_TYPE (value) == GST_TYPE_LIST) {