-
Notifications
You must be signed in to change notification settings - Fork 580
[doc] Nits with builtin::trim #23212
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Labels
Comments
Grinnz
added a commit
to Grinnz/perl5
that referenced
this issue
Apr 22, 2025
Grinnz
added a commit
to Grinnz/perl5
that referenced
this issue
Apr 22, 2025
Grinnz
added a commit
to Grinnz/perl5
that referenced
this issue
Apr 22, 2025
IMHO To the perlfaq chatper adecuate. |
Grinnz
added a commit
to Grinnz/perl5
that referenced
this issue
May 4, 2025
khwilliamson
pushed a commit
that referenced
this issue
May 10, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Where
builtin::trim
Description
Don't suggest String::Util. it uses suboptimal implementations throughout, If you need to suggest an alternative, builtin::compat is a good candidate, name included. It also has better backwards compatibliity: 5.8 vs. String::Util's 5.14.
Don't propagate the slower combined regex. Instead of:
use:
s/\A\s+//u, s/\s+\z//u for my $copy = $str;
The text was updated successfully, but these errors were encountered: