> Now you assume that you need to pass arguments to these functions.
> Which is not the case. Here's the proto:
>
> /* {{{ proto int rand([int min, int max])
> Returns a random number */
>
> So revert that patch. mt_rand() / rand() accecpt either 2 parameters or
> none at all. It was at least LESS wrong before. Now it's totally wrong.
_if_ I remember right (not sure), there are now 2 proto's, one with no args,
and one with2 args.
That would mean it is correct now, though I agree with you that that proto
is better.
I prefer your patch though, although it is IMO a hack. But the result was
correct.
Jeroen
>
> --Jani
>
> p.s. Egon, you still haven't explained me why my patch was wrong?
>
>
> On 12 Aug 2001 [EMAIL PROTECTED] wrote:
>
> >ID: 12550
> >Updated by: jeroen
> >Reported By: [EMAIL PROTECTED]
> >Old Status: Open
> >Status: Closed
> >Bug Type: Documentation problem
> >Operating System: N/A
> >PHP Version: 4.0.6
> >New Comment:
> >
> >Fixed. Hope this _is_ accepted...
> >
> >--Jeroen
> >
> >Previous Comments:
> >------------------------------------------------------------------------
> >
> >[2001-08-09 03:17:38] [EMAIL PROTECTED]
> >
> >Reopened. My workaround wasn't accepted either. :)
> >
> >--Jani
> >
> >
> >------------------------------------------------------------------------
> >
> >[2001-08-09 01:56:09] [EMAIL PROTECTED]
> >
> >I have committed yet another fix for this..
> >
> >--Jani
> >
> >
> >------------------------------------------------------------------------
> >
> >[2001-08-08 16:59:08] [EMAIL PROTECTED]
> >
> >Reopened, since DSSSL doesn't act as expected, and I'm not allowed to do
a workaround.
> >
> >------------------------------------------------------------------------
> >
> >[2001-08-06 17:09:31] [EMAIL PROTECTED]
> >
> >IT's changes in the CVS, and will be online tomorrow...
> >
> >------------------------------------------------------------------------
> >
> >[2001-08-03 21:09:30] [EMAIL PROTECTED]
> >
> >The documentation is wrong. The proto is this:
> >
> >mt_rand([int min, int max])
> >
> >--Jani
> >
> >
> >------------------------------------------------------------------------
> >
> >The remainder of the comments for this report are too long. To view
> >the rest of the comments, please view the bug report online at
> > http://bugs.php.net/?id=12550
> >
> >
> >Edit this bug report at http://bugs.php.net/?id=12550&edit=1
> >
>
--
PHP Development Mailing List <http://www.php.net/>
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
To contact the list administrators, e-mail: [EMAIL PROTECTED]
