-
Notifications
You must be signed in to change notification settings - Fork 529
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
perldiag: incorrect list of contexts where unescaped left braces are not deprecated #17185
Comments
From @ntyniThis is a bug report for perl from ntyni@debian.org, As reported by Jakub Wilk in https://bugs.debian.org/942037 : The perldiag manpage, in the section "Unescaped left brace in regex is The contexts where no warnings or errors are raised are: But /foo({bar)/ does trigger the warning: $ perl -pe '/foo({bar)/' < /dev/null Flags: Site configuration information for perl 5.30.0: Configured by Debian at Tue Oct 8 17:25:35 UTC 2019. Summary of my perl5 (revision 5 version 30 subversion 0) configuration: Locally applied patches: @INC for perl 5.30.0: Environment for perl 5.30.0: |
@ntyni we take pull requests now! Dow you have a patch already? |
@khwilliamson Replied on the mailing list: This is a temporary problem, to be fixed as a side effect of actually making whatever is fatal, fatal. |
On Tue, Oct 22, 2019 at 12:20:26PM -0700, Todd Rinaldo wrote:
@khwilliamson Replied on the mailing list:
This is a temporary problem, to be fixed as a side effect of actually making whatever is fatal, fatal.
So is /foo({bar)/ going to be fatal in 5.32 (as the warning says)
or not (as the documentation says) ?
…--
Niko
|
Not fatal |
This warning is simply deleted. The possible places where an unescaped left brace is illegal has been scaled back to avoid breaking more existing code, and this context will remain legal.
This commit removes this message: be45a1e |
Migrated from rt.perl.org#134498 (status was 'new')
Searchable as RT134498$
The text was updated successfully, but these errors were encountered: