Skip to content

Conversation

jrfnl
Copy link
Contributor

@jrfnl jrfnl commented Oct 20, 2025

  1. The grapheme_levenshtein() function is new to PHP 8.5, so the $locale parameter being added does not "change" the function (in the context of changes between PHP 8.4 and 8.5).
  2. The grapheme_substr() function has also been given the new $locale parameter, but was not listed in the changelog entry.

Refs:

/cc @youkidearitai

1. The `grapheme_levenshtein()` function is new to PHP 8.5, so the `$locale` parameter being added does not "change" the function (in the context of changes between PHP 8.4 and 8.5).
2. The `grapheme_substr()` function has also been given the new `$locale` parameter, but was not listed in the changelog entry.

Refs:
* https://wiki.php.net/rfc/grapheme_levenshtein
* php@bdcea11
* https://wiki.php.net/rfc/grapheme_add_locale_for_case_insensitive
* php@ad75c26
Copy link
Member

@Ayesh Ayesh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

With the change @youkidearitai mentioned, LGTM 🙏

@TimWolla TimWolla merged commit cea52f7 into php:PHP-8.5 Oct 20, 2025
1 of 2 checks passed
TimWolla added a commit that referenced this pull request Oct 20, 2025
* PHP-8.5:
  PHP 8.5 | UPGRADING: fix entry about new grapheme $locale parameter (#20239)
  uri: Make uri_parser_rfc3986.h usable for external extensions (#20173)
  Fix missing deprecation message for default case statement followed by semicolon (#20172)
@jrfnl jrfnl deleted the feature/8.5-upgrading-fix-grapheme-locale-entry branch October 20, 2025 11:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants