Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:115953
MIME-Version: 1.0
References: <CABTDVhBRpD=nWBk2xGLEmdcWwOzCeqGeqH5vLSugPFOeEN9Mog@mail.gmail.com>
 <CAF+90c-kLHe-M4jcMRa=_8Lhakcyrtam109WKga6XLQqmJYp0Q@mail.gmail.com>
 <CABTDVhBYB1-TkxTNPCkedE0k0UBWv0eUNfmSr+KDWkpgcw12JA@mail.gmail.com> <CA+kxMuR21VF3Ax-N7ensrNAhs1O9kVQn=BuEBqoMV4FmDHpF0g@mail.gmail.com>
In-Reply-To: <CA+kxMuR21VF3Ax-N7ensrNAhs1O9kVQn=BuEBqoMV4FmDHpF0g@mail.gmail.com>
Date: Sun, 5 Sep 2021 13:06:28 +0900
Message-ID: <CABTDVhA_zAfRPx+9AtFUzVWySOaU6kG0H1Dxhsg2Gn3M0fTkQw@mail.gmail.com>
To: Dan Ackroyd <Danack@basereality.com>, PHP internals <internals@lists.php.net>
Content-Type: multipart/alternative; boundary="000000000000cc9db405cb37a82d"
Subject: Re: [PHP-DEV] [RFC] Random Extension 3.0
From: zeriyoshi@gmail.com (Go Kudo)

--000000000000cc9db405cb37a82d
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

This may not have been the best way to put it. The point I was trying to
make was that while BC Breaks do occur, they are very easy to solve.

The impact on downstream projects will be significant, and there may be
many extensions affected, since this change concerns a frequently used RNG
feature. However, in other words, it may need to be sorted out as soon as
possible.

However, I also understand the theory that this should be done in
conjunction with a major language change. Perhaps I should have suggested
this for PHP 8.0.

Either way, I'll try to separate these suggestions if necessary, but if we
were to try to provide an OOP API without BC Break, what do you think would
be the ideal form?

Regards,
Go Kudo

2021=E5=B9=B49=E6=9C=885=E6=97=A5(=E6=97=A5) 4:24 Dan Ackroyd <Danack@baser=
eality.com>:

> On Fri, 3 Sept 2021 at 18:41, Go Kudo <zeriyoshi@gmail.com> wrote:
> >
> > Nikita wrote:
> >> this one also moves all of the existing RNG-related functionality
> >> from ext/standard to ext/random.
> >
> > There are several reasons for this.
> >
> > - The `random` extension name is already used by ext/standard and may
> > interfere with the build system.
> > - Due to the namespace rules for new extensions, it is inappropriate to
> > rename an extension.
> > - Due to history, the rand / mt_rand dependency is tricky and I thought
> it
> > was time to sort it out.
> > - I don't think it's a good idea to have multiple header files for a
> single
> > domain feature.
>
> Have you considered how much work that is going to incur on downstream
> projects?
>
> If there's a good reason for moving stuff around then it can be
> appropriate, but the RFC phrasing of:  "At the same time, the
> following internal APIs will also be relocated. If you want to use
> them, you can _simply_ include ext/random/random.h." sounds pretty
> dismissive of causing possibly unneeded work for downstream projects.
>
> cheers
> Dan
> Ack
>

--000000000000cc9db405cb37a82d--