Leigh, I would _very_ much prefer to keep this BC. If we do the right thing, and there are no repercussions, I'd actually remove the warning. If you think we might want to disallow this in the future, use an E_DEPRECATED, but a warning seems pointless unless it's doing the wrong thing, or it's going to start doing the wrong thing at some point.
- Davey On Mon, Aug 8, 2016 at 2:36 PM, Leigh <lei...@gmail.com> wrote: > Hi all, > > There has been an unforeseen break with rand() when the minimum value is > greater than the maximum. > > Prior to 7.1 rand() would happily accept backwards parameters and return a > value, however in the 7.1 branch it now emits a warning and returns false. > > I've preemptively committed a fix to allow min > max and return a value as > in previous versions, but have kept the warning. > > Looking for some feedback/opinions on whether anyone else thinks this > should be fixed differently (or not at all). > > N.B. this also changes the behaviour of mt_rand to now accept min > max >
