RE: [PHP-CVS] cvs: php4 /ext/standard basic_functions.c
| From: | Sean R. Bright | Date: | Sun, 30 Dec 2001 16:39:01 +0000 |
| Subject: | RE: [PHP-CVS] cvs: php4 /ext/standard basic_functions.c | ||
| References: | 1 | Groups: | php.cvs |
| Request: | Send a blank email to php-cvs+get-8866@lists.php.net to get a copy of this message | ||
No. zend_parse_parameters() takes care of that.
> -----Original Message-----
> From: Jon Parise [mailto:jon@php.net]
> Sent: Sunday, December 30, 2001 11:15 AM
> To: Zak Greant
> Cc: php-cvs@lists.php.net
> Subject: Re: [PHP-CVS] cvs: php4 /ext/standard basic_functions.c
>
>
> On Sun, Dec 30, 2001 at 09:36:30AM -0000, Zak Greant wrote:
>
> > - if (ZEND_NUM_ARGS() != 1 || zend_get_parameters_ex(1,
> &str) == FAILURE) {
> > - WRONG_PARAM_COUNT;
> > - }
> > - convert_to_string_ex(str);
> > -
> > - if (Z_TYPE_PP(str) != IS_STRING) {
> > + if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC,
> "s", &str, &str_len) == FAILURE) {
> > RETURN_FALSE;
> > }
>
> Shouldn't this still return WRONG_PARAM_COUNT on failure?
>
> --
> Jon Parise (jon@csh.rit.edu) . Information Technology (2001)
> http://www.csh.rit.edu/~jon/ : Computer Science House Member
>
> --
> PHP CVS Mailing List (http://www.php.net/)
> To unsubscribe, e-mail: php-cvs-unsubscribe@lists.php.net
> For additional commands, e-mail: php-cvs-help@lists.php.net
> To contact the list administrators, e-mail:
> php-list-admin@lists.php.net
>