luc14n0/glib
1
0
Fork 0
mirror of https://gitlab.gnome.org/GNOME/glib.git synced 2025-05-21 21:12:11 +02:00
Code Issues Packages Projects Releases Wiki Activity

Improve MIN, MAX, ABS, CLAMP macros

Browse Source 
When using gcc or clang, define these common macros in a way
that only evaluates the argument once. This is the intention.
This commit is contained in:
Matthias Clasen 2025-03-15 20:31:25 -04:00
parent ddf2839762
commit c2c9db0629
1 changed files with 42 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

49
glib/gmacros.h
View File

@ -936,20 +936,55 @@
#endif
#undef MAX
#define MAX(a, b) (((a) > (b)) ? (a) : (b))
#undef MIN
#define MIN(a, b) (((a) < (b)) ? (a) : (b))
#undef ABS
#define ABS(a) (((a) < 0) ? -(a) : (a))
#undef CLAMP
#define CLAMP(x, low, high) (((x) > (high)) ? (high) : (((x) < (low)) ? (low) : (x)))
#ifdef glib_typeof
#define MAX(a, b) \
(G_GNUC_EXTENSION ({ \
glib_typeof(a) _a = (a); \
glib_typeof(b) _b = (b); \
_a > _b ? _a : _b; \
}))
#define MIN(a, b) \
(G_GNUC_EXTENSION ({ \
glib_typeof(a) _a = (a); \
glib_typeof(b) _b = (b); \
_a < _b ? _a : _b; \
}))
#define ABS(a) \
(G_GNUC_EXTENSION ({ \
glib_typeof(a) _a = (a); \
_a < 0 ? -_a : _a; \
}))
#define CLAMP(x,low,high) \
(G_GNUC_EXTENSION ({ \
glib_typeof(x) _x = (x); \
glib_typeof(low) _low = (low); \
glib_typeof(high) _high = (high); \
_x > _high ? _high : (_x < _low ? _low : _x); \
}))
#define G_APPROX_VALUE(a,b,epsilon) \
(G_GNUC_EXTENSION ({ \
glib_typeof(a) _a = (a); \
glib_typeof(b) _b = (b); \
glib_typeof(epsilon) _epsilon = (epsilon); \
((_a > _b ? _a - _b : _b - _a) < _epsilon); \
}))
#else
#define MAX(a, b) (((a) > (b)) ? (a) : (b))
#define MIN(a, b) (((a) < (b)) ? (a) : (b))
#define ABS(a) (((a) < 0) ? -(a) : (a))
#define CLAMP(x, low, high) (((x) > (high)) ? (high) : (((x) < (low)) ? (low) : (x)))
#define G_APPROX_VALUE(a, b, epsilon) \
(((a) > (b) ? (a) - (b) : (b) - (a)) < (epsilon))
#endif
/* Count the number of elements in an array. The array must be defined
* as such; using this with a dynamically allocated array will give
* incorrect results.