compositor: remove check for below zero for unsigned value
authorLuis de Bethencourt <luis@debethencourt.com>
Fri, 28 Aug 2015 14:21:11 +0000 (15:21 +0100)
committerLuis de Bethencourt <luis@debethencourt.com>
Fri, 28 Aug 2015 14:21:13 +0000 (15:21 +0100)
CLAMP checks both if value is '< 0' and '> max'. Value will never be a negative
number since it is an unsigned integer. Removing that check and only checking if
it is bigger than max by using MIN().

CID 1320707

gst/compositor/compositor.c

index 9aa651f..66927ba 100644 (file)
@@ -362,10 +362,10 @@ clamp_rectangle (guint x, guint y, guint w, guint h, guint outer_width,
    * the case where (say, with negative xpos/ypos or w/h greater than the output
    * size) the non-obscured portion of the frame could be outside the bounds of
    * the video itself and hence not visible at all */
-  clamped.x = CLAMP (x, 0, outer_width);
-  clamped.y = CLAMP (y, 0, outer_height);
-  clamped.w = CLAMP (x2, 0, outer_width) - clamped.x;
-  clamped.h = CLAMP (y2, 0, outer_height) - clamped.y;
+  clamped.x = MIN (x, outer_width);
+  clamped.y = MIN (y, outer_height);
+  clamped.w = MIN (x2, outer_width) - clamped.x;
+  clamped.h = MIN (y2, outer_height) - clamped.y;
 
   return clamped;
 }