From 2872f8239739c0a6dddeacf4624b6d90b67f72da Mon Sep 17 00:00:00 2001
From: "Kevin P. Fleming" <kpfleming@digium.com>
Date: Tue, 11 Nov 2008 17:57:20 +0000
Subject: [PATCH] use some fancy compiler magic (thanks to Matthew Woehlke on
the gcc-help mailing list) to restore type-safety to S_OR by going back to a
macro, but preserve the side-effect-safe usage of the macro arguments
git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@155967 65c4cc65-6c06-0410-ace0-fbb531ad65f3
---
include/asterisk/strings.h | 10 ++--------
1 file changed, 2 insertions(+), 8 deletions(-)
diff --git a/include/asterisk/strings.h b/include/asterisk/strings.h
index 6c5eba7837..5ad362e569 100644
--- a/include/asterisk/strings.h
+++ b/include/asterisk/strings.h
@@ -52,19 +52,13 @@ static force_inline int ast_strlen_zero(const char *s)
/*! \brief returns the equivalent of logic or for strings:
* first one if not empty, otherwise second one.
*/
-static force_inline char *S_OR(const char *a, const char *b)
-{
- return ast_strlen_zero(a) ? (char *) b : (char *) a;
-}
+#define S_OR(a, b) ({typeof(&((a)[0])) __x = (a); ast_strlen_zero(__x) ? (b) : __x;})
/*! \brief returns the equivalent of logic or for strings, with an additional boolean check:
* second one if not empty and first one is true, otherwise third one.
* example: S_COR(usewidget, widget, "<no widget>")
*/
-static force_inline char *S_COR(unsigned char a, const char *b, const char *c)
-{
- return a && !ast_strlen_zero(b) ? (char *) b : (char *) c;
-}
+#define S_COR(a, b, c) ({typeof(&((b)[0])) __x = (b); (a) && !ast_strlen_zero(__x) ? (__x) : (c);})
/*!
\brief Gets a pointer to the first non-whitespace character in a string.
--
GitLab