summaryrefslogtreecommitdiff
path: root/ext/reflection/php_reflection.c
diff options
context:
space:
mode:
authorDmitry Stogov <dmitry@zend.com>2015-03-12 16:53:51 +0300
committerDmitry Stogov <dmitry@zend.com>2015-03-12 16:53:51 +0300
commit8633685675692e9a926cf9f97b1c406c37497ddf (patch)
treee0c82ad945a575f90f90dfd402c7e2dc013a3acc /ext/reflection/php_reflection.c
parent0e30c543ec8e6c371e0aef6e125e7b90f4b1b790 (diff)
downloadphp-git-8633685675692e9a926cf9f97b1c406c37497ddf.tar.gz
Use specialized macro for string zval creation
Diffstat (limited to 'ext/reflection/php_reflection.c')
-rw-r--r--ext/reflection/php_reflection.c30
1 files changed, 15 insertions, 15 deletions
diff --git a/ext/reflection/php_reflection.c b/ext/reflection/php_reflection.c
index 80e04046ed..4ab0559eda 100644
--- a/ext/reflection/php_reflection.c
+++ b/ext/reflection/php_reflection.c
@@ -1208,7 +1208,7 @@ static void reflection_parameter_factory(zend_function *fptr, zval *closure_obje
if (fptr->type == ZEND_INTERNAL_FUNCTION) {
ZVAL_STRING(&name, ((zend_internal_arg_info*)arg_info)->name);
} else {
- ZVAL_STR(&name, zend_string_copy(arg_info->name));
+ ZVAL_STR_COPY(&name, arg_info->name);
}
} else {
ZVAL_NULL(&name);
@@ -1636,7 +1636,7 @@ ZEND_METHOD(reflection_function, __toString)
GET_REFLECTION_OBJECT_PTR(fptr);
string_init(&str);
_function_string(&str, fptr, intern->ce, "");
- RETURN_STR(str.buf);
+ RETURN_NEW_STR(str.buf);
}
/* }}} */
@@ -1777,7 +1777,7 @@ ZEND_METHOD(reflection_function, getFileName)
}
GET_REFLECTION_OBJECT_PTR(fptr);
if (fptr->type == ZEND_USER_FUNCTION) {
- RETURN_STR(zend_string_copy(fptr->op_array.filename));
+ RETURN_STR_COPY(fptr->op_array.filename);
}
RETURN_FALSE;
}
@@ -1831,7 +1831,7 @@ ZEND_METHOD(reflection_function, getDocComment)
}
GET_REFLECTION_OBJECT_PTR(fptr);
if (fptr->type == ZEND_USER_FUNCTION && fptr->op_array.doc_comment) {
- RETURN_STR(zend_string_copy(fptr->op_array.doc_comment));
+ RETURN_STR_COPY(fptr->op_array.doc_comment);
}
RETURN_FALSE;
}
@@ -2288,7 +2288,7 @@ ZEND_METHOD(reflection_parameter, __construct)
if (fptr->type == ZEND_INTERNAL_FUNCTION) {
ZVAL_STRING(&name, ((zend_internal_arg_info*)arg_info)[position].name);
} else {
- ZVAL_STR(&name, zend_string_copy(arg_info[position].name));
+ ZVAL_STR_COPY(&name, arg_info[position].name);
}
} else {
ZVAL_NULL(&name);
@@ -2324,7 +2324,7 @@ ZEND_METHOD(reflection_parameter, __toString)
GET_REFLECTION_OBJECT_PTR(param);
string_init(&str);
_parameter_string(&str, param->fptr, param->arg_info, param->offset, param->required, "");
- RETURN_STR(str.buf);
+ RETURN_NEW_STR(str.buf);
}
/* }}} */
@@ -2668,7 +2668,7 @@ ZEND_METHOD(reflection_parameter, getDefaultValueConstantName)
precv = _reflection_param_get_default_precv(INTERNAL_FUNCTION_PARAM_PASSTHRU, param);
if (precv && Z_TYPE_P(RT_CONSTANT(&param->fptr->op_array, precv->op2)) == IS_CONSTANT) {
- RETURN_STR(zend_string_copy(Z_STR_P(RT_CONSTANT(&param->fptr->op_array, precv->op2))));
+ RETURN_STR_COPY(Z_STR_P(RT_CONSTANT(&param->fptr->op_array, precv->op2)));
}
}
/* }}} */
@@ -2805,7 +2805,7 @@ ZEND_METHOD(reflection_method, __toString)
GET_REFLECTION_OBJECT_PTR(mptr);
string_init(&str);
_function_string(&str, mptr, intern->ce, "");
- RETURN_STR(str.buf);
+ RETURN_NEW_STR(str.buf);
}
/* }}} */
@@ -3530,7 +3530,7 @@ ZEND_METHOD(reflection_class, __toString)
GET_REFLECTION_OBJECT_PTR(ce);
string_init(&str);
_class_string(&str, ce, &intern->obj, "");
- RETURN_STR(str.buf);
+ RETURN_NEW_STR(str.buf);
}
/* }}} */
@@ -3587,7 +3587,7 @@ ZEND_METHOD(reflection_class, getFileName)
}
GET_REFLECTION_OBJECT_PTR(ce);
if (ce->type == ZEND_USER_CLASS) {
- RETURN_STR(zend_string_copy(ce->info.user.filename));
+ RETURN_STR_COPY(ce->info.user.filename);
}
RETURN_FALSE;
}
@@ -3641,7 +3641,7 @@ ZEND_METHOD(reflection_class, getDocComment)
}
GET_REFLECTION_OBJECT_PTR(ce);
if (ce->type == ZEND_USER_CLASS && ce->info.user.doc_comment) {
- RETURN_STR(zend_string_copy(ce->info.user.doc_comment));
+ RETURN_STR_COPY(ce->info.user.doc_comment);
}
RETURN_FALSE;
}
@@ -4883,7 +4883,7 @@ ZEND_METHOD(reflection_property, __toString)
GET_REFLECTION_OBJECT_PTR(ref);
string_init(&str);
_property_string(&str, &ref->prop, NULL, "");
- RETURN_STR(str.buf);
+ RETURN_NEW_STR(str.buf);
}
/* }}} */
@@ -5134,7 +5134,7 @@ ZEND_METHOD(reflection_property, getDocComment)
}
GET_REFLECTION_OBJECT_PTR(ref);
if (ref->prop.doc_comment) {
- RETURN_STR(zend_string_copy(ref->prop.doc_comment));
+ RETURN_STR_COPY(ref->prop.doc_comment);
}
RETURN_FALSE;
}
@@ -5222,7 +5222,7 @@ ZEND_METHOD(reflection_extension, __toString)
GET_REFLECTION_OBJECT_PTR(module);
string_init(&str);
_extension_string(&str, module, "");
- RETURN_STR(str.buf);
+ RETURN_NEW_STR(str.buf);
}
/* }}} */
@@ -5588,7 +5588,7 @@ ZEND_METHOD(reflection_zend_extension, __toString)
GET_REFLECTION_OBJECT_PTR(extension);
string_init(&str);
_zend_extension_string(&str, extension, "");
- RETURN_STR(str.buf);
+ RETURN_NEW_STR(str.buf);
}
/* }}} */