[RFC] Deprecate and remove INTL_IDNA_VARIANT_2003

php.internals

Christoph Becker

9 years ago
Hi! Following up on <http://news.php.net/php.internals/97164>, I have written the RFC <https://wiki.php.net/rfc/deprecate-and-remove-intl_idna_variant_2003> and hereby placing it under discussion. I appreciate reviews and comments!
-- Christoph M. Becker

Joe Watkins

9 years ago
Evening Christoph, +1, and good catch noticing the upstream change. Cheers Joe On Sat, Jan 7, 2017 at 2:06 PM, Christoph M. Becker <cmbecker69@gmx.de> wrote:

Sara Golemon

9 years ago
On Sat, Jan 7, 2017 at 6:06 AM, Christoph M. Becker <cmbecker69@gmx.de> wrote:
> Following up on <http://news.php.net/php.internals/97164>, I have > written the RFC > > <https://wiki.php.net/rfc/deprecate-and-remove-intl_idna_variant_2003> > > and hereby placing it under discussion. > > I appreciate reviews and comments! >
I noticed that recently as well (combing through compiler warnings) but ended up deciding "If it ain't broke, don't fix it." A deprecation is certainly in order, though I'd say we shouldn't change the default in 7.2. I'd be fine deprecating in 7.2, changing default in 7.4, then removing in 8.0 It's profoundly non-broken atm, so there's no need to rush. -Sara

Christoph Becker

9 years ago
On 08.01.2017 at 01:38, Sara Golemon wrote:
> On Sat, Jan 7, 2017 at 6:06 AM, Christoph M. Becker <cmbecker69@gmx.de> wrote: >> Following up on <http://news.php.net/php.internals/97164>, I have >> written the RFC >> >> <https://wiki.php.net/rfc/deprecate-and-remove-intl_idna_variant_2003> >> >> and hereby placing it under discussion. >> >> I appreciate reviews and comments! >> > I noticed that recently as well (combing through compiler warnings)
There's also a respective ticket in the tracker: <https://bugs.php.net/bug.php?id=73874>.
> but ended up deciding "If it ain't broke, don't fix it." > > A deprecation is certainly in order, though I'd say we shouldn't > change the default in 7.2. I'd be fine deprecating in 7.2, changing > default in 7.4, then removing in 8.0
Thanks, that's fine for me. I'm going to change the RFC accordingly. Cheers, Christoph