Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:121432
Message-ID: <7a6178b9-a967-4b9f-b091-4057013838f9@gmail.com>
Date: Thu, 19 Oct 2023 15:33:26 +0200
MIME-Version: 1.0
User-Agent: Mozilla Thunderbird
To: internals@lists.php.net
References: <CAEPPVa2zCt674R=dp=hxvOmyxdumQJ+Axs97VXhygdGsuoy=vw@mail.gmail.com>
Content-Language: en-US
In-Reply-To: <CAEPPVa2zCt674R=dp=hxvOmyxdumQJ+Axs97VXhygdGsuoy=vw@mail.gmail.com>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 7bit
Subject: Re: [PHP-DEV] [RFC] [Discussion] Add multibyte trim function
 (mb_trim, mb_ltrim and mb_rtrim)
From: dossche.niels@gmail.com (Niels Dossche)

Hi Yuya

On 19/10/2023 13:57, youkidearitai wrote:
> Hi, internals.
> 
> 8ctopus san can't send email, so I'm writing new RFC for multibyte
> trim function.
> https://wiki.php.net/rfc/mb_trim
> https://github.com/php/php-src/pull/12459
> 
> I would like to under discussion.

Thanks for this.

I have a question about the .. support:
You state "Mapping with other character codes may be incompatible", for one of the reasons not to support this.
Can you please explain this a bit more?

Also, I notice at the top of your RFC it still says: "Status: Draft", I think you forgot to change this.
Similarly, the RFC is not listed under discussion on https://wiki.php.net/rfc.

> 
> Regards.
> Yuya
> 

Kind regards
Niels