gaudieffects: use CLAMP in dodge
authorLuis de Bethencourt <luis@debethencourt.com>
Thu, 17 May 2012 08:45:52 +0000 (09:45 +0100)
committerLuis de Bethencourt <luis@debethencourt.com>
Thu, 17 May 2012 09:54:31 +0000 (10:54 +0100)
No need to have a gate_int () function duplicating the already existing and
established CLAMP () function.

gst/gaudieffects/gstdodge.c

index 062ea6809b61348d6e2e4bfc679b5ac8d6d8fd1a..14ef7c446a6acf264c02d709323b9effb034fcfc 100644 (file)
@@ -93,7 +93,6 @@ enum
 
 /* Initializations */
 
-static gint gate_int (gint value, gint min, gint max);
 static void transform (guint32 * src, guint32 * dest, gint video_area);
 
 /* The capabilities of the inputs and outputs. */
@@ -256,25 +255,12 @@ gst_dodge_plugin_init (GstPlugin * dodge)
 
 /*** Now the image processing work.... ***/
 
-/* Keep the values inbounds. */
-static gint
-gate_int (gint value, gint min, gint max)
-{
-  if (value < min) {
-    return min;
-  } else if (value > max) {
-    return max;
-  } else {
-    return value;
-  }
-}
-
 /* Transform processes each frame. */
 static void
 transform (guint32 * src, guint32 * dest, gint video_area)
 {
-  guint32 in, red, green, blue;
-  gint x;
+  guint32 in;
+  gint x, red, green, blue;
 
   for (x = 0; x < video_area; x++) {
     in = *src++;
@@ -287,9 +273,9 @@ transform (guint32 * src, guint32 * dest, gint video_area)
     green = (256 * green) / (256 - green);
     blue = (256 * blue) / (256 - blue);
 
-    red = gate_int (red, 0, 255);
-    green = gate_int (green, 0, 255);
-    blue = gate_int (blue, 0, 255);
+    red = CLAMP (red, 0, 255);
+    green = CLAMP (green, 0, 255);
+    blue = CLAMP (blue, 0, 255);
 
     *dest++ = (red << 16) | (green << 8) | blue;
   }