Re: [Discussion] grapheme cluster for str_split function
| From: | youkidearitai | Date: | Tue, 05 Mar 2024 07:25:43 +0000 |
| Subject: | Re: [Discussion] grapheme cluster for str_split function | ||
| References: | 1 2 | Groups: | php.internals |
| Request: | Send a blank email to internals+get-122561@lists.php.net to get a copy of this message | ||
2024年3月5日(火) 5:52 Niels Dossche <dossche.niels@gmail.com>:
>
> Hi Yuya
>
> This sounds useful.
>
> I do have a question about the function signature:
> function grapheme_str_split(string $string, int $length = 1): array {}
>
> This always returns an array.
> However, looking at your PR it seems you return NULL on failure, but the return type in the
> signature isn't nullable.
> Also, from a quick look, it seems other functions return false instead of null on failure. So
> perhaps the return type should be array|false.
>
> What do you think? :)
>
> Kind regards
> Niels
>
> On 03/03/2024 00:21, youkidearitai wrote:
> > Hi, Internals
> >
> > I noticed PHP does not have grapheme cluster for str_split function.,
> > Until now, you had to use the PCRE function's \X.
> >
> > Therefore, I try create
grapheme_str_split function.
> > https://github.com/php/php-src/pull/13580
> > It is possible to convert array per emoji and variation selectors using ICU.
> >
> > If it's fine, I'll create an RFC.
> >
> > Regards
> > Yuya
> >
Hi, Niels
Thank you for your comment.
Indeed, returns false is make sense.
Therefore, I changed to returns false when invalid UTF-8 strings.
Regards
Yuya
--
---------------------------
Yuya Hamada (tekimen)
- https://tekitoh-memdhoi.info
- https://github.com/youkidearitai
-----------------------------