Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:129795
Precedence: list
MIME-Version: 1.0
References: <CAEPPVa3Do84JJTE5WwoiXSqvgTpPkjfdSwqRCDaGD4OydsZjHA@mail.gmail.com>
 <f440b955-a389-4919-bfe9-9f22addf75c3@bastelstu.be>
In-Reply-To: <f440b955-a389-4919-bfe9-9f22addf75c3@bastelstu.be>
Date: Tue, 20 Jan 2026 17:14:05 +0900
Message-ID: <CAEPPVa0QS3ay0CqfWwHpi=8mrp69L4ZcWBRoyR4BmWUARUD9Jw@mail.gmail.com>
Subject: Re: [PHP-DEV][UNDER DISCUSSION][RFC] grapheme_strrev function
To: =?UTF-8?Q?Tim_D=C3=BCsterhus?= <tim@bastelstu.be>
Cc: php internals <internals@lists.php.net>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
From: youkidearitai@gmail.com (youkidearitai)

2026=E5=B9=B41=E6=9C=8820=E6=97=A5(=E7=81=AB) 3:34 Tim D=C3=BCsterhus <tim@=
bastelstu.be>:
>
> Hi
>
> On 1/16/26 08:20, youkidearitai wrote:
> > I creating a new function for strrev for grapheme cluster unit.
> > grapheme_strrev function.
> >
> > My investigate, Sometime found mb_strrev function for userland.
> > However, I think create a grapheme cluster unit for strrev function.
> > Because multi code point in human language and emoji.
> >
> > I created an RFC and Pull Request.
> > Feel free to comment.
> > https://wiki.php.net/rfc/grapheme_strrev
> > https://github.com/php/php-src/pull/20949
>
> Thank you for your RFC. Aligning the feature set of the grapheme
> functions with those of mbstring and the bytestring-based functions
> definitely makes sense to me. I am thus in favor of the RFC.
>
> For the RFC document itself:
>
> You did not fill in the "RFC Impact" section. I think it can just be
> "None" for each of them. The only impact is the conflict with a possibly
> existing function. You already mentioned that in the breaking changes
> section. Adding new functions doesn't have any relevant impact on tools
> or IDEs, this is something that regularly happens.
>
> For Future Scope it can probably also be "None"? Even if you plan to add
> more grapheme functions, they are independent of grapheme_strrev.
>
> In the References section, please add a link to the mailing list
> archives of the discussion. This makes it easy for folks to find the
> discussion in the future. The correct link is:
> https://news-web.php.net/php.internals/129765
>
> Best regards
> Tim D=C3=BCsterhus

Hi, Tim

Thank you for your feedback!
I'm glad your agree.
And apply from your feedback.


Regards
Yuya


--=20
---------------------------
Yuya Hamada (tekimen)
- https://tekitoh-memdhoi.info
- https://github.com/youkidearitai
-----------------------------