Convert macros to inline functions in Zend SmartStr

Author: Girgias

Zend/zend_smart_str.c

@@ -73,6 +73,18 @@ static size_t zend_compute_escaped_string_len(const char *s, size_t l) {
 	return len;
 }
 
+extern ZEND_API inline void smart_str_appendl(smart_str *dest, const char *src, size_t length);
+extern ZEND_API inline void smart_str_appends_ex(smart_str *dest, const char *src, bool persistent);
+extern ZEND_API inline void smart_str_appends(smart_str *dest, const char *src);
+extern ZEND_API inline void smart_str_append(smart_str *dest, const zend_string *src);
+extern ZEND_API inline void smart_str_appendc(smart_str *dest, char ch);
+extern ZEND_API inline void smart_str_append_smart_str(smart_str *dest, const smart_str *src);
+extern ZEND_API inline void smart_str_free(smart_str *str);
+extern ZEND_API inline void smart_str_append_long(smart_str *dest, zend_long num);
+extern ZEND_API inline void smart_str_append_unsigned(smart_str *dest, zend_ulong num);
+extern ZEND_API inline void smart_str_sets(smart_str *dest, const char *src);
+
+
 ZEND_API void ZEND_FASTCALL smart_str_append_escaped(smart_str *str, const char *s, size_t l) {
 	char *res;
 	size_t i, len = zend_compute_escaped_string_len(s, l);

Zend/zend_smart_str.h

@@ -21,28 +21,30 @@
 #include "zend_globals.h"
 #include "zend_smart_str_public.h"
 
-#define smart_str_appends_ex(dest, src, what) \
-	smart_str_appendl_ex((dest), (src), strlen(src), (what))
-#define smart_str_appends(dest, src) \
-	smart_str_appendl((dest), (src), strlen(src))
-#define smart_str_extend(dest, len) \
-	smart_str_extend_ex((dest), (len), 0)
-#define smart_str_appendc(dest, c) \
-	smart_str_appendc_ex((dest), (c), 0)
-#define smart_str_appendl(dest, src, len) \
-	smart_str_appendl_ex((dest), (src), (len), 0)
-#define smart_str_append(dest, src) \
-	smart_str_append_ex((dest), (src), 0)
-#define smart_str_append_smart_str(dest, src) \
-	smart_str_append_smart_str_ex((dest), (src), 0)
-#define smart_str_sets(dest, src) \
-	smart_str_setl((dest), (src), strlen(src));
-#define smart_str_append_long(dest, val) \
-	smart_str_append_long_ex((dest), (val), 0)
-#define smart_str_append_unsigned(dest, val) \
-	smart_str_append_unsigned_ex((dest), (val), 0)
-#define smart_str_free(dest) \
-	smart_str_free_ex((dest), 0)
+ZEND_API inline void smart_str_appendl(smart_str *dest, const char *src, size_t length)
+{
+	smart_str_appendl_ex(dest, src, length, 0);
+}
+ZEND_API inline void smart_str_appends_ex(smart_str *dest, const char *src, bool persistent)
+{
+	smart_str_appendl_ex(dest, src, strlen(src), persistent);
+}
+ZEND_API inline void smart_str_appends(smart_str *dest, const char *src)
+{
+	smart_str_appendl_ex(dest, src, strlen(src), 0);
+}
+ZEND_API inline void smart_str_append(smart_str *dest, const zend_string *src)
+{
+	smart_str_append_ex(dest, src, 0);
+}
+ZEND_API inline void smart_str_appendc(smart_str *dest, char ch)
+{
+	smart_str_appendc_ex(dest, ch, 0);
+}
+ZEND_API inline void smart_str_append_smart_str(smart_str *dest, const smart_str *src)
+{
+	smart_str_append_smart_str_ex(dest, src, 0);
+}
 
 BEGIN_EXTERN_C()
 
@@ -78,6 +80,11 @@ static zend_always_inline char* smart_str_extend_ex(smart_str *dest, size_t len,
 	return ret;
 }
 
+ZEND_API inline char* smart_str_extend(smart_str *dest, size_t length)
+{
+	return smart_str_extend_ex(dest, length, 0);
+}
+
 static zend_always_inline void smart_str_free_ex(smart_str *str, zend_bool persistent) {
 	if (str->s) {
 		zend_string_release_ex(str->s, persistent);
@@ -86,6 +93,11 @@ static zend_always_inline void smart_str_free_ex(smart_str *str, zend_bool persi
 	str->a = 0;
 }
 
+ZEND_API inline void smart_str_free(smart_str *str)
+{
+	smart_str_free_ex(str, 0);
+}
+
 static zend_always_inline void smart_str_0(smart_str *str) {
 	if (str->s) {
 		ZSTR_VAL(str->s)[ZSTR_LEN(str->s)] = '\0';
@@ -136,15 +148,29 @@ static zend_always_inline void smart_str_append_long_ex(smart_str *dest, zend_lo
 	smart_str_appendl_ex(dest, result, buf + sizeof(buf) - 1 - result, persistent);
 }
 
+ZEND_API inline void smart_str_append_long(smart_str *dest, zend_long num)
+{
+	smart_str_append_long_ex(dest, num, 0);
+}
+
 static zend_always_inline void smart_str_append_unsigned_ex(smart_str *dest, zend_ulong num, zend_bool persistent) {
 	char buf[32];
 	char *result = zend_print_ulong_to_buf(buf + sizeof(buf) - 1, num);
 	smart_str_appendl_ex(dest, result, buf + sizeof(buf) - 1 - result, persistent);
 }
 
+ZEND_API inline void smart_str_append_unsigned(smart_str *dest, zend_ulong num)
+{
+	smart_str_append_unsigned_ex(dest, num, 0);
+}
+
 static zend_always_inline void smart_str_setl(smart_str *dest, const char *src, size_t len) {
 	smart_str_free(dest);
 	smart_str_appendl(dest, src, len);
 }
 
+ZEND_API inline void smart_str_sets(smart_str *dest, const char *src)
+{
+	smart_str_setl(dest, src, strlen(src));
+}
 #endif