Re: [PATCH] Add 'Z' flag to zend_parse_parameters*()
| From: | Jon Parise | Date: | Wed, 09 Jul 2003 03:15:02 +0000 |
| Subject: | Re: [PATCH] Add 'Z' flag to zend_parse_parameters*() | ||
| References: | 1 2 3 4 5 6 7 8 9 | Groups: | php.internals |
| Request: | Send a blank email to internals+get-3321@lists.php.net to get a copy of this message | ||
On Tue, Jul 08, 2003 at 07:05:18PM -0500, fuhs@purdue.edu wrote:
> One more time. I hope it's not too obvious that I'm new at this :)
>
> This patch would add a 'Z' flag to zend_parse_parameters*().
>
> Example:
> #typdef zvar zval**
> zvar a_zvar;
> zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "Z", &zvar);
^^^^
That should be &a_var, right?
> @@ -471,7 +480,7 @@ static int zend_parse_va_args(int num_ar
> case 's': case 'b':
> case 'r': case 'a':
> case 'o': case 'O':
> - case 'z':
> + case 'z': case 'Z':
> max_num_args++;
> break;
It looks like you have an extra tab in front of that new 'case'.
--
Jon Parise (jon@php.net) :: The PHP Project (http://www.php.net/)