Re: Re: cvs: php4 /ext/standard rand.c
| From: | rasmus at lerdorf dot on dot ca | Date: | Wed, 05 Sep 2001 20:41:44 +0000 |
| Subject: | Re: Re: cvs: php4 /ext/standard rand.c | ||
| References: | 1 | Groups: | php.dev |
| Request: | Send a blank email to php-dev+get-65178@lists.php.net to get a copy of this message | ||
> > On Wed, 05 Sep 2001, Sterling Hughes wrote:
> > > sterling Wed Sep 5 16:52:45 2001 EDT
> > >
> > > Modified files:
> > > /php4/ext/standard rand.c
> > > Log:
> > > a bit of api cleanup... move range stuff into a macro (properly :)
> >
> > I have to ask once again. Why not use zend_parse_parameters() while you
> > are cleaning up API anyway?
> >
> I dunno, I'm not used to it, brain no like new information!!!!!
>
> Isn't more of an extension thing though? I'm not sure, but I
> remember Zeev saying that it was slower, so should it be used in the
> core set of functions? If it is, I'll start use
> zend_parse_parameters() in future cleanups though...
It is slower in the case of functions that don't take any arguments. For
ones that do take arguments I don't see a performance issue. And the
error message it produces is more informative.
-Rasmus