Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:71605
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: error (pb1.pair.com: domain basereality.com from 209.85.220.177 cause and error)
MIME-Version: 1.0
In-Reply-To: <CA+kxMuQvHpzM2GbUxRC9u5EPV3ZZz9F1qtDPcVRx3ZyAB22RUw@mail.gmail.com>
References: <CAGa2bXb7iVGbqpQh8TkDDyr-FmCWTFg1rcXj23ScErhNmikbNw@mail.gmail.com>
	<CA+kxMuTBjp=OhN0sHDrciJWKzre1mDKt_BSTja4UJ7_-JGnkGg@mail.gmail.com>
	<CAGa2bXb3UYFnUpFJBtNh=j9nP96ge7pXF-_AhG+tWwOPRSHS6A@mail.gmail.com>
	<CA+kxMuQvHpzM2GbUxRC9u5EPV3ZZz9F1qtDPcVRx3ZyAB22RUw@mail.gmail.com>
Date: Sun, 26 Jan 2014 22:28:01 +0000
Message-ID: <CA+kxMuQ25NhV9_Pyy+sBxakPDkX40FVzwAxGcmc0R7-MjDS-7w@mail.gmail.com>
To: Yasuo Ohgaki <yohgaki@ohgaki.net>
Cc: "internals@lists.php.net" <internals@lists.php.net>
Content-Type: text/plain; charset=windows-1252
Content-Transfer-Encoding: quoted-printable
Subject: Re: [PHP-DEV] [VOTE] RFC: Multibyte Char Handling
From: danack@basereality.com (Dan Ackroyd)

Sorry, for rapid posting but I realise I should have been explicit in
my last message. The RFC being voted on has "Backward Incompatible
Changes: None."

The referenced RFC which apparently would be included has:

"Removed (deprecated) functions and reasons behind it

mb_check_encoding() =96 Not that usable as it is advertised, period.
First of all, validation in terms of encoding is just as same as
filtering through the converter supplied with the same value for the
input and output encoding. Thus just use mb_convert_encoding().
mb_convert_case() =96 Use mb_strtoupper(), mb_strtolower() and mb_strtotitl=
e()
mb_convert_kana() =96 This can't be standard-compliant. In addition,
part of the functionality is already covered by Normalizer of intl
extension, so we need to carefully consider what is actually needed
here again.
mb_convert_variables() =96 This can be implemented as a script.
mb_decode_mimeheader() and mb_encode_mimeheader() =96 Non-standard complian=
cy.
mb_decode_numericentity() =96 Removed in favor of html_entity_decode().
mb_encode_numericentity() =96 Removed in favor of htmlentities() and
htmlspecialchars().
mb_encoding_aliases() =96 Just unnecessary.
mb_ereg_match() =96 Use mb_ereg()
mb_ereg_search(), mb_ereg_search_getpos(), mb_ereg_search_getregs(),
mb_ereg_search_init(), mb_ereg_search_pos(), mb_ereg_search_regs() and
mb_ereg_search_setpos() =96 I rarely heard a script that actively uses
these functions. They involve an internal state that is not visible to
users, and thus it most likely causes confusion when used across the
function calls. Need to be reimplemented as a class.
mb_eregi() =96 Use mb_regex_options() and mb_ereg()
mb_eregi_replace() =96 I wonder why this function was added in the first
place because giving 'i' option to mb_ereg_replace() works in the same
way.
mb_detect_order(), mb_get_info(), mb_http_input(), mb_http_output(),
mb_language() and mb_substitute_character() =96 ini_set() and ini_get()
are your friends, I guess=85
mb_regex_encoding() =96 It is really confusing that the current mbstring
allows two different encoding defaults for regex functions and the
rest. Those settings are unified in the alternative version and so
this is no longer necessary.
mb_send_mail() =96 The behavior of this function relies on the
pseudo-locale setting called =93mbstring.language=94 that supports just a
limited set of possible locales. As not everyone can benefit from the
function and most significant applications implement their own mail
functions, I suppose this is no longer wanted.
mb_strrchr() =96 Use mb_strrpos().
mb_strrichr() =96 Use mb_strripos()."

None is not the same as a huge number of function changes.

cheers
Dan
Ack




On Sun, Jan 26, 2014 at 10:23 PM, Dan Ackroyd <danack@basereality.com> wrot=
e:
>> Compile mbstringp-ng as default compiled module, when mbstring-ng is rea=
dy.
>
> How can people know what they're voting on when the module is
> apparently not ready for use?
>
> The referenced RFC  (https://wiki.php.net/rfc/altmbstring) is at least
> slightly ambigous, and which parts of it are going to be included in
> the 'Multibyte Char Handling' vote ?
>
> cheers
> Dan
>
> On Sun, Jan 26, 2014 at 10:10 PM, Yasuo Ohgaki <yohgaki@ohgaki.net> wrote=
:
>> Hi Dan,
>>
>> On Mon, Jan 27, 2014 at 6:59 AM, Dan Ackroyd <danack@basereality.com> wr=
ote:
>>>
>>> This vote is not listed as 'In voting phase' on the RFC home page -
>>> https://wiki.php.net/rfc
>>
>>
>> Thank you.  Updated
>>
>>> Also, as Nikic mentioned, this RFC has changed from the original
>>> request to merging a different library into PHP, 3 days ago. Voting on
>>> such a change in such a short time period seems wrong.
>>
>>
>> I forgot a line for mbstring-ng implementation.
>> I've fixed voting date and extended 4 days.
>>
>> Thank you.
>>
>> --
>> Yasuo Ohgaki
>> yohgaki@ohgaki.net