From b0eacbf35013982a63b663b84d85e06f8a9f63fa Mon Sep 17 00:00:00 2001 From: rubidium Date: Wed, 9 Dec 2009 00:41:18 +0000 Subject: (svn r18438) -Codechange [FS#3337]: introduce a template for Clamp (fonsinchen) and reuse template functions in their non-template counterparts. --- src/core/math_func.hpp | 41 ++++++++++++++++++++++++++++++----------- 1 file changed, 30 insertions(+), 11 deletions(-) (limited to 'src/core') diff --git a/src/core/math_func.hpp b/src/core/math_func.hpp index b5e921a7c..576db697d 100644 --- a/src/core/math_func.hpp +++ b/src/core/math_func.hpp @@ -67,7 +67,7 @@ static FORCEINLINE T min(const T a, const T b) */ static FORCEINLINE int min(const int a, const int b) { - return (a < b) ? a : b; + return min(a, b); } /** @@ -81,7 +81,7 @@ static FORCEINLINE int min(const int a, const int b) */ static FORCEINLINE uint minu(const uint a, const uint b) { - return (a < b) ? a : b; + return min(a, b); } /** @@ -131,7 +131,7 @@ static FORCEINLINE T *AlignPtr(T *x, uint n) } /** - * Clamp an integer between an interval. + * Clamp a value between an interval. * * This function returns a value which is between the given interval of * min and max. If the given value is in this interval the value itself @@ -145,14 +145,37 @@ static FORCEINLINE T *AlignPtr(T *x, uint n) * @param max the maximum of the interval. * @returns A value between min and max which is closest to a. * @see ClampU(uint, uint, uint) + * @see Clamp(int, int, int) */ -static FORCEINLINE int Clamp(const int a, const int min, const int max) +template +static FORCEINLINE T Clamp(const T a, const T min, const T max) { if (a <= min) return min; if (a >= max) return max; return a; } +/** + * Clamp an integer between an interval. + * + * This function returns a value which is between the given interval of + * min and max. If the given value is in this interval the value itself + * is returned otherwise the border of the interval is returned, according + * which side of the interval was 'left'. + * + * @note The min value must be less or equal of max or you get some + * unexpected results. + * @param a The value to clamp/truncate. + * @param min The minimum of the interval. + * @param max the maximum of the interval. + * @returns A value between min and max which is closest to a. + * @see ClampU(uint, uint, uint) + */ +static FORCEINLINE int Clamp(const int a, const int min, const int max) +{ + return Clamp(a, min, max); +} + /** * Clamp an unsigned integer between an interval. * @@ -171,9 +194,7 @@ static FORCEINLINE int Clamp(const int a, const int min, const int max) */ static FORCEINLINE uint ClampU(const uint a, const uint min, const uint max) { - if (a <= min) return min; - if (a >= max) return max; - return a; + return Clamp(a, min, max); } /** @@ -192,9 +213,7 @@ static FORCEINLINE uint ClampU(const uint a, const uint min, const uint max) */ static FORCEINLINE int32 ClampToI32(const int64 a) { - if (a <= INT32_MIN) return INT32_MIN; - if (a >= INT32_MAX) return INT32_MAX; - return (int32)a; + return (int32)Clamp(a, INT32_MIN, INT32_MAX); } /** @@ -206,7 +225,7 @@ static FORCEINLINE int32 ClampToI32(const int64 a) */ static FORCEINLINE uint16 ClampToU16(const uint64 a) { - return (uint16)(a <= UINT16_MAX ? a : UINT16_MAX); + return (uint16)min(a, UINT16_MAX); } /** -- cgit v1.2.3-54-g00ecf