Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:102460
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 209.85.214.44 as permitted sender)
MIME-Version: 1.0
In-Reply-To: <CAJW__o1aYSJQewBuPPQcRSMj_eGiM1ff9vWQ9Nsii1kE6NgcAQ@mail.gmail.com>
References: <CAF+90c_+GZNrV1SC9q=Y3C=L3LMnoXX7J3zK6=kS9-sMXHZZow@mail.gmail.com>
 <CAJW__o1aYSJQewBuPPQcRSMj_eGiM1ff9vWQ9Nsii1kE6NgcAQ@mail.gmail.com>
Date: Tue, 26 Jun 2018 20:16:29 +0200
Message-ID: <CAF+90c_O-hZP+-C34-_x4wQTUv98ZuZ-iiuJKoDhZvoY4Trt6A@mail.gmail.com>
To: Kalle Sommer Nielsen <kalle@php.net>
Cc: Internals <internals@lists.php.net>
Content-Type: multipart/alternative; boundary="0000000000004b7ab0056f8f7fb7"
Subject: Re: [PHP-DEV] [RFC] Deprecations for PHP 7.3
From: nikita.ppv@gmail.com (Nikita Popov)

--0000000000004b7ab0056f8f7fb7
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

On Tue, Jun 26, 2018 at 6:31 PM, Kalle Sommer Nielsen <kalle@php.net> wrote=
:

> Hi
>
> Den s=C3=B8n. 24. jun. 2018 kl. 18.47 skrev Nikita Popov <nikita.ppv@gmai=
l.com
> >:
> > If you have a minor deprecation in mind, but were too lazy to write an
> RFC
> > for it, please write me a mail until tomorrow, so that it might be
> included
> > as part of this proposal. As time is limited I don't want to include
> > anything larger or controversial in this RFC though.
>
> As suggested in the past, I would like to add the following to this
> list (if its not too late):
>
>  - The (real) type-cast and its function, is_real(). There doesn't
> seem to be any support for reals in settype() anyway (side note: in
> the implementation of settype() it claims "double" is deprecated)
>  - Function variants that already exists as constants, php_sapi_name()
> > PHP_SAPI, pi() > M_PI, phpversion() > PHP_VERSION etc
>  - enable_dl ini directive, as dl() is only operational for CLI and Embed
> anyway
>  - pdo_odbc.db2_instance_name ini directive, its been marked as
> deprecated and mentioned in the manual since PHP 5.1.11 that it will
> *certainly* be removed in the future
>

Feel free to add this one to the RFC, if you know what the background for
the deprecation is. Given that the manual already marks it as deprecated
for so long, this seems like a mere technicality.


>  - The 'b' constant string prefix, its not used and was meant as to
> make code ready for PHP6, should the time come where we want to add a
> feature that uses this, we can always re-add it
>
> Other things thats been suggested by others in the past:
>
>  - Second parameter of spl_autoload() and its associated function
> spl_autoload_extensions()
>  - hebrevc() -- same as hebrev() + nl2br(), perhaps even deprecate
> hebrev() too (see next one)
>  - convert_cyr_string() -- Point to mb_convert_encoding() / iconv
>  - Remove get_magic_quotes_gpc(), not been working for what, 5+ years now=
?
>  - allow_url_include ini option, this can lead to all kinds of
> security complications anyway
>  - The alternative string interpolation syntaxes (${varName},
> ${varName['offset']}, ${expr}) and make them more consistent
> ({$varName}, {$varName['offset']}, {${expr}})
>  - The historial parameter handling that works both ways for
> implode(), should be unified to match that of explode()
>
> I could probably think of more, but thats all I got for now. I can and
> will of course help produce the relevant patches should it be needed
> for most of these things.
>

Given the recent discussions about having a PHP 7.4 release for additional
deprecations, I'm going to punt on the rest. While they may be no-brainers
to you and I, a lot of those are actually quite controversial (e.g. there
was an RFC to deprecate b' in 7.2 and it was declined -- go figure!)

I'd suggest that you already open up the next deprecation RFC, to make sure
these are not forgotten. Some of them should probably get separate
proposals altogether (especially the var interpolation syntax).

Nikita

--0000000000004b7ab0056f8f7fb7--