Clean up some casts.
authorAlex Converse <aconverse@google.com>
Fri, 24 Jul 2015 21:59:03 +0000 (14:59 -0700)
committerAlex Converse <aconverse@google.com>
Wed, 29 Jul 2015 18:26:51 +0000 (11:26 -0700)
Change-Id: I264ca534cd7d4755906e20aea47e7a2523bca611

vp9/encoder/vp9_ratectrl.c

index 2be2a64..7427ccf 100644 (file)
@@ -281,14 +281,14 @@ int vp9_rc_get_default_min_gf_interval(
   // Assume we do not need any constraint lower than 4K 20 fps
   static const double factor_safe = 3840 * 2160 * 20.0;
   const double factor = width * height * framerate;
-  const double default_interval =
-      MIN(MAX_GF_INTERVAL, MAX(MIN_GF_INTERVAL, (int)(framerate * 0.125)));
+  const int default_interval =
+      clamp((int)(framerate * 0.125), MIN_GF_INTERVAL, MAX_GF_INTERVAL);
 
   if (factor <= factor_safe)
-    return (int)default_interval;
+    return default_interval;
   else
-    return (int)MAX(default_interval,
-                    (int)(MIN_GF_INTERVAL * factor / factor_safe + 0.5));
+    return MAX(default_interval,
+               (int)(MIN_GF_INTERVAL * factor / factor_safe + 0.5));
   // Note this logic makes:
   // 4K24: 5
   // 4K30: 6