On 09.07.2016 at 11:18, Leigh wrote:
> On Sat, 9 Jul 2016 at 10:16 Christoph Becker wrote:
>
>> Assuming that "Make array_rand() more efficient" is indeed only about
>> performance (and not about fixes for the broken implementation), it
>> wouldn't be a bug fix, but as array_rand()'s behavior wo
On 09.07.2016 at 10:49, Pierre Joye wrote:
> On Jul 9, 2016 3:19 PM, "Leigh" wrote:
>>
>> On Sat, 9 Jul 2016 at 08:48 Pierre Joye wrote:
>>>
>>> So, I voted no then as it is clear that you do not see a problem to
>>> break codes without a single warning or time to adapt before.
>>>
>>> The other
On Sat, 9 Jul 2016 at 10:16 Christoph Becker wrote:
>
> Assuming that "Make array_rand() more efficient" is indeed only about
> performance (and not about fixes for the broken implementation), it
> wouldn't be a bug fix, but as array_rand()'s behavior would change
> anyway as noted by the RFC, it
On 09.07.2016 at 10:18, Leigh wrote:
> On Sat, 9 Jul 2016 at 08:48 Pierre Joye wrote:
>
>> So, I voted no then as it is clear that you do not see a problem to
>> break codes without a single warning or time to adapt before.
>>
>> The other sections are fine and voted yes.
>
> For the part where
On Jul 9, 2016 4:05 PM, "Leigh" wrote:
>
> On Sat, 9 Jul 2016 at 09:49 Pierre Joye wrote:
>>
>>
>> I am sorry but this PR possibly breaks BC and cases have been clearly
explained how and why. Asking to show production code publically is not
something that you should request.
>>
>> I can write a
On Sat, 9 Jul 2016 at 09:49 Pierre Joye wrote:
>
> I am sorry but this PR possibly breaks BC and cases have been clearly
> explained how and why. Asking to show production code publically is not
> something that you should request.
>
> I can write a sample app to show you how but given the expla
On Jul 9, 2016 3:49 PM, "Pierre Joye" wrote:
>
>
> On Jul 9, 2016 3:19 PM, "Leigh" wrote:
> >
> >
> > On Sat, 9 Jul 2016 at 08:48 Pierre Joye wrote:
> >>
> >> So, I voted no then as it is clear that you do not see a problem to
> >> break codes without a single warning or time to adapt before.
>
On Jul 9, 2016 3:19 PM, "Leigh" wrote:
>
>
> On Sat, 9 Jul 2016 at 08:48 Pierre Joye wrote:
>>
>> So, I voted no then as it is clear that you do not see a problem to
>> break codes without a single warning or time to adapt before.
>>
>> The other sections are fine and voted yes.
>
>
> For the par
On Sat, 9 Jul 2016 at 08:48 Pierre Joye wrote:
> So, I voted no then as it is clear that you do not see a problem to
> break codes without a single warning or time to adapt before.
>
> The other sections are fine and voted yes.
>
For the part where you voted no. Still nobody has presented (publi
On Thu, Jul 7, 2016 at 7:56 PM, Pierre Joye wrote:
>
> On Jul 7, 2016 7:53 PM, "Leigh" wrote:
>>
>> On 7 July 2016 at 13:39, Pierre Joye wrote:
>> > Hi
>> >
>> > Looks good but missing an option to choose the default mode.
>> >
>> > I would choose BC as default at least for one release (7.1).
>
On Jul 7, 2016 7:53 PM, "Leigh" wrote:
>
> On 7 July 2016 at 13:39, Pierre Joye wrote:
> > Hi
> >
> > Looks good but missing an option to choose the default mode.
> >
> > I would choose BC as default at least for one release (7.1).
>
> It's been that way since 5.2.1, I think it's had enough rele
On 7 July 2016 at 13:39, Pierre Joye wrote:
> Hi
>
> Looks good but missing an option to choose the default mode.
>
> I would choose BC as default at least for one release (7.1).
It's been that way since 5.2.1, I think it's had enough releases already :)
--
PHP Internals - PHP Runtime Developm
Hi
Looks good but missing an option to choose the default mode.
I would choose BC as default at least for one release (7.1).
I tend to vote against fixing mt_rand because of that.
On Jul 7, 2016 5:39 PM, "Leigh" wrote:
> As the discussion thread has been quiet for a while, moving this RFC to
As the discussion thread has been quiet for a while, moving this RFC to voting.
https://wiki.php.net/rfc/rng_fixes
https://github.com/php/php-src/pull/1986
--
PHP Internals - PHP Runtime Development Mailing List
To unsubscribe, visit: http://www.php.net/unsub.php
14 matches
Mail list logo
