diff options
Diffstat (limited to 'lib/ccan/cast')
| -rw-r--r-- | lib/ccan/cast/cast.h | 20 | ||||
| -rw-r--r-- | lib/ccan/cast/test/compile_ok-static.c | 10 |
2 files changed, 22 insertions, 8 deletions
diff --git a/lib/ccan/cast/cast.h b/lib/ccan/cast/cast.h index b108b0c864..1f3a7aac1e 100644 --- a/lib/ccan/cast/cast.h +++ b/lib/ccan/cast/cast.h @@ -15,8 +15,8 @@ * only differs in signed/unsigned, not in type or const-ness. */ #define cast_signed(type, expr) \ - ((type)(expr) \ - + BUILD_ASSERT_OR_ZERO(cast_sign_compatible(type, (expr)))) + (0 ? BUILD_ASSERT_OR_ZERO(cast_sign_compatible(type, (expr))) : \ + (type)(expr)) /** * cast_const - remove a const qualifier from a pointer. @@ -26,6 +26,10 @@ * This ensures that you are only removing the const qualifier from an * expression. The expression must otherwise match @type. * + * We cast via intptr_t to suppress gcc's -Wcast-qual (which SAMBA + * uses), and via the ? : so Sun CC doesn't complain about the result + * not being constant. + * * If @type is a pointer to a pointer, you must use cast_const2 (etc). * * Example: @@ -40,8 +44,8 @@ * } */ #define cast_const(type, expr) \ - ((type)((intptr_t)(expr) \ - + BUILD_ASSERT_OR_ZERO(cast_const_compat1((expr), type)))) + (0 ? BUILD_ASSERT_OR_ZERO(cast_const_compat1((expr), type)) : \ + (type)(intptr_t)(expr)) /** * cast_const2 - remove a const qualifier from a pointer to a pointer. @@ -52,8 +56,8 @@ * expression. The expression must otherwise match @type. */ #define cast_const2(type, expr) \ - ((type)((intptr_t)(expr) \ - + BUILD_ASSERT_OR_ZERO(cast_const_compat2((expr), type)))) + (0 ? BUILD_ASSERT_OR_ZERO(cast_const_compat2((expr), type)) : \ + (type)(intptr_t)(expr)) /** * cast_const3 - remove a const from a pointer to a pointer to a pointer.. @@ -64,8 +68,8 @@ * expression. The expression must otherwise match @type. */ #define cast_const3(type, expr) \ - ((type)((intptr_t)(expr) \ - + BUILD_ASSERT_OR_ZERO(cast_const_compat3((expr), type)))) + (0 ? BUILD_ASSERT_OR_ZERO(cast_const_compat3((expr), type)) : \ + (type)(intptr_t)(expr)) /** diff --git a/lib/ccan/cast/test/compile_ok-static.c b/lib/ccan/cast/test/compile_ok-static.c new file mode 100644 index 0000000000..98b667e834 --- /dev/null +++ b/lib/ccan/cast/test/compile_ok-static.c @@ -0,0 +1,10 @@ +/* OpenIndiana's CC (aka suncc) has issues with constants: make sure + * we are one! */ +#include <ccan/cast/cast.h> + +static char *p = cast_const(char *, (const char *)"hello"); + +int main(int argc, char *argv[]) +{ + return p[0] == argv[0][0]; +} |