Re: [PATCH] extend zend_parse_parameters() functionality (2nd attempt)
| From: | Andrei Zmievski | Date: | Fri, 27 Jun 2003 18:33:01 +0000 |
| Subject: | Re: [PATCH] extend zend_parse_parameters() functionality (2nd attempt) | ||
| References: | 1 | Groups: | php.internals |
| Request: | Send a blank email to internals+get-2904@lists.php.net to get a copy of this message | ||
On Fri, 27 Jun 2003, fuhs@purdue.edu wrote:
> I propose that a new type, 'Z', be added in order to allow the extension coders
> access to the zval** which was available with the now deprecated
> zend_get_parameters().
>
> I came to this conclusion after tracing some segfaults to a section of code
> similar to this:
>
> zval *zend_value;
> if(zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "z", &zend_value) ==
> FAILURE)
> return;
>
> convert_to_array_ex(&zend_value);
>
> I realized after sifting through the Zend code that the proper handling of zval*
> types is just as important as that of the zval's. This modification will
> give extension programmers access to legitimate zval*'s and their corresponding
> zval.
If you want zval**'s, use zend_get_parameters().
-Andrei
"The most exciting phrase to hear in science, the one that heralds new
discoveries, is not "Eureka!" but "That's funny..." -- Isaac Asimov.