Skip to content
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

exclude in regexp character class fails when matched agains empty string #2416

Closed
p6rt opened this issue Apr 28, 2011 · 5 comments
Closed

exclude in regexp character class fails when matched agains empty string #2416

p6rt opened this issue Apr 28, 2011 · 5 comments

Comments

@p6rt
Copy link

p6rt commented Apr 28, 2011

Migrated from rt.perl.org#89470 (status was 'resolved')

Searchable as RT89470$

@p6rt
Copy link
Author

p6rt commented Apr 28, 2011

From @bbkr

[15​:34] <bbkr_> rakudo​: say so "" ~~ m/<[a..z]-[x]>/
[15​:34] <p6eval> rakudo 8533c3​: OUTPUT«Cannot take substr outside stringâ�¤ in
<anon> at line 22​:/tmp/zpqtyrD_Awâ�¤ in 'Cool​::match' at line
2661​:CORE.settingâ�¤ in main program body at line 22​:/tmp/zpqtyrD_Awâ�¤Â»

[15​:35] <moritz> bbkr_​: similar cases are known and have been fixed - I don't
think this one
[15​:35] * bbkr_ reports just in case

@p6rt
Copy link
Author

p6rt commented Nov 20, 2011

From @jnthn

On Thu Apr 28 06​:37​:28 2011, pawel.pabian@​implix.com wrote​:

[15​:34] <bbkr_> rakudo​: say so "" ~~ m/<[a..z]-[x]>/
[15​:34] <p6eval> rakudo 8533c3​: OUTPUT«Cannot take substr outside
string� in
<anon> at line 22​:/tmp/zpqtyrD_Awâ�¤ in 'Cool​::match' at line
2661​:CORE.settingâ�¤ in main program body at line 22​:/tmp/zpqtyrD_Awâ�¤Â»

Works now​:

say so "" ~~ m/<[a..z]-[x]>/
Bool​::False
say so "x" ~~ m/<[a..z]-[x]>/
Bool​::False
say so "z" ~~ m/<[a..z]-[x]>/
Bool​::True

Marked as testneeded.

/jnthn

@p6rt
Copy link
Author

p6rt commented Nov 20, 2011

The RT System itself - Status changed from 'new' to 'open'

@p6rt
Copy link
Author

p6rt commented Jan 30, 2012

From @moritz

Now tested in S05-metasyntax/charset.t

@p6rt
Copy link
Author

p6rt commented Jan 30, 2012

@moritz - Status changed from 'open' to 'resolved'

@p6rt p6rt closed this as completed Jan 30, 2012
@p6rt p6rt added the testneeded label Jan 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant