value: micro-optimisation: avoid some unnecessary g_value_unset()
authorTim-Philipp Müller <tim.muller@collabora.co.uk>
Mon, 26 Dec 2011 11:26:05 +0000 (11:26 +0000)
committerTim-Philipp Müller <tim.muller@collabora.co.uk>
Mon, 26 Dec 2011 11:27:07 +0000 (11:27 +0000)
We know there's nothing to do here and can save us the function
calls and GValueTable lookups.

gst/gstvalue.c

index 4f2b80d..bf9bb13 100644 (file)
@@ -1315,8 +1315,9 @@ gst_value_free_fraction_range (GValue * value)
   GValue *vals = (GValue *) value->data[0].v_pointer;
 
   if (vals != NULL) {
-    g_value_unset (&vals[0]);
-    g_value_unset (&vals[1]);
+    /* we know the two values contain fractions without internal allocs */
+    /* g_value_unset (&vals[0]); */
+    /* g_value_unset (&vals[1]); */
     g_slice_free1 (2 * sizeof (GValue), vals);
     value->data[0].v_pointer = NULL;
   }
@@ -1465,8 +1466,9 @@ gst_value_set_fraction_range_full (GValue * value,
   gst_value_set_fraction (&end, numerator_end, denominator_end);
   gst_value_set_fraction_range (value, &start, &end);
 
-  g_value_unset (&start);
-  g_value_unset (&end);
+  /* we know the two values contain fractions without internal allocs */
+  /* g_value_unset (&start); */
+  /* g_value_unset (&end);   */
 }
 
 /**