-
Notifications
You must be signed in to change notification settings - Fork 576
[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
IMHO To the perlfaq chatper adecuate. |
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: