Re: Bug #12550 Updated: rand() and mt_rand() don't behave as documented

> 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 jeroen@php.net wrote: > > >ID: 12550 > >Updated by: jeroen > >Reported By: sulka@sulake.com > >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] sniper@php.net > > > >Reopened. My workaround wasn't accepted either. :) > > > >--Jani > > > > > >------------------------------------------------------------------------ > > > >[2001-08-09 01:56:09] sniper@php.net > > > >I have committed yet another fix for this.. > > > >--Jani > > > > > >------------------------------------------------------------------------ > > > >[2001-08-08 16:59:08] tom@php.net > > > >Reopened, since DSSSL doesn't act as expected, and I'm not allowed to do a workaround. > > > >------------------------------------------------------------------------ > > > >[2001-08-06 17:09:31] tom@php.net > > > >IT's changes in the CVS, and will be online tomorrow... > > > >------------------------------------------------------------------------ > > > >[2001-08-03 21:09:30] sniper@php.net > > > >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 > > >