summaryrefslogtreecommitdiff
path: root/ext/reflection
diff options
context:
space:
mode:
authorsj-i <sji@sj-i.dev>2021-01-19 02:02:09 +0900
committerNikita Popov <nikita.ppv@gmail.com>2021-01-19 09:49:39 +0100
commit6a1e74eb323813d183abdaccb0bb504f5979b199 (patch)
tree338d51ef1f526089a247f9a0105da5dc5dc35b0a /ext/reflection
parenta9efcb1561a10bf0324f23c0f30b9447aa753a47 (diff)
downloadphp-git-6a1e74eb323813d183abdaccb0bb504f5979b199.tar.gz
Delete an outdated comment for read_property and read_dimension. [ci skip]
The comment is outdated as of PHP7. For example, in the current code, zend_assign_op_overloaded_property() expects the refcount of the returned value from read_property to be greater than 0 when the returned value is a refcounted, to call zval_ptr_dtor() later. History: - The same description was originally written in a commit in 2004 alongside the write side. https://github.com/php/php-src/commit/c8c0e979826b677de5f786920289056242ea5c54 - One requested to put the description in somewhere. https://externals.io/message/7789 - Then it was added as a comment in zend_object_handler.h . https://github.com/php/php-src/commit/7d3215d33321173c58db4d86b7398e16c5c55d13 - At the time the comment was written, there were at least three places where the code actually set the reference count to 0 in read_properties. - https://github.com/php/php-src/blob/7d3215d33321173c58db4d86b7398e16c5c55d13/ext/dom/php_dom.c#L229 - https://github.com/php/php-src/blob/7d3215d33321173c58db4d86b7398e16c5c55d13/ext/mysqli/mysqli.c#L190 - https://github.com/php/php-src/blob/7d3215d33321173c58db4d86b7398e16c5c55d13/ext/simplexml/simplexml.c#L245 - All three of the above were removed during the development of PHP 7. - https://github.com/php/php-src/commit/2f0a758fbbf39ff8684d167f86c708cc361db782 - https://github.com/php/php-src/commit/2402d6cbbc5e04362b23b183f9129a8db230bcce - https://github.com/php/php-src/commit/a975c7e0fe95a94c119eb1c8e519b2357aa50dcc - In the current code, even when they generate and return a refcounted value, its refcount would be 1. Closes GH-6618.
Diffstat (limited to 'ext/reflection')
0 files changed, 0 insertions, 0 deletions