On Sun, Aug 12, 2018 at 7:00 PM, Christoph M. Becker <cmbecke...@gmx.de> wrote:
> Hi! > > Quite a while ago we have accepted the “Use default_charset As Default > Character Encoding” RFC[1], which also contains the clause “Old > iconv.*/mbstring.* php.ini parameters will be removed for master PHP6” > (obviously that was written before we decided to skip PHP 6, and as such > would have to be applied to PHP 7). I'm quite happy, though, that this > hasn't happened yet, since it seems that the encoding names may be > different between the extensions. At least the php.net manual states[2]: > > | Some systems (like IBM AIX) use "ISO8859-1" instead of "ISO-8859-1" so > | this value has to be used in configuration options and function > | parameters. > > It appears to be worthwhile to review the deprecation of the respective > iconv.* ini options. Not quite sure about the mbstring.* ini options. > > Also, if we stick with the deprecation of the iconv.* ini settings, we > also should deprecate iconv_set_encoding(), which currently changes the > deprecated iconv.* ini options, and as such throws a deprecation > warning. However, if the ini options would be removed, what should the > function do? Nothing? Change the php.*_encoding settings. In my > opinion, both would be rather confusing. iconv_get_encoding() has a > similar, albeit minor, issue. > > [1] <https://wiki.php.net/rfc/default_encoding> > [2] <http://php.net/manual/en/iconv.configuration.php> > IIRC the new ini settings also don't work with Zend Multibyte yet. That would need to be fixed before we drop them. Nikita