as discussed on the -dev list, introduce a macro to simplify the

writing of common constructs like

	chan = !ast_strlen_zero(cdr->channel) ? cdr->channel : "<unknown>";

(or the if/then/else form) into

	chan = S_OR(cdr->channel, "<unknown>");

The name can be changed if we find a better (and not too long) one;
currently, it is S as String, OR as it mimics the behaviour of
the || operator, but applied to strings.



git-svn-id: http://svn.digium.com/svn/asterisk/trunk@14747 f38db490-d61c-443f-a65b-d21fe96a405b

1 files changed, 5 insertions, 0 deletions

diff --git a/include/asterisk/strings.h b/include/asterisk/strings.h
index c9015544a..13bb970ef 100644
--- a/include/asterisk/strings.h
+++ b/include/asterisk/strings.h
@@ -35,6 +35,11 @@ static force_inline int ast_strlen_zero(const char *s)
 	return (!s || (*s == '\0'));
 }
 
+/*! \brief returns the equivalent of logic or for strings:
+ * first one if not empty, otherwise second one.
+ */
+#define S_OR(a, b)	(!ast_strlen_zero(a) ? (a) : (b))
+
 /*!
   \brief Gets a pointer to the first non-whitespace character in a string.
   \param ast_skip_blanks function being used