On 2020-04-05 21:23, w...@wkhudgins.info wrote:
On 2020-03-26 04:19, Guilliam Xavier wrote:
On Thu, Mar 26, 2020 at 9:54 AM Nikita Popov <nikita....@gmail.com>
wrote:
On Thu, Mar 26, 2020 at 3:36 AM <w...@wkhudgins.info> wrote:
> Hi,
>
> Hope everyone is doing alright. I just raised a new RFC
> (https://wiki.php.net/rfc/add_str_starts_with_and_ends_with_functions ,
> github patch: https://github.com/php/php-src/pull/5300) for adding
> str_starts_with and str_ends_with to PHP. I would like to open this RFC
> up to discussion.
>
> I raised a similar RFC about 9 months ago
> (https://wiki.php.net/rfc/add_str_begin_and_end_functions) that was
> narrowly rejected. A major criticism of that RFC was the inclusion of
> case-insensitive versions of str_starts_with and str_ends_with. I have
> incorporated feedback from that experienced and narrowed the new RFC to
> only propose str_starts_with and str_ends_with.
>
I was in favor of the previous RFC, so also in favor of this one :)
These
are going to complement the recently added str_contains() nicely.
Notes on text:
> After that RFC was closed, a code freeze was in place for PHP8's release.
PHP8 has now been released and several individuals have requested the
str_starts_with and str_ends_with functionality again. This is a
simple but
highly desired functionality for PHP.
I think you mean PHP 7.4 here. PHP 8.0 is not yet released, and not
in
feature freeze either.
> Add str_starts_with, and str_starts_with_ci() functions
The second function should probably be str_ends_with() :)
Nikita
Hi Will,
First, thank you for re-working on this, I would love to see it
happen! :)
Nevertheless, apart from some typos (already reported), I still think
the RFC needs a "Motivation" section with actual examples of userland
implementations (using functions like
str[r]pos/strncmp/substr[_compare]... often with strlen, or even
preg_match/fnmatch... [with escaping]) and the downsides of each (e.g.
CPU-inefficient, memory-inefficient, error-prone, hard to
understand...) plus how they handle empty strings.
(And also probably shorten the introduction ^^ or move parts into a
new subsection)
PS: you could also add a link to the str_contains RFC
Best regards,
Hello,
I have revised the RFC,
https://wiki.php.net/rfc/add_str_starts_with_and_ends_with_functions ,
based on feedback from Nikita and Guilliam (thank you).
Please review the revised RFC and share any feedback during the coming
days.
Thank you,
Will
I will be moving this RFC to a vote on Monday April 20, 2020. I will
send out another email with the [VOTE] tag on that date when I update
the RFC to include the voting macro.
Thanks,
Will
--
PHP Internals - PHP Runtime Development Mailing List
To unsubscribe, visit: http://www.php.net/unsub.php
