diff options
author | Nikita Popov <nikita.ppv@gmail.com> | 2020-05-26 14:06:36 +0200 |
---|---|---|
committer | Nikita Popov <nikita.ppv@gmail.com> | 2020-05-26 14:42:27 +0200 |
commit | 314ab47e55845862be62683e331aa247025d073d (patch) | |
tree | a18ac2d3ede754a0c19fdb0e3e495ef6443c77ce /ext/reflection/php_reflection.c | |
parent | fb370ecbe5a7ea397ea40ead52edb42959ebedd4 (diff) | |
download | php-git-314ab47e55845862be62683e331aa247025d073d.tar.gz |
Fix zend_assign_to_typed_ref() implementation
There was some confusion going on here regarding the original
value vs the copied value.
I've dropped the needs_copy variable, because this code is not
inlined, so it would always be true anyway.
What we need to do is perform a move-assignment of the copied
value (in which case we don't care about performing the assignment
before destroying garbage), and destroying the original value
for the VAR/TMP cases. This is a bit complicated by the fact that
references are passed in via a separate ref variable, so we can't
just ptr_dtor the original variable.
Diffstat (limited to 'ext/reflection/php_reflection.c')
0 files changed, 0 insertions, 0 deletions