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
Decoding problem with s///i in perl 5.14 #11707
Comments
From am@jobindex.dkHello. I have noticed this unexpected behaviour in perl 5.14.2: perl -e 'use 5.012; my $x="br\x{f8}ndby"; $x =~ s/b/X/gi; say $x;' outputs "Xrøndby" (in a latin1 terminal) -- that is, the last "b" is not replaced with an "X". I have found that it works as expected in 5.12.2 as well as the blead branch. I do not know when the problem appeared, but I bisected the fix to commit 0a185c4. The commit message seems to consider this an optimization rather than a bugfix. So I wanted to make you aware of this, and suggest inclusion on the 5.14 branch. Thanks, |
From @nwc10On Wed, Oct 19, 2011 at 07:37:32AM -0700, Anders Melchiorsen wrote:
I ran ../perl/Porting/bisect.pl --start v5.12.0 --end v5.14.0 -e 'use 5.012; my $x="br\x{f8}ndby"; $x =~ s/b/X/gi; say $x; exit($x =~ tr/X// == 1)' it reports: HEAD is now at a33c29b foldEQ_utf8_flags: Add no-mixing ASCII option regexec.c: refactor find-by-class EXACTish code This code is way out-of-date, using upper and lower case instead of fold-case. :100644 100644 e5e6e276f66621c881924ce7bc298011244831cf a7f55267c7dee49163360ecb00ce715ca423fbdb M regexec.c I checked - that commit commit in v5.13.9 introduced the problem. Neither commit adds or changes any tests. Should there be tests? Nicholas Clark |
The RT System itself - Status changed from 'new' to 'open' |
From @khwilliamsonOn 10/20/2011 06:28 AM, Nicholas Clark wrote:
I did not add tests in either case because I did not think that either |
From @khwilliamsonNow resolved with commit bbdd8ba |
From [Unknown Contact. See original ticket]Now resolved with commit bbdd8ba |
@khwilliamson - Status changed from 'open' to 'resolved' |
Migrated from rt.perl.org#101710 (status was 'resolved')
Searchable as RT101710$
The text was updated successfully, but these errors were encountered: