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

Strange error on regex /<[..b]>/ in Rakudo #839

Closed
p6rt opened this issue Mar 26, 2009 · 5 comments
Closed

Strange error on regex /<[..b]>/ in Rakudo #839

p6rt opened this issue Mar 26, 2009 · 5 comments

Comments

@p6rt
Copy link

p6rt commented Mar 26, 2009

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

Searchable as RT64220$

@p6rt
Copy link
Author

p6rt commented Mar 26, 2009

From @masak

<masak`> rakudo​: /<[..b]>/
<p6eval> rakudo 7d9cd9​: OUTPUT«Cannot get character of empty string [...]
* masak` submits rakudobug

The error looks like some piece of code internal to PGE is assuming
that there'll be something before the '..' in the regex.

@p6rt
Copy link
Author

p6rt commented Jul 30, 2009

From @kyleha

This is an automatically generated mail to inform you that tests are now available in t/spec/S05-metasyntax/regex.t

commit f5dc1ad05d028848c9c30182166125a1db470604
Author​: kyle <kyle@​c213334d-75ef-0310-aa23-eaa082d1ae64>
Date​: Thu Jul 30 03​:37​:49 2009 +0000

  [t/spec] Test for RT #​64220
 
  git-svn-id​: http://svn.pugscode.org/pugs@&#8203;27811 c213334d-75ef-0310-aa23-eaa082d1ae64

Inline Patch
diff --git a/t/spec/S05-metasyntax/regex.t b/t/spec/S05-metasyntax/regex.t
index d332d75..979bcff 100644
--- a/t/spec/S05-metasyntax/regex.t
+++ b/t/spec/S05-metasyntax/regex.t
@@ -1,7 +1,7 @@
 use v6;
 use Test;
 
-plan 26;
+plan 27;
 
 # L<S05/Regexes are now first-class language, not strings>
 
@@ -94,3 +94,6 @@ ok ';' ~~ /\;/,             'escaped ";" in m// works';
     ok  $!  ~~ Exception, 'use of missing named rule dies';
     ok "$!" ~~ /nosuchrule/, 'error message mentions the missing rule';
 }
+
+#?rakudo todo 'RT #64220'
+eval_lives_ok '/<[..b]>/', '/<[..b]>/ lives';

@p6rt
Copy link
Author

p6rt commented Jul 30, 2009

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

@p6rt
Copy link
Author

p6rt commented Apr 2, 2010

From @moritz

Now passes tests

@p6rt
Copy link
Author

p6rt commented Apr 2, 2010

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

@p6rt p6rt closed this as completed Apr 2, 2010
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant