summaryrefslogtreecommitdiff
path: root/ext/oci8
diff options
context:
space:
mode:
authorNikita Popov <nikita.ppv@gmail.com>2020-01-20 10:34:17 +0100
committerNikita Popov <nikita.ppv@gmail.com>2020-01-20 10:34:17 +0100
commit68112224221d47169c69ea353beeb73c58ebbe3a (patch)
tree6df6967fd6e3641706b14146f741c771e258ca21 /ext/oci8
parent5947437d47cc3bf8f289e52e6361fccad4b08ae3 (diff)
downloadphp-git-68112224221d47169c69ea353beeb73c58ebbe3a.tar.gz
Eliminate uses of ZVAL_ZVAL and friends
Instead add RETURN_COPY(_VALUE) macros will the expected behavior. RETURN_ZVAL doesn't make any sense since PHP 7, but has stuck around, probably because the alternative was to write directly to the return_value variable.
Diffstat (limited to 'ext/oci8')
-rw-r--r--ext/oci8/oci8_interface.c4
1 files changed, 1 insertions, 3 deletions
diff --git a/ext/oci8/oci8_interface.c b/ext/oci8/oci8_interface.c
index b48294e3a5..299788cab6 100644
--- a/ext/oci8/oci8_interface.c
+++ b/ext/oci8/oci8_interface.c
@@ -2382,11 +2382,9 @@ PHP_FUNCTION(oci_collection_element_get)
PHP_OCI_ZVAL_TO_COLLECTION(tmp, collection);
- if (php_oci_collection_element_get(collection, element_index, &value)) {
+ if (php_oci_collection_element_get(collection, element_index, return_value)) {
RETURN_FALSE;
}
-
- RETURN_ZVAL(&value, 1, 1);
}
/* }}} */